-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[RHDEVDOCS-2386]: Jenkins to Tekton Migration Guide #34792
[RHDEVDOCS-2386]: Jenkins to Tekton Migration Guide #34792
Conversation
✔️ Deploy Preview for osdocs ready! 🔨 Explore the source changes: e404bf2 🔍 Inspect the deploy log: https://app.netlify.com/sites/osdocs/deploys/610193e592711f000843fc4c 😎 Browse the preview: https://deploy-preview-34792--osdocs.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
modules/jt-extending-tekton-using-custom-tasks-and-scripts.adoc
Outdated
Show resolved
Hide resolved
modules/jt-migrating-a-sample-pipeline-from-jenkins-to-tekton.adoc
Outdated
Show resolved
Hide resolved
modules/jt-migrating-from-jenkins-plugins-to-tekton-hub-tasks.adoc
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
I left optional suggestion for rephrasing. Almost all of which can be saved for a later PR and iteration if this PR needs to get merged ASAP.
Great work!
@jc-berger: changing LGTM is restricted to collaborators In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
New changes are detected. LGTM label has been removed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sounix000 Amazing work! Left some comments and suggestions. PTAL. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
@jc-berger: changing LGTM is restricted to collaborators In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Jenkins to Tekton migration guide content Resolved Pavol's comments Refactoring into modules Fixed file-not-found error fixed errors pointed out by Preeti Fixed wrong file name Fixed typos Peer review comments from Jake Incorporated Sri's comments
/cherrypick enterprise-4.8 |
/cherrypick enterprise-4.9 |
@Preeticp: new pull request created: #34956 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@Preeticp: new pull request created: #34957 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Aligned team: Dev Tools
OCP version for cherry-picking:
4.8
,4.9
JIRA issues:
Epic: Document Jenkins to Tekton migration guide
NOTE: Content for the following JIRA issues are not included in this PR, and will be added afterwards:
- Document basic common use-cases
- Dealing with Jenkins shared library in Tekton
- Mapping Pipeline CRD structure to Jenkinsfile structure
Preview pages: Migrating from Jenkins to Tekton
Reviewer: Rupali Behera
Peer Review: Preeti Chandrasekhar