Skip to content

Update to rabbitmq cluster-operator v2.11.0#405

Closed
steveb wants to merge 1 commit into
openstack-k8s-operators:mainfrom
steveb:rabbitmq
Closed

Update to rabbitmq cluster-operator v2.11.0#405
steveb wants to merge 1 commit into
openstack-k8s-operators:mainfrom
steveb:rabbitmq

Conversation

@steveb

@steveb steveb commented Jun 19, 2025

Copy link
Copy Markdown

This is the version which other operators have when this is an indirect dependency [1].

[1] https://github.com/openstack-k8s-operators/openstack-operator/blob/main/apis/go.mod#L86

@openshift-ci openshift-ci Bot requested review from abays and viroel June 19, 2025 04:49
@openshift-ci

openshift-ci Bot commented Jun 19, 2025

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: steveb
Once this PR has been reviewed and has the lgtm label, please assign dprince for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

This is the version which other operators have when this is an indirect
dependency [1].

[1] https://github.com/openstack-k8s-operators/openstack-operator/blob/main/apis/go.mod#L86
@openshift-merge-robot

Copy link
Copy Markdown
Contributor

PR needs rebase.

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.

Comment thread go.mod
github.com/openstack-k8s-operators/lib-common/modules/common v0.6.1-0.20250508141203-be026d3164f7
github.com/openstack-k8s-operators/lib-common/modules/test v0.6.1-0.20250508141203-be026d3164f7
github.com/rabbitmq/cluster-operator/v2 v2.9.0
github.com/rabbitmq/cluster-operator/v2 v2.11.0

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.

This should be v2.9.0, we use a fork of the upstream operator. Should be fixed in whichever operator depends on the wrong version

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.

the replace bellow will override to whatever we bump it here. I think we could do this change, but in general it won't have any impact.

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.

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.

we also have https://github.com/openstack-k8s-operators/renovate-config/blob/main/default.json5#L155 to keep it bellow 2.10. a1dc0fb16970a819eb1e7c818754c4c280b0a5d5 in the openstack-operator which bumped it to 2.11 was landed before that.

@openshift-ci

openshift-ci Bot commented Jul 18, 2025

Copy link
Copy Markdown
Contributor

@steveb: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/precommit-check 9003801 link true /test precommit-check
ci/prow/infra-operator-build-deploy-kuttl 9003801 link true /test infra-operator-build-deploy-kuttl
ci/prow/functional 9003801 link true /test functional

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

@stuggi

stuggi commented Oct 6, 2025

Copy link
Copy Markdown
Contributor

closing this PR, we recently bumped to v2.16.0

@stuggi stuggi closed this Oct 6, 2025
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.

4 participants