Skip to content

background-worker: Use regex to list queues for backend-notifications#4351

Merged
akshaymankar merged 1 commit intodevelopfrom
backend-notif-pusher-filter
Nov 28, 2024
Merged

background-worker: Use regex to list queues for backend-notifications#4351
akshaymankar merged 1 commit intodevelopfrom
backend-notif-pusher-filter

Conversation

@akshaymankar
Copy link
Member

With consumable-notifications we expect a lot of queues to exist in RabbitMQ. Using regex to filter here would ensure this loop doesn't keep going over all of those queues over and over again.

https://wearezeta.atlassian.net/browse/WPB-11811

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

With consumable-notifications we expect a lot of queues to exist in RabbitMQ.
Using regex to filter here would ensure this loop doesn't keep going over all of
those queues over and over again.
@echoes-hq echoes-hq bot added the echoes/initiative: scale Enterprise Readiness Initiatives label Nov 27, 2024
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Nov 27, 2024
@akshaymankar akshaymankar merged commit 9cce5ad into develop Nov 28, 2024
@akshaymankar akshaymankar deleted the backend-notif-pusher-filter branch November 28, 2024 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

echoes/initiative: scale Enterprise Readiness Initiatives ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants