Skip to content

Add z stream and y stream instructions to user upgrade docs. - #787

Open
aidenfine wants to merge 3 commits into
theforeman:masterfrom
aidenfine:z-stream-upgrade-doc
Open

Add z stream and y stream instructions to user upgrade docs.#787
aidenfine wants to merge 3 commits into
theforeman:masterfrom
aidenfine:z-stream-upgrade-doc

Conversation

@aidenfine

Copy link
Copy Markdown
Contributor

Why are you introducing these changes? (Problem description, related links)

Old user docs did not include correct steps for Z stream upgrades. I also included the optional foremanctl pull-images step in Y stream upgrade that could reduce downtown during upgrade.

What are the changes introduced in this pull request?

  • y stream and z stream upgrade doc changes

Comment thread docs/user/upgrade.md Outdated
- This deploy command will pull new images and run all upgrade jobs required by Foreman, its dependencies, and your configured plugins. Expect this deploy to take longer than typical deploys.

# Z Stream
## Upgrading foremanctl from RPM install (Z stream)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ewoud would like us to use "update" for the Z-stream changes.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Comment thread docs/user/upgrade.md Outdated
Comment on lines +9 to +11
# Y Stream

## Upgrading foremanctl from RPM install (Y Stream)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
# Y Stream
## Upgrading foremanctl from RPM install (Y Stream)
# Upgrading foremanctl Y Stream (ex. x.1 to x.2)
## Upgrading foremanctl from RPM install

This reads better to me. Feel free to ignore since it's just a style suggestion.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I like it will change it.

Comment thread docs/user/upgrade.md Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
2. Consider backing up your Foreman environment before upgrade. See [Backup and Restore](backup-restore.md).

This needs updated to the new doc name.

Comment thread docs/user/upgrade.md
4. Upgrade the foremanctl package:
- `dnf upgrade foremanctl`
5. Run upgrade tasks by re-deploying your Foreman environment:
5. (Optional) Pre-pull container images to reduce downtime during deploy:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is a great add. Love it.

Comment thread docs/user/upgrade.md Outdated
- This deploy command will pull new images (if not already pulled in the previous step) and run all upgrade jobs required by Foreman, its dependencies, and your configured plugins. Expect this deploy to take longer than typical deploys.

## Upgrading foremanctl from disconnected RPM install
## Upgrading foremanctl from disconnected RPM install (Y stream)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
## Upgrading foremanctl from disconnected RPM install (Y stream)
## Upgrading foremanctl from disconnected RPM install

Just a suggestion to match style tweak above.

Comment thread docs/user/upgrade.md Outdated
- This deploy command will restart services with the updated images and run all upgrade jobs required by Foreman, its dependencies, and your configured plugins. Expect this deploy to take longer than typical deploys.


## Upgrading foremanctl from disconnected RPM install (Z stream)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
## Upgrading foremanctl from disconnected RPM install (Z stream)
## Updating foremanctl from disconnected RPM install

Comment thread docs/user/upgrade.md Outdated
All steps must be run as root user.

1. On your disconnected environment, run `foremanctl health` to ensure your existing Foreman server is healthy. Correct any issues before continuing.
2. Consider backing up your Foreman environment before upgrade. See [Backup](backup.md).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One more place to fix the link, sorry. Also "upgrade" -> "update".

Comment thread docs/user/upgrade.md Outdated
5. Run upgrade tasks by re-deploying your Foreman environment:
- `foremanctl deploy`
- Please see [Parameters](parameters.md) for additional deployment options.
- This deploy command will restart services with the updated images and run all upgrade jobs required by Foreman, its dependencies, and your configured plugins. Expect this deploy to take longer than typical deploys.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I confirmed that "This deploy command will restart services" is inaccurate when the image tag does not change. I tested this and systemctl restart foreman.target works best as a step after the deploy since migrations run during the deploy and then the restart flips execution over to the new code on the new images.

Comment thread docs/user/upgrade.md
6. On a connected machine, pull required images and prepare them for transfer:
- `foremanctl pull-images`
- Confirm the correct images were downloaded by running `podman images`. All images from your previous-version disconnected environment should be present on the connected environment. If images are missing, ensure foremanctl features parameters are identical between machines.
- Run `podman save $(podman images --format "{{.Repository}}:{{.Tag}}" | tr '\n' ' ') -o <filename>.tar` to export all downloaded images as a tarball.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Aiden have you had any issues with this command? Someone ran into an issue with this downstream (no container tag fails it) but every time I've tarballed the podman images with this it's worked for me.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I have honestly not run it since I am working on this in parallel with a few other things I have been taking peoples word that it works currently.

I trust that if you and someone else mentioned it works its okay to put in here.

Comment thread docs/user/upgrade.md Outdated
11. Run upgrade tasks by re-deploying your Foreman environment:
- `foremanctl deploy`
- Please see [Parameters](parameters.md) for additional deployment options.
- This deploy command will restart services with the updated images and run all upgrade jobs required by Foreman, its dependencies, and your configured plugins. Expect this deploy to take longer than typical deploys.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you add the same additional step as above please? I've confirmed services do not restart as of today.

@aidenfine

aidenfine commented Aug 26, 2026

Copy link
Copy Markdown
Contributor Author

@qcjames53 Does running foremanctl deploy also not restart the services on the Y stream upgrades? I added the systemclt restart foreman.target step to the Z stream only

- Updated backup doc href due to filename change
- Small style changes on titles
- Add systemclt restart foreman.target to Z stream instructions
- Changed Z stream naming from 'upgrade' to 'update'
@qcjames53

Copy link
Copy Markdown
Contributor

@qcjames53 Does running foremanctl deploy also not restart the services on the Y stream upgrades? I added the systemclt restart foreman.target step to the Z stream only

They restart on y stream. The new image tag sets a flag to restart the foreman services on deploy.

@qcjames53 qcjames53 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Everything looks great to me! Thanks, Aiden.

Comment thread docs/user/upgrade.md Outdated
- Please see [Parameters](parameters.md) for additional deployment options.
- This deploy command will pull new images and run all upgrade jobs required by Foreman, its dependencies, and your configured plugins. Expect this deploy to take longer than typical deploys.

# Updating foremanctl Z Stream (x.y.1, x.y.2)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
# Updating foremanctl Z Stream (x.y.1, x.y.2)
# Updating foremanctl z-stream (x.y.1, x.y.2)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I would almost prefer a separate guide to re-enforce update vs upgrade like we do today in our documentation.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

^ 'y-stream' as well. Thanks Aiden.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I dont mind creating separate files, I will make a commit tomorrow related to this.

@aidenfine

Copy link
Copy Markdown
Contributor Author

Sorry guys got distracted with something else, just pushed the changes to separate the guides

@aidenfine

Copy link
Copy Markdown
Contributor Author

@ehelms How does this look now?
cc @qcjames53

@qcjames53 qcjames53 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks good to me still. Thanks for the update. 👍

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants