Skip to content

Bump cinder image tag - #2512

Draft
chouroukblizzard wants to merge 1 commit into
stackhpc/2025.1from
cinder_netapp_self_signed
Draft

Bump cinder image tag#2512
chouroukblizzard wants to merge 1 commit into
stackhpc/2025.1from
cinder_netapp_self_signed

Conversation

@chouroukblizzard

Copy link
Copy Markdown

This includes backport of cinder self signed certs patch stackhpc/cinder#153

This includes backport of cinder self signed certs patch stackhpc/cinder#153
@chouroukblizzard
chouroukblizzard requested a review from a team as a code owner August 13, 2026 15:46
@github-actions github-actions Bot added the waiting-review PR is waiting for a review label Aug 13, 2026
@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 371ff6c4-d58c-44b2-a652-6022a30e6c22

📥 Commits

Reviewing files that changed from the base of the PR and between bf2614a and c732be5.

📒 Files selected for processing (1)
  • etc/kayobe/kolla-image-tags.yml
📜 Recent review details
⏰ Context from checks skipped due to timeout. (6)
  • GitHub Check: stackhpc/check
  • GitHub Check: Tox docs with Python 3.12
  • GitHub Check: Tox pep8 with Python 3.12
  • GitHub Check: Tox releasenotes with Python 3.12
  • GitHub Check: Ansible 2.18 lint with Python 3.12
  • GitHub Check: Ansible 2.17 lint with Python 3.10
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2026-07-27T12:28:31.134Z
Learnt from: seunghun1ee
Repo: stackhpc/stackhpc-kayobe-config PR: 2454
File: etc/kayobe/pulp.yml:566-568
Timestamp: 2026-07-27T12:28:31.134Z
Learning: For `etc/kayobe/stackhpc.yml`, the configured Kolla-Ansible source `stackhpc/kolla-ansiblestackhpc/2026.1` no longer defines or deploys InfluxDB and does not contain `kolla_enable_influxdb`. InfluxDB should not be required in `etc/kayobe/pulp.yml` `stackhpc_pulp_images_kolla`; a remaining `kolla_enable_influxdb` setting in ci-builder configuration is obsolete and does not enable a deployable service.

Applied to files:

  • etc/kayobe/kolla-image-tags.yml
🔇 Additional comments (1)
etc/kayobe/kolla-image-tags.yml (1)

10-13: LGTM!


📝 Walkthrough

Updates the Cinder image tags for Rocky 9, Rocky 10, and Ubuntu Noble deployments. The new tags include the backported self-signed certificates patch from stackhpc/cinder#153.

Walkthrough

The image tag configuration adds cinder mappings for Rocky 9, Rocky 10, and Ubuntu Noble deployments.

Changes

Cinder image tags

Layer / File(s) Summary
Add Cinder image tag mappings
etc/kayobe/kolla-image-tags.yml
The configuration adds cinder image tag mappings for Rocky 9, Rocky 10, and Ubuntu Noble.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Mergeability Score: ⚪ Minimal · up to c732b

The PR updates the Cinder image tag to include self-signed certificate support, and no actionable merge-blocking risk remains beyond normal checks and review.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

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.

@chouroukblizzard
chouroukblizzard marked this pull request as draft August 13, 2026 15:47
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.

1 participant