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

Update the Helm procedure to use oc-mirror for air-gapped OCP environments #868

Conversation

Fortune-Ndlovu
Copy link
Member

@Fortune-Ndlovu Fortune-Ndlovu commented Jan 21, 2025

IMPORTANT: Do Not Merge - To be merged by Docs Team Only

Version(s):

This PR updates the air-gapped installation documentation for Helm using oc-mirror. The key updates include:

  • Adding detailed steps for:
  • Creating the ImageSetConfiguration file.
  • Running the mirroring process.
  • Transferring and extracting the archive in an air-gapped environment.
  • Installing and verifying the Helm chart.

Curious about the visual output take a look at this gist of the update: https://gist.github.com/Fortune-Ndlovu/8b93a529ae6e79e7d79bf256d078bf38

Issue:
Related Issue: https://issues.redhat.com/browse/RHIDP-5351

Issue where the testing was verified: https://issues.redhat.com/browse/RHIDP-1723

Preview
Deep link: https://redhat-developer.github.io/red-hat-developers-documentation-rhdh/pr-868/install-rhdh-air-gapped/#proc-install-rhdh-airgapped-environment-ocp-helm_title-install-rhdh-air-grapped

@rhdh-bot
Copy link
Collaborator

rhdh-bot commented Jan 21, 2025

@Fortune-Ndlovu
Copy link
Member Author

Closing PR because docs member is already working on this PR update

@Fortune-Ndlovu
Copy link
Member Author

Fortune-Ndlovu commented Feb 25, 2025

I have just committed the single file update (proc-install-rhdh-airgapped-environment-ocp-helm.adoc) with the correct message and co-author you folks.
What was done:

  • Stashed the file update to avoid conflict
  • Reset the branch to upstream/main to start fresh
  • Popped the stashed changes back in
  • Committed with the proper message and authorship
  • Now we just need to force push with lease to sync everything

…vironments

Co-authored-by: Lindsey Frazier <[email protected]>
Co-authored-by: Armel Soro <[email protected]>
@Fortune-Ndlovu Fortune-Ndlovu force-pushed the RHIDP-5351-oc-mirror-helm-procedure-update branch from 7d21f55 to c4724c2 Compare February 26, 2025 12:35
@openshift-cherrypick-robot
Copy link
Contributor

@Fortune-Ndlovu: only redhat-developer org members may request cherry picks. If you are already part of the org, make sure to change your membership to public. Otherwise you can still do the cherry-pick manually.

In response to this:

/cherry-pick release-1.5

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.

@openshift-cherrypick-robot
Copy link
Contributor

@Fortune-Ndlovu: once the present PR merges, I will cherry-pick it on top of release-1.5 in a new PR and assign it to you.

In response to this:

/cherry-pick release-1.5

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.

@linfraze
Copy link
Member

/cherry-pick release-1.5

@openshift-cherrypick-robot
Copy link
Contributor

@linfraze: once the present PR merges, I will cherry-pick it on top of release-1.5 in a new PR and assign it to you.

In response to this:

/cherry-pick release-1.5

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.

Copy link
Member

@linfraze linfraze left a comment

Choose a reason for hiding this comment

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

This PR has been co-authored and reviewed

@linfraze linfraze added Ready to merge 💂 Technical review done ⛅ Any procedure has been succesfully tested labels Feb 26, 2025
…cp-helm.adoc
@jmagak jmagak merged commit 7200a92 into redhat-developer:main Feb 27, 2025
3 checks passed
@openshift-cherrypick-robot
Copy link
Contributor

@linfraze: #868 failed to apply on top of branch "release-1.5":

Applying: fix: Update the Helm procedure to use oc-mirror for air-gapped OCP environments
Using index info to reconstruct a base tree...
M	modules/installation/proc-install-rhdh-airgapped-environment-ocp-helm.adoc
Falling back to patching base and 3-way merge...
Auto-merging modules/installation/proc-install-rhdh-airgapped-environment-ocp-helm.adoc
CONFLICT (content): Merge conflict in modules/installation/proc-install-rhdh-airgapped-environment-ocp-helm.adoc
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"
Patch failed at 0001 fix: Update the Helm procedure to use oc-mirror for air-gapped OCP environments

In response to this:

/cherry-pick release-1.5

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants