Skip to content

Bump cinder image tag to fix cinder NetApp ONTAP and self-signed certs - #2512

Merged
MoteHue merged 1 commit into
stackhpc/2025.1from
cinder_netapp_self_signed
Aug 20, 2026
Merged

Bump cinder image tag to fix cinder NetApp ONTAP and self-signed certs#2512
MoteHue merged 1 commit into
stackhpc/2025.1from
cinder_netapp_self_signed

Conversation

@chouroukblizzard

@chouroukblizzard chouroukblizzard commented Aug 13, 2026

Copy link
Copy Markdown

This fixes compatibility between Cinder and NetApp ONTAP when the storage backend is deployed using self-signed certificates.
stackhpc/cinder#153 has been merged and is included for the 2025.1 image build.

@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

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: ab334680-96e3-4847-903c-6bb7c98af032

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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
@chouroukblizzard
chouroukblizzard force-pushed the cinder_netapp_self_signed branch from c732be5 to d2071d8 Compare August 17, 2026 09:50
@chouroukblizzard chouroukblizzard changed the title Bump cinder image tag Bump cinder image tag to fix cinder NetApp ONTAP and self-signed certs Aug 18, 2026
@chouroukblizzard
chouroukblizzard marked this pull request as ready for review August 18, 2026 08:09
@chouroukblizzard
chouroukblizzard force-pushed the cinder_netapp_self_signed branch from d2071d8 to e456c9d Compare August 18, 2026 08:36
This includes backport of cinder self signed certs patch stackhpc/cinder#153

@MoteHue MoteHue 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.

LGTM, thanks!

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.

2 participants