Skip to content

Only purge the backend-notifications.* queues #4623

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from
Draft

Only purge the backend-notifications.* queues #4623

wants to merge 1 commit into from

Conversation

supersven
Copy link
Contributor

This cleans left-overs up and avoids a race-condition with the background-worker which may still be attached.

Checklist

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

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jun 19, 2025
@supersven
Add expiry policy on queue
8e4adf4 
This should delete the queue after it's unused for over 1 Minute.
@supersven supersven force-pushed the sventennie/avoid_background-worker_race-condition branch from 4c9b5e2 to 8e4adf4 Compare June 20, 2025 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@supersven @zebot