Skip to content

uni05epsilon - deploy rabbitmq-notifications from the start#712

Open
lmiccini wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
lmiccini:epsilon_notifications
Open

uni05epsilon - deploy rabbitmq-notifications from the start#712
lmiccini wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
lmiccini:epsilon_notifications

Conversation

@lmiccini
Copy link
Copy Markdown
Contributor

@lmiccini lmiccini commented Mar 9, 2026

The commit moves the rabbitmq-notifications RabbitMQ instance from being deployed in the post-Ceph stage to being deployed from the start (control-plane stage).

@lmiccini lmiccini requested a review from a team as a code owner March 9, 2026 11:39
@openshift-ci openshift-ci bot requested review from cjeanner and karelyatin March 9, 2026 11:39
@lmiccini lmiccini force-pushed the epsilon_notifications branch from 24dfa23 to 4907a1f Compare March 9, 2026 11:42
@fultonj
Copy link
Copy Markdown
Contributor

fultonj commented Mar 9, 2026

Please add a commit message that says something like the following (or feel free to re-write). Aside from that, this PR looks good to me.

The commit moves the rabbitmq-notifications RabbitMQ instance from being deployed in the post-Ceph stage to being deployed from the start (control-plane stage).

The commit moves the rabbitmq-notifications RabbitMQ instance from being deployed in the post-Ceph stage to being deployed from the start (control-plane stage).
@lmiccini
Copy link
Copy Markdown
Contributor Author

Sorry John, just dropped this in a rush to see if a ci job would be better off with such change, amending the commit msg.

@lmiccini lmiccini force-pushed the epsilon_notifications branch from 4907a1f to 7502afd Compare March 10, 2026 06:31
@fultonj
Copy link
Copy Markdown
Contributor

fultonj commented Mar 10, 2026

/approve
/lgtm

Copy link
Copy Markdown
Contributor

@abays abays left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

Needs final approval from @openstack-k8s-operators/ciops since it touches a uni job

@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Mar 12, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abays, fultonj, lmiccini

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

@fultonj fultonj added the depends-on-ciops Tag to help ci-ops automation catch changes which might break unijobs label Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved depends-on-ciops Tag to help ci-ops automation catch changes which might break unijobs lgtm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants