Skip to content

openstack-k8s-operators dependency bump branch: main - #2212

Merged
openshift-merge-bot[bot] merged 1 commit into
mainfrom
openstack-dependency-bump/main
Nov 5, 2026
Merged

openshift-merge-bot[bot] merged 1 commit into
mainfrom
openstack-dependency-bump/main

Conversation

@github-actions

@github-actions github-actions Bot commented Nov 4, 2026

Copy link
Copy Markdown

Automated changes by create-pull-request GitHub action

@openshift-ci

openshift-ci Bot commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

Hi @github-actions[bot]. Thanks for your PR.

I'm waiting for a openstack-k8s-operators member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@coderabbitai

coderabbitai Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Summary

Summary by CodeRabbit

  • New Features
    • Added support for the namespaced PodRemediator resource, including controls for pausing or disabling remediation and configuring consent and polling intervals.
    • Added an optional Memcached instance setting for CloudKitty API and processor resources, defaulting to memcached.
  • Updates
    • Updated supported OpenStack operator versions and images.

The pull request adds a PodRemediator v1beta1 CRD and related manager permissions, adds Memcached instance fields to two CloudKitty CRDs, and updates operator API dependency versions and image digests.

Changes: Operator resources and release pins

Layer / File(s) Summary
PodRemediator schema and permissions
bindata/crds/remediation.openstack.org_podremediators.yaml, config/operator/bundle-rbac/infra-operator-manager-role.yaml
Adds the PodRemediator v1beta1 schema, including spec controls, status conditions, and namespace-scope validation. Updates manager permissions for core resources, PodRemediator resources, NodeHealthChecks, and self-node-remediation resources.
CloudKitty Memcached fields
bindata/crds/telemetry.openstack.org_cloudkittyapis.yaml, bindata/crds/telemetry.openstack.org_cloudkittyprocs.yaml
Adds the optional spec.memcachedInstance field, defaulting to memcached, to both CRD schemas.
Operator API and image pins
api/go.mod, go.mod, config/operator/manager_operator_images.yaml, hack/export_operator_related_images.sh
Updates pinned API dependency versions and manager image digests for the listed operators.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Change: Other

Merge Risk: Minimal · up to 4b06e

No actionable merge-blocking issue is established; the namespace-authorization concern remains conditional on permissions and controller behavior not shown here.

Pre-merge checks: Passed 5
Check name Status Explanation
Title check Passed The title "openstack-k8s-operators dependency bump branch: main" names the dependency-bump nature of the change and the target repository family. The changeset updates pinned pseudo-versions of openst…
Description check Passed The description states the changes were created by the create-pull-request GitHub Action. This relates to the changeset, which is an automated dependency update. It does not describe specific files, b…
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
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: Title check

Explanation

The title "openstack-k8s-operators dependency bump branch: main" names the dependency-bump nature of the change and the target repository family. The changeset updates pinned pseudo-versions of openstack-k8s-operators API dependencies (Barbican, Infra, Manila, Neutron, Nova, Telemetry, Test, Watcher) in go.mod files and related image references. The title is partially related: it describes the type of change but not the specific modules, and it includes a branch name fragment rather than a clear summary. It is not off-topic, so it does not fail.

Full details: Description check

Explanation

The description states the changes were created by the create-pull-request GitHub Action. This relates to the changeset, which is an automated dependency update. It does not describe specific files, but the check is lenient and only requires that the description is not off-topic.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 1 files. (7 skipped: 7 unsupported.)

Finishing Touches
Generate docstrings
  • Commit to this branch
  • Create a new PR
Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Central YAML (base), Organization UI (inherited)
  • Review profile: CHILL
  • Plan: Enterprise
  • Run ID: 8771230c-8d60-4205-a38a-385fbbef28ad
Commits

Reviewing files that changed from the base of the PR and between c3e4a8e and 4b06ee9.

Files ignored due to path filters: 2
  • api/go.sum is excluded by !**/*.sum
  • go.sum is excluded by !**/*.sum
Files selected for processing: 8
  • api/go.mod
  • bindata/crds/remediation.openstack.org_podremediators.yaml
  • bindata/crds/telemetry.openstack.org_cloudkittyapis.yaml
  • bindata/crds/telemetry.openstack.org_cloudkittyprocs.yaml
  • config/operator/bundle-rbac/infra-operator-manager-role.yaml
  • config/operator/manager_operator_images.yaml
  • go.mod
  • hack/export_operator_related_images.sh

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

@centosinfra-prod-github-app

Copy link
Copy Markdown

@jlarriba

jlarriba commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

I think this is needed to fix the kuttl tests that are not passing in my PR openstack-k8s-operators/telemetry-operator#1135 can we approve it?

@stuggi stuggi 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

@openshift-ci

openshift-ci Bot commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: github-actions[bot], stuggi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot added the approved label Nov 5, 2026
@openshift-merge-bot
openshift-merge-bot Bot merged commit c62fdaf into main Nov 5, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants