Skip to content

Build Rocky Linux 10 container images#2252

Open
owenjones wants to merge 1 commit intostackhpc/2025.1from
rl10/container-images
Open

Build Rocky Linux 10 container images#2252
owenjones wants to merge 1 commit intostackhpc/2025.1from
rl10/container-images

Conversation

@owenjones
Copy link
Copy Markdown
Member

@owenjones owenjones commented Apr 2, 2026

Changes:

  • Updated workflow to build Rocky 10 Kolla container images
  • Updated ci-builder environment to enable valkey image building on Rocky 10 (and enable redis image building on Rocky 9/Ubuntu Noble)
  • Updated ci-builder environment with pulp tags for Rocky 10 repositories
  • Added new OVN-LTS repository into pulp repository configuration
  • Added Rocky 10 repositories into Kolla configuration
  • Added new stackhpc_enable_ovn_lts_repo flag to Kolla configuration to inject OVN-LTS repository file
  • Updated Kolla build customisations to separate out Rocky 9 and 10 customisations, and added customisations for Rocky 10
  • Added valkey images to pulp image list, and updated unbuildable images lists to reflect

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 enables Epoxy container image builds on Rocky Linux 10 by adding the required repository configurations, version variables, and image customizations. It also introduces Valkey support for Rocky 10 while maintaining Redis for older distributions. Feedback highlights critical issues where YAML block scalars incorrectly convert lists into strings, which would break Jinja2 loops during the build process. Additionally, the review identifies shell variable escaping and indentation errors in a manually generated repository file, and suggests a more robust method for accessing version-specific customizations.

@owenjones owenjones force-pushed the rl10/container-images branch from c838aee to ad3c5bd Compare April 2, 2026 16:10
@owenjones owenjones force-pushed the rl10/container-images branch from ad3c5bd to 6823e3f Compare April 7, 2026 08:35
@owenjones owenjones force-pushed the rl10/container-images branch from 282db56 to b0c4a0a Compare April 7, 2026 12:39
@owenjones owenjones force-pushed the rl10/container-images branch from b0c4a0a to 00248f2 Compare April 8, 2026 08:43
Changes:
* Updated workflow to build Rocky 10 Kolla container images
* Updated ci-builder environment to enable valkey image building on
  Rocky 10 (and enable redis image building on Rocky 9/Ubuntu Noble)
* Updated ci-builder environment with pulp tags for Rocky 10
  repositories
* Added new OVN-LTS repository into pulp repository configuration
* Added Rocky 10 repositories into Kolla configuration
* Added new `stackhpc_enable_ovn_lts_repo` flag to Kolla configuration
  to inject OVN-LTS repository into apt sources
* Updated Kolla build customisations to separate out Rocky 9 and 10
  customisations, and added customisations for Rocky 10
* Added valkey images to pulp image list, and updated unbuildable
  images lists to reflect
@owenjones
Copy link
Copy Markdown
Member Author

owenjones commented Apr 8, 2026

Valkey images are buildable if specified in the workflow, but currently don't get fed into the list of default regexes even when kolla_enable_valkey is set so won't get otherwise built - upstream patch is currently going through the motions, will backport it once merged

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.

1 participant