Skip to content

Add support for Rocky Linux 9.8#2349

Open
priteau wants to merge 4 commits into
stackhpc/2025.1from
rl98
Open

Add support for Rocky Linux 9.8#2349
priteau wants to merge 4 commits into
stackhpc/2025.1from
rl98

Conversation

@priteau

@priteau priteau commented Jun 5, 2026

Copy link
Copy Markdown
Member

No description provided.

@priteau priteau requested a review from a team as a code owner June 5, 2026 09:15

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request updates various package repository versions, kolla image tags, and introduces support for Rocky Linux 9.8, including a temporary workaround in kolla-build.conf for missing 9.8 container images. Feedback points out an inconsistency in pulp.yml where the documentation comment states the default Rocky 9 minor version is 8, but the variable remains configured as '7'.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread etc/kayobe/pulp.yml Outdated
Comment on lines 211 to 212
# Rocky 9 minor version number. Supported values: 6, 7, 8. Default is 8.
stackhpc_pulp_repo_rocky_9_minor_version: '7'

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.

medium

The comment states that the default minor version is 8, but the variable stackhpc_pulp_repo_rocky_9_minor_version is still set to '7'. To align with the comment and default to Rocky Linux 9.8, the variable should be updated to '8'.

# Rocky 9 minor version number. Supported values: 6, 7, 8. Default is 8.
stackhpc_pulp_repo_rocky_9_minor_version: '8'
References
  1. When documenting supported values for a variable, ensure all currently supported values are listed to avoid confusion, even if a new default is introduced.

@Alex-Welsh Alex-Welsh Jun 5, 2026

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.

The bot seems to be correct

@Alex-Welsh Alex-Welsh left a comment

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.

Needs a reno

Comment thread etc/kayobe/pulp.yml Outdated
Comment on lines 211 to 212
# Rocky 9 minor version number. Supported values: 6, 7, 8. Default is 8.
stackhpc_pulp_repo_rocky_9_minor_version: '7'

@Alex-Welsh Alex-Welsh Jun 5, 2026

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.

The bot seems to be correct

Comment thread etc/kayobe/pulp-repo-versions.yml Outdated
Comment on lines +158 to +160
stackhpc_pulp_repo_ubuntu_cloud_archive_version: 20260603T234508
stackhpc_pulp_repo_ubuntu_noble_security_version: 20260604T023511
stackhpc_pulp_repo_ubuntu_noble_version: 20260604T023511

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.

Are you purposefully rebuilding ubuntu as well?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, because I bumped all third-party repositories too. And this way we are sure the OpenStack code is the same across all distributions.

@github-actions github-actions Bot added the waiting-author-response PR is waiting for the author to respond label Jun 5, 2026
@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Happy Friday @priteau, this is a friendly reminder that this PR is waiting for your changes or response. Please take a look when you have a moment!

Note: Once your changes are ready, remove the waiting-author-response label and add the waiting-review label.

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

Labels

waiting-author-response PR is waiting for the author to respond

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants