Skip to content

Conversation

@amolnar-gh
Copy link
Contributor

@amolnar-gh amolnar-gh commented Apr 2, 2024

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Apr 2, 2024
@openshift-ci-robot
Copy link

openshift-ci-robot commented Apr 2, 2024

@amolnar-rh: This pull request references TELCODOCS-1707 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.16.0" version, but no target version was set.

In response to this:

Version(s): 4.16

Issue: https://issues.redhat.com/browse/TELCODOCS-1707

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Apr 2, 2024
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Apr 2, 2024

@openshift-ci-robot
Copy link

openshift-ci-robot commented Apr 2, 2024

@amolnar-rh: This pull request references TELCODOCS-1707 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.16.0" version, but no target version was set.

In response to this:

Version(s): 4.16

Issue: https://issues.redhat.com/browse/TELCODOCS-1707

Link to docs preview: https://74025--ocpdocs-pr.netlify.app/openshift-enterprise/latest/scalability_and_performance/ztp_far_edge/ztp-image-based-upgrade#ztp-image-based-upgrade-concept_ztp-image-based-upgrade

QE review:

  • QE has approved this change.

Additional information:

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Apr 2, 2024
@amolnar-gh amolnar-gh force-pushed the TELCODOCS-1707 branch 4 times, most recently from 8c13dad to ca358d4 Compare April 8, 2024 14:45
@openshift-ci openshift-ci bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Apr 9, 2024
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 11, 2024
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 11, 2024
@openshift-ci-robot
Copy link

openshift-ci-robot commented Apr 11, 2024

@amolnar-rh: This pull request references TELCODOCS-1707 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.16.0" version, but no target version was set.

In response to this:

Version(s): 4.16

Issue: https://issues.redhat.com/browse/TELCODOCS-1707

Link to docs preview: https://74025--ocpdocs-pr.netlify.app/openshift-enterprise/latest/edge_computing/ztp-image-based-upgrade

QE review:

  • QE has approved this change.

Additional information:

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 openshift-eng/jira-lifecycle-plugin repository.

@amolnar-gh amolnar-gh force-pushed the TELCODOCS-1707 branch 2 times, most recently from 6b7d1fa to 7398541 Compare April 11, 2024 14:31
@openshift-ci openshift-ci bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Apr 11, 2024
@amolnar-gh amolnar-gh force-pushed the TELCODOCS-1707 branch 2 times, most recently from 8855afd to 95a97c6 Compare April 12, 2024 13:07
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 17, 2024
@openshift-merge-robot
Copy link

PR needs rebase.

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-sigs/prow repository.

mcp: "master"
sourceFiles:
...
- fileName: OadpSecret.yaml

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- fileName: OadpSecret.yaml
- fileName: custom-crs/OadpSecret.yaml

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the original commit was fine (no need for "custom-crs") since we should have this in ZTP? (same for the other "custom-crs/" changes...we can revert them)

Please checkout @Missxiaoguo latest PR and just copy the examples from there openshift-kni/cnf-features-deploy#1945 ?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These source crs have now pushed to 4.16 ztp container, but we are good for now. We will need to update this a bit postGA when ready

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah good to know! (ignore my previous comment..."custom-crs/" is good in that case!)

Copy link

@browsell browsell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please leave at 500

@amolnar-gh
Copy link
Contributor Author

Please leave at 500

@browsell Ack, reverted the change

@openshift-ci
Copy link

openshift-ci bot commented Jun 18, 2024

@amolnar-rh: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/validate-portal 6979fb2 link true /test validate-portal
ci/prow/validate-asciidoc 6979fb2 link true /test validate-asciidoc
ci/prow/deploy-preview 6979fb2 link true /test deploy-preview

Full PR test history. Your PR dashboard.

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-sigs/prow repository. I understand the commands that are listed here.

@amolnar-gh amolnar-gh mentioned this pull request Jun 24, 2024
1 task
* `Upgrade`
* `Rollback` (Optional)
image::../images/696_OpenShift_Lifecycle_Agent_0624_0.png[Stages of the image-based upgrade]
Copy link
Contributor

@aireilly aireilly Jun 24, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add proper figure titles, eg.,

.Image Title
image::/path/to/file.png[alt text, width, height]

@amolnar-gh
Copy link
Contributor Author

amolnar-gh commented Jul 4, 2024

Closing this PR as the content has been merged in separate, smaller PRs for OCP 4.16.

Implementing remaining comments in the scope of this bug: https://issues.redhat.com/browse/TELCODOCS-1932
Related open PRs:

@amolnar-gh amolnar-gh closed this Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.