Update disconnected update procedure for containerization - #5229
Update disconnected update procedure for containerization#5229aneta-petrova wants to merge 2 commits into
Conversation
|
The PR preview for e67e7c1 is available at theforeman-foreman-documentation-preview-pr-5229.surge.sh The following output files are affected by this PR: |
f26f7cb to
66f5d55
Compare
| $ podman save $(podman images \ | ||
| --format "{{.Repository}}:{{.Tag}}" | tr '\n' ' ') \ | ||
| --output _My_Container_Images_.tar |
There was a problem hiding this comment.
This is failing for me due some of the images being untagged.
There was a problem hiding this comment.
I'll look into this. Concerning 😬
github-actions
Bot
added
Needs tech review
b564f9b to
0846a27
Compare
0846a27 to
b4e46d4
Compare
| * Back up your disconnected {ProjectServer}. | ||
| For more information, see {AdministeringDocURL}backing-up-{project-context}[Backing up {Project}] in _{AdministeringDocTitle}_. |
There was a problem hiding this comment.
One comment here, backup technically is not a requirement/pre req and upgrading can be done without it.
Maybe we could change the wording here or move it down to procedure?
For example in the foremanctl docs we used, "Consider backing up your Foreman environment before upgrade."
There was a problem hiding this comment.
Thanks! Applied here and in the connected update procedure.
| You will be exporting the following repositories: | ||
| + | ||
| -- | ||
| * `Red Hat Enterprise Linux 9 for x86_64 - BaseOS RPMs 9` |
There was a problem hiding this comment.
@aidenfine There's one question that occurred to me: Here we are listing the three repositories required for the update, but the (already merged and containerized) procedure for updating a connected server doesn't include these repositories: Updating Satellite Server.
Is this an oversight? Should the connected update procedure also ask users to synchronize these repositories?
There was a problem hiding this comment.
That is a good point. Thanks for bringing this up, there is an updated PR on the foremanctl docs side, based on the new information on Y stream updates. I think that would be a good place to include this. I will add that into the PR.
The PR is here theforeman/foremanctl#787
3 participants
What changes are you introducing?
Adding a procedure for updating a disconnected and containerized server.
Why are you introducing these changes? (Explanation, links to references, issues, etc.)
theforeman/foremanctl#621
https://redhat.atlassian.net/browse/SAT-48361
Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)
N/A
Contributor checklists
Please cherry-pick my commits into: