Skip to content

feat: bump magnum-capi-helm to v1.4.0 - #2657

Open
jackhodgkiss wants to merge 1 commit into
stackhpc/2026.1from
bump-magnum-capi-helm
Open

jackhodgkiss wants to merge 1 commit into
stackhpc/2026.1from
bump-magnum-capi-helm

Conversation

@jackhodgkiss

Copy link
Copy Markdown
Contributor

Update magnum-capi-helm to v1.4.0 which resolves an issue with a tenant cluster being incorrectly reported as unhealthy.

@coderabbitai

coderabbitai Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

📝 Walkthrough

The configuration adds Magnum image tags for Rocky 10 and Ubuntu Noble, updates the Magnum CAPI Helm driver pin to v1.4.0, and adds a release note about the upgrade and a tenant-cluster health fix.

Changes: Magnum configuration and release note

Layer / File(s) Summary
Update Magnum image and driver versions
etc/kayobe/kolla.yml, etc/kayobe/kolla-image-tags.yml, releasenotes/notes/*
The configuration pins magnum-capi-helm to v1.4.0 and adds Magnum image tags for Rocky 10 and Ubuntu Noble. The release note records the upgrade and a fix for tenant clusters reported as unhealthy.

Priority: ➖ Normal

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: 🟡 Moderate · up to 4044f

Promote both Magnum tags before merging and deploying this update; otherwise production image sync or the Magnum rollout can fail because the tags are missing.

Pre-merge checks: Passed 3
Check name Status Explanation
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (3 skipped: 3 unsupported.)

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added the waiting-review PR is waiting for a review label Oct 7, 2026
@jackhodgkiss

Copy link
Copy Markdown
Contributor Author

@jackhodgkiss
jackhodgkiss force-pushed the bump-magnum-capi-helm branch from 2448a82 to ae123e0 Compare October 8, 2026 10:17
@Alex-Welsh
Alex-Welsh force-pushed the bump-magnum-capi-helm branch from ae123e0 to 0bfe69d Compare October 12, 2026 09:35
@Alex-Welsh

Alex-Welsh commented Oct 12, 2026 •

Copy link
Copy Markdown
Member

rebased re-rebuild build is building http://github.com/stackhpc/stackhpc-kayobe-config/actions/runs/38230957685

@Alex-Welsh Alex-Welsh added waiting-author-response PR is waiting for the author to respond and removed waiting-review PR is waiting for a review labels Oct 12, 2026
@jackhodgkiss
jackhodgkiss force-pushed the bump-magnum-capi-helm branch 3 times, most recently from a868426 to 4201867 Compare October 16, 2026 12:46
@jackhodgkiss
jackhodgkiss force-pushed the bump-magnum-capi-helm branch from 4201867 to a9e1fcb Compare October 16, 2026 13:40
@jackhodgkiss
jackhodgkiss marked this pull request as ready for review October 16, 2026 13:40
@jackhodgkiss
jackhodgkiss requested a review from a team as a code owner October 16, 2026 13:40
@github-actions github-actions Bot added waiting-review PR is waiting for a review and removed waiting-author-response PR is waiting for the author to respond labels Oct 16, 2026
@priteau
priteau enabled auto-merge October 16, 2026 14:01
Update `magnum-capi-helm` to `v1.4.0` which resolves an issue with a
tenant cluster being incorrectly reported as `unhealthy`.

Signed-off-by: Jack Hodgkiss <jack@stackhpc.com>

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1


ℹ️ Review info
⚙️ Run configuration
  • Configuration used: Repository: stackhpc/stackhpc-kayobe-config/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: e7df6b39-8a2f-494c-9b7b-2ebf569c5274
📥 Commits

Reviewing files that changed from the base of the PR and between fe4e839 and 4044fa2.

📒 Files selected for processing: 3
  • etc/kayobe/kolla-image-tags.yml
  • etc/kayobe/kolla.yml
  • releasenotes/notes/bump-magnum-capi-helm-1a6bd663a5c397ee.yaml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Comment on lines +26 to +28
magnum:
rocky-10: 2026.1-rocky-10-20261016T131951
ubuntu-noble: 2026.1-ubuntu-noble-20261016T131951

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🩺 Stability & Availability | 🟠 Major | ⚡ Quick win

Promote both Magnum tags before merging this mapping.

The observed promotion used the base mapping, which has no magnum entry. It therefore did not copy these tags from stackhpc-dev to stackhpc. The production tag check rejects missing tags, and Magnum deployment can fail if it runs before a later promotion. Merging only queues an asynchronous promotion and does not gate production sync or deployment. Promote both tags before merging this change.

This branch was successfully deployed

1 active deployment
SMS Lab — 4044fa24 Deployed Oct 20, 2026 by jackhodgkiss via aio (Rocky 10 aarch64 OVN) / All in one #8217
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting-review PR is waiting for a review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants