Skip to content

Conversation

@morenod
Copy link
Contributor

@morenod morenod commented Oct 6, 2025

Move rabbitmq notifications queue from enabling at nova/cinder/watcher level to openstack controlplane level after that usage is available since openstack-k8s-operators/openstack-operator#1591.

delayStartSeconds parameter removed from rabbitmq notifications queue creation because it is not necessary, it is automatically added by controlplane when creating a new queue

@morenod morenod force-pushed the global_notifications branch from 45a54c3 to 1437713 Compare October 6, 2025 15:20
@softwarefactory-project-zuul
Copy link

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/b79c31cff8fb479caf80a059fe7823bc

✔️ openstack-meta-content-provider-master SUCCESS in 2h 34m 14s
✔️ watcher-operator-validation-master SUCCESS in 1h 53m 39s (non-voting)
✔️ openstack-meta-content-provider-epoxy SUCCESS in 2h 53m 41s
✔️ watcher-operator-validation-epoxy SUCCESS in 1h 54m 50s
watcher-operator-validation-epoxy-ocp4-16 FAILURE in 20m 07s
✔️ noop SUCCESS in 0s
✔️ watcher-operator-kuttl SUCCESS in 43m 57s

@morenod
Copy link
Contributor Author

morenod commented Oct 6, 2025

check-rdo

@softwarefactory-project-zuul
Copy link

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/f9416ac601264548b137ce1f44fe627c

✔️ openstack-meta-content-provider-master SUCCESS in 2h 40m 26s
✔️ watcher-operator-validation-master SUCCESS in 1h 56m 22s (non-voting)
✔️ openstack-meta-content-provider-epoxy SUCCESS in 2h 31m 17s
✔️ watcher-operator-validation-epoxy SUCCESS in 1h 51m 30s
watcher-operator-validation-epoxy-ocp4-16 FAILURE in 19m 19s
✔️ noop SUCCESS in 0s
✔️ watcher-operator-kuttl SUCCESS in 41m 38s

@morenod
Copy link
Contributor Author

morenod commented Oct 7, 2025

check-rdo

@morenod
Copy link
Contributor Author

morenod commented Oct 7, 2025

CI Results:

  • watcher-operator-validation-master (notifications enabled):

  • Notifications examples on decision engine:
    2025-10-07 10:24:06.260 1 INFO watcher.decision_engine.model.notification.nova [None req-c440e984-43d6-442f-835a-3546a3fc11c1 92b444a9c39a48a68af149e6c12b4cd6 2a13312658fd42d787f39d6ba1cdda55 - - - -] Event 'instance.create.start' received from nova-compute:compute-0.ctlplane.example.com with metadata {'message_id': 'b779ffa6-791a-4b11-a238-7ffbbf1b5a59', 'timestamp': '2025-10-07 10:24:06.249984'}

@cescgina
Copy link
Contributor

cescgina commented Oct 7, 2025

/lgtm

I think this is good to go. @morenod not sure the openshift-ci bot added the do-not-merge label, maybe it's because you have [WIP] in the PR title

@openshift-ci openshift-ci bot added the lgtm label Oct 7, 2025
@morenod morenod changed the title [WIP] Move rabbitmq notifications queue to controlplane level Move rabbitmq notifications queue to controlplane level Oct 7, 2025
@amoralej
Copy link
Contributor

amoralej commented Oct 8, 2025

/approve

@openshift-ci
Copy link

openshift-ci bot commented Oct 8, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: amoralej

The full list of commands accepted by this bot can be found here.

The pull request process is described here

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 Oct 8, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit 1ca6ebb into openstack-k8s-operators:main Oct 8, 2025
6 checks passed
@morenod morenod deleted the global_notifications branch October 8, 2025 10:42
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.

3 participants