Skip to content

Add version label to rabbitmq cluster#1805

Open
lmiccini wants to merge 3 commits intoopenstack-k8s-operators:mainfrom
lmiccini:rabbitmqversion_label
Open

Add version label to rabbitmq cluster#1805
lmiccini wants to merge 3 commits intoopenstack-k8s-operators:mainfrom
lmiccini:rabbitmqversion_label

Conversation

@lmiccini
Copy link
Contributor

@lmiccini lmiccini commented Feb 11, 2026

This commit adds a RabbitmqVersion label to the rabbitmq cluster, so that it can be used to trigger skip-version upgrades, like from 3.9 to 4.X.

@openshift-ci openshift-ci bot requested review from rebtoor and stuggi February 11, 2026 05:52
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 11, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lmiccini
Once this PR has been reviewed and has the lgtm label, please assign rabi 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

@lmiccini lmiccini mentioned this pull request Feb 11, 2026
Closed
@lmiccini lmiccini force-pushed the rabbitmqversion_label branch 19 times, most recently from cf85737 to 8abe851 Compare February 13, 2026 08:49
@softwarefactory-project-zuul
Copy link

softwarefactory-project-zuul bot commented Feb 13, 2026

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/openstack-k8s-operators/openstack-operator for 1805,8abe8518dbfc05f47862e490d10adb8e79338a9a

@lmiccini lmiccini force-pushed the rabbitmqversion_label branch from 8abe851 to 389e2f0 Compare February 13, 2026 12:42
@lmiccini
Copy link
Contributor Author

lmiccini commented Feb 13, 2026

/test functional

@lmiccini lmiccini force-pushed the rabbitmqversion_label branch from 389e2f0 to 3abcc1c Compare February 13, 2026 15:13
@softwarefactory-project-zuul
Copy link

softwarefactory-project-zuul bot commented Feb 14, 2026

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/d2e2c37789874d3eb25ec653a4f131d7

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 08m 25s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 17m 41s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 35m 45s
✔️ adoption-standalone-to-crc-ceph-provider SUCCESS in 2h 54m 56s
❌ openstack-operator-tempest-multinode FAILURE in 1h 40m 08s

@lmiccini lmiccini force-pushed the rabbitmqversion_label branch 16 times, most recently from 1bd33b2 to a893725 Compare February 17, 2026 17:25
@softwarefactory-project-zuul
Copy link

softwarefactory-project-zuul bot commented Feb 17, 2026

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/7b1eeda307954020ac2639259fe4415b

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 13m 56s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 20m 08s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 44m 32s
❌ adoption-standalone-to-crc-ceph-provider FAILURE in 3h 00m 10s
✔️ openstack-operator-tempest-multinode SUCCESS in 1h 41m 33s

lmiccini added 2 commits March 4, 2026 17:05
@lmiccini
Add target version annotation to rabbitmq cluster
7d20a84 
This commit adds the rabbitmq.openstack.org/target-version annotation
to the RabbitMQ cluster resource, using the constant from the
infra-operator API package. This annotation is used by the infra-operator
to trigger version upgrades, including skip-version upgrades (e.g., from
3.9 to 4.x) that require storage wipe.

The annotation value is sourced from OpenStackVersion.Status.ServiceDefaults.RabbitmqVersion,
allowing the openstack-operator to declaratively control RabbitMQ version
upgrades through the OpenStackVersion resource.

Jira: https://issues.redhat.com/browse/OSPRH-21178
@lmiccini
TEMP replace rabbitmq image
d9dae9c
@lmiccini lmiccini force-pushed the rabbitmqversion_label branch from a893725 to d9dae9c Compare March 4, 2026 16:13
@github-actions
Copy link

github-actions bot commented Mar 4, 2026 •
edited
Loading

OpenStackControlPlane CRD Size Report

Metric Value
CRD JSON size 322062 bytes (315KB)
Base branch size 322062 bytes
Change +0.00%
Status yellow — growing
Threshold reference
Color Range Meaning
🟢 green < 300KB Comfortable
🟡 yellow 300–400KB Growing
🟠 orange 400–750KB Concerning
🔴 red > 750KB Approaching 1.5MB etcd limit (cut in half to allow space for update)

@lmiccini lmiccini force-pushed the rabbitmqversion_label branch 5 times, most recently from 5314303 to 2620c09 Compare March 5, 2026 11:27
@lmiccini lmiccini force-pushed the rabbitmqversion_label branch from 2620c09 to f390b73 Compare March 5, 2026 13:02
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 5, 2026

@lmiccini: The following test 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/openstack-operator-build-deploy-kuttl-4-18 f390b73 link true /test openstack-operator-build-deploy-kuttl-4-18

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rebtoor rebtoor Awaiting requested review from rebtoor

@stuggi stuggi Awaiting requested review from stuggi

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

do-not-merge/hold

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lmiccini @openshift-merge-robot