Skip to content
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

OSDOCS-5773: Adding and updating example oc-mirror image set configs #60093

Merged
merged 1 commit into from
Jun 28, 2023

Conversation

bergerhoffer
Copy link
Contributor

@bergerhoffer bergerhoffer commented May 16, 2023

Version(s):
4.13+

Issue:
https://issues.redhat.com/browse/OSDOCS-5773

Link to docs preview:
https://60093--docspreview.netlify.app/openshift-enterprise/latest/installing/disconnected_install/installing-mirroring-disconnected.html#oc-mirror-image-set-examples_installing-mirroring-disconnected

QE review:

  • QE has approved this change.

Additional information:
This will be backported to 4.12, but with changes to remove any 4.13-specific content (like targetCatalog).

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

openshift-ci-robot commented May 16, 2023

@bergerhoffer: This pull request references OSDOCS-5773 which is a valid jira issue.

In response to this:

Version(s):

Issue:

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 kubernetes/test-infra repository.

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 16, 2023
@openshift-ci-robot
Copy link

openshift-ci-robot commented May 16, 2023

@bergerhoffer: This pull request references OSDOCS-5773 which is a valid jira issue.

In response to this:

Version(s):
4.13+

Issue:
https://issues.redhat.com/browse/OSDOCS-5773

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 kubernetes/test-infra repository.

@bergerhoffer bergerhoffer added this to the Planned for 4.13 GA milestone May 16, 2023
@openshift-ci openshift-ci bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label May 16, 2023
@openshift-ci-robot
Copy link

openshift-ci-robot commented May 16, 2023

@bergerhoffer: This pull request references OSDOCS-5773 which is a valid jira issue.

In response to this:

Version(s):
4.13+

Issue:
https://issues.redhat.com/browse/OSDOCS-5773

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:
This will be backported to 4.12, but with changes to remove any 4.13-specific content (like targetCatalog).

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.

@openshift-ci-robot
Copy link

openshift-ci-robot commented May 16, 2023

@bergerhoffer: This pull request references OSDOCS-5773 which is a valid jira issue.

In response to this:

Version(s):
4.13+

Issue:
https://issues.redhat.com/browse/OSDOCS-5773

Link to docs preview:
http://file.rdu.redhat.com/~ahoffer/2023/OSDOCS-5773/installing/disconnected_install/installing-mirroring-disconnected.html#oc-mirror-image-set-examples_installing-mirroring-disconnected

QE review:

  • QE has approved this change.

Additional information:
This will be backported to 4.12, but with changes to remove any 4.13-specific content (like targetCatalog).

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.

@kalexand-rh
Copy link
Contributor

The branch/enterprise-4.14 label has been added to this PR.

This is because your PR targets the main branch and is labeled for enterprise-4.13. And any PR going into main must also target the latest version branch (enterprise-4.14).

If the update in your PR does NOT apply to version 4.14 onward, please re-target this PR to go directly into the appropriate version branch or branches (enterprise-4.x) instead of main.

@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented May 17, 2023

🤖 Updated build preview is available at:
https://60093--docspreview.netlify.app

Build log: https://circleci.com/gh/ocpdocs-previewbot/openshift-docs/18388

@openshift-ci-robot
Copy link

openshift-ci-robot commented May 17, 2023

@bergerhoffer: This pull request references OSDOCS-5773 which is a valid jira issue.

In response to this:

Version(s):
4.13+

Issue:
https://issues.redhat.com/browse/OSDOCS-5773

Link to docs preview:
https://60093--docspreview.netlify.app/openshift-enterprise/latest/installing/disconnected_install/installing-mirroring-disconnected.html#oc-mirror-image-set-examples_installing-mirroring-disconnected

QE review:

  • QE has approved this change.

Additional information:
This will be backported to 4.12, but with changes to remove any 4.13-specific content (like targetCatalog).

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.

Copy link

@sherine-k sherine-k left a comment

Choose a reason for hiding this comment

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

Hi @bergerhoffer
Thanks for this update!
I added just a small clarification for the heads only use-case

modules/oc-mirror-image-set-config-examples.adoc Outdated Show resolved Hide resolved
@bergerhoffer bergerhoffer force-pushed the OSDOCS-5773 branch 5 times, most recently from bf754d6 to a993373 Compare May 25, 2023 15:53
@bergerhoffer
Copy link
Contributor Author

@sherine-k @aguidirh I updated this to include two more examples:

  • having to provide the default channel
  • including the full catalog (not just channel heads).

Please take a look and let me know your feedback on these ones. Thanks!

Diff is here: https://github.com/openshift/openshift-docs/compare/cfe3fd8646b2f8714f3f57140d275e88108d8286..a993373fa92701359013008f475fd616a275bad1

Previews:

@bergerhoffer
Copy link
Contributor Author

/label peer-review-needed

@bergerhoffer bergerhoffer changed the title [WIP] OSDOCS-5773: Adding and updating example oc-mirror image set configs OSDOCS-5773: Adding and updating example oc-mirror image set configs May 25, 2023
@openshift-ci openshift-ci bot added peer-review-needed Signifies that the peer review team needs to review this PR and removed do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. labels May 25, 2023
Copy link

@sherine-k sherine-k left a comment

Choose a reason for hiding this comment

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

Thanks @bergerhoffer
This is going to help a lot, especially the default channel inclusion example!

@kcarmichael08
Copy link
Contributor

/label peer-review-in-progress

@openshift-ci openshift-ci bot added the peer-review-in-progress Signifies that the peer review team is reviewing this PR label May 26, 2023
Copy link
Contributor

@kcarmichael08 kcarmichael08 left a comment

Choose a reason for hiding this comment

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

Some minor suggestions and I think a typo; otherwise lgtm!

modules/oc-mirror-image-set-config-examples.adoc Outdated Show resolved Hide resolved
modules/oc-mirror-image-set-config-examples.adoc Outdated Show resolved Hide resolved
modules/oc-mirror-oci-format.adoc Show resolved Hide resolved
@kcarmichael08
Copy link
Contributor

/remove-label peer-review-in-progress
/remove-label peer-review-needed
/label peer-review-done

@openshift-ci openshift-ci bot added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-in-progress Signifies that the peer review team is reviewing this PR peer-review-needed Signifies that the peer review team needs to review this PR labels May 26, 2023
@bergerhoffer
Copy link
Contributor Author

Updated for 2 pieces of feedback, will wait for input from oc-mirror team on two open issues.

@bergerhoffer
Copy link
Contributor Author

@sherine-k @zhouying7780 I updated to use 'stable' for the channel of that example, and used 4.0.1 as the version. I did not make any changes to the first comment you made @zhouying7780.

Let me know how this looks now, thanks!

@bergerhoffer
Copy link
Contributor Author

@zhouying7780 Can you PTAL?

@zhouying7780
Copy link

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 28, 2023
@openshift-ci-robot
Copy link

openshift-ci-robot commented Jun 28, 2023

@bergerhoffer: This pull request references OSDOCS-5773 which is a valid jira issue.

In response to this:

Version(s):
4.13+

Issue:
https://issues.redhat.com/browse/OSDOCS-5773

Link to docs preview:
https://60093--docspreview.netlify.app/openshift-enterprise/latest/installing/disconnected_install/installing-mirroring-disconnected.html#oc-mirror-image-set-examples_installing-mirroring-disconnected

QE review:

  • QE has approved this change.

Additional information:
This will be backported to 4.12, but with changes to remove any 4.13-specific content (like targetCatalog).

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.

@bergerhoffer bergerhoffer merged commit 8894df2 into openshift:main Jun 28, 2023
1 check passed
@bergerhoffer
Copy link
Contributor Author

/cherrypick enterprise-4.13

@bergerhoffer
Copy link
Contributor Author

/cherrypick enterprise-4.14

@openshift-cherrypick-robot

@bergerhoffer: new pull request created: #61815

In response to this:

/cherrypick enterprise-4.13

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.

@openshift-cherrypick-robot

@bergerhoffer: new pull request created: #61816

In response to this:

/cherrypick enterprise-4.14

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch/enterprise-4.13 branch/enterprise-4.14 jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. peer-review-done Signifies that the peer review team has reviewed this PR size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants