Skip to content

[18.0-fr5] Return early when notification TransportURL is not ready#1109

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:18.0-fr5from
openshift-cherrypick-robot:cherry-pick-1082-to-18.0-fr5
May 4, 2026
Merged

[18.0-fr5] Return early when notification TransportURL is not ready#1109
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:18.0-fr5from
openshift-cherrypick-robot:cherry-pick-1082-to-18.0-fr5

Conversation

@openshift-cherrypick-robot
Copy link
Copy Markdown

@openshift-cherrypick-robot openshift-cherrypick-robot commented Apr 28, 2026

This is an automated cherry-pick of #1082

/assign gibizer

Resolves: OSPRH-26923

When a RabbitMQ notification pod is deleted or restarted, the transporturl status
changes from ready to others(creating).

This change make reconciler return early when rabbitmq status is
creating or failed, this prevents empty transporturl from being
propogated in configs.

Adds tests to verify same

Resolves: OSPRH-26269
Copy link
Copy Markdown
Contributor

@mrkisaolamb mrkisaolamb left a comment

Choose a reason for hiding this comment

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

lgtm

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 4, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mrkisaolamb, openshift-cherrypick-robot

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 May 4, 2026
@openshift-merge-bot openshift-merge-bot Bot merged commit fdb84e8 into openstack-k8s-operators:18.0-fr5 May 4, 2026
7 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.

4 participants