Skip to content

Conversation

@elelaysh
Copy link
Contributor

@elelaysh elelaysh commented Dec 5, 2025

  • upgrade stackhpc/2025.1 to Rocky Linux 9.7
  • DOCA 3.2.1 for Rocky Linux 9.7
  • all packages upgrade (docker, grafana, ...)

ansible/maintenance/rocky-97-ofed-upgrade.yml is an upgrade playbook
for sites using a local pulp server and doca.

See #2061 for container and overcloud host upgrade

@elelaysh elelaysh requested a review from a team as a code owner December 5, 2025 16:41
@elelaysh elelaysh marked this pull request as draft December 5, 2025 16:41
Copy link
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 adds support for Rocky Linux 9.7 by updating various configuration files. The changes include updating the default Rocky Linux minor version, adding new repository versions for 9.7, and adjusting DOCA version handling. My review focuses on improving maintainability and correcting potential errors in the configuration. I've pointed out a hardcoded branch reference that should be made more generic, a misleading comment, and a likely invalid CVE identifier in the Trivy allowlist.

@elelaysh elelaysh force-pushed the rl-9.7 branch 2 times, most recently from 31bc446 to 5a7be0c Compare December 8, 2025 09:08
elelaysh and others added 6 commits January 22, 2026 11:56
- DOCA 3.2.1 for RL 9.7
- Bump Rocky 9 Security SIG repo, add source
Remove DOCA 3.2.0: a single version is enough for RL 9.7
removes the RockyLinux minor version in the name and path
when DOCA version is greater than 3.2.0.
Doesn't apply to DOCA modules because they are still compiled
for a specific RL minor version.
Latest version for RockyLinux is 29.1.4,
Ubuntu noble has 27.5.1.
Tested on multinode.

Fix install-doca.yml to not install doca-ofed anymore (avoid dkms).

The stackhpc_doca_kernel_version_matrix variable contains kernel module
versions to install for last 2 supported minor RockyLinux versions.
It must be changed after a new pre-compiled kernel module version has been built.
@stackhpc-zuul
Copy link

stackhpc-zuul bot commented Jan 22, 2026

Starting check jobs.
Warning:
Failed to create check run stackhpc/check: 401 Client Error: Unauthorized for url: https://api.github.com/repos/stackhpc/stackhpc-kayobe-config/check-runs

@priteau priteau enabled auto-merge January 22, 2026 14:33
@priteau priteau disabled auto-merge January 22, 2026 14:33
@priteau priteau dismissed Alex-Welsh’s stale review January 22, 2026 14:34

Alex is happy!

@priteau priteau enabled auto-merge January 22, 2026 14:34
@priteau priteau disabled auto-merge January 22, 2026 14:34
@priteau priteau merged commit 197c19a into stackhpc/2025.1 Jan 22, 2026
20 of 22 checks passed
@priteau priteau deleted the rl-9.7 branch January 22, 2026 14:34
@elelaysh
Copy link
Contributor Author

thanks for the merge 👍

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.

6 participants