Skip to content

Support skipping Rocky Linux version check#2201

Merged
Alex-Welsh merged 2 commits intostackhpc/2024.1from
rocky-97-ofed-upgrade-fix-caracal
Mar 9, 2026
Merged

Support skipping Rocky Linux version check#2201
Alex-Welsh merged 2 commits intostackhpc/2024.1from
rocky-97-ofed-upgrade-fix-caracal

Conversation

@priteau
Copy link
Copy Markdown
Member

@priteau priteau commented Mar 8, 2026

This can be useful if the playbook has been interrupted before finishing, but the rocky-release package is already updated to 9.7.

(cherry picked from commit 0640fd1)

This can be useful if the playbook has been interrupted before
finishing, but the rocky-release package is already updated to 9.7.

(cherry picked from commit 0640fd1)
@priteau priteau requested a review from elelaysh March 8, 2026 22:18
@priteau priteau self-assigned this Mar 8, 2026
@priteau priteau requested a review from a team as a code owner March 8, 2026 22:18
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a skip_rocky_96_version_check variable to bypass the initial OS version check in the Rocky Linux 9.7 upgrade playbook. This is useful for re-running an interrupted playbook. My review includes a suggestion to improve the robustness of this check by modifying it to explicitly allow either version 9.6 for initial runs or 9.7 for re-runs, preventing accidental runs on unsupported OS versions. This approach aligns with defining configuration variables with sensible defaults for override.

Copy link
Copy Markdown
Contributor

@elelaysh elelaysh left a comment

Choose a reason for hiding this comment

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

LGTM

@Alex-Welsh Alex-Welsh merged commit 0bdb4b5 into stackhpc/2024.1 Mar 9, 2026
23 checks passed
@Alex-Welsh Alex-Welsh deleted the rocky-97-ofed-upgrade-fix-caracal branch March 9, 2026 11:57
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.

3 participants