Skip to content

uni05epsilon - deploy rabbitmq-notifications from the start#712

Merged
softwarefactory-project-zuul[bot] merged 1 commit intoopenstack-k8s-operators:mainfrom
lmiccini:epsilon_notifications
Apr 23, 2026
Merged

uni05epsilon - deploy rabbitmq-notifications from the start#712
softwarefactory-project-zuul[bot] merged 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).

@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

@fultonj fultonj added the depends-on-ciops Tag to help ci-ops automation catch changes which might break unijobs label Mar 23, 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).
@fultonj fultonj force-pushed the epsilon_notifications branch from 7502afd to ed7be15 Compare April 22, 2026 19:22
@openshift-ci openshift-ci Bot removed the lgtm label Apr 22, 2026
@adrianfusco
Copy link
Copy Markdown
Contributor

/approve
/lgtm

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Apr 23, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abays, adrianfusco, fultonj, kstrenkova, 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

@softwarefactory-project-zuul
Copy link
Copy Markdown
Contributor

@softwarefactory-project-zuul softwarefactory-project-zuul Bot merged commit a3d9ea5 into openstack-k8s-operators:main Apr 23, 2026
7 checks passed
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.

5 participants