Skip to content

background-worker/dead-user-notifs-watcher: Reconnect when connection with RabbitMQ is broken#4652

Merged
akshaymankar merged 15 commits intodevelopfrom
dead-user-notifs-reconnect
Jul 10, 2025
Merged

background-worker/dead-user-notifs-watcher: Reconnect when connection with RabbitMQ is broken#4652
akshaymankar merged 15 commits intodevelopfrom
dead-user-notifs-reconnect

Conversation

@akshaymankar
Copy link
Member

@akshaymankar akshaymankar commented Jul 8, 2025

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

Alternative approach for #4632 (Test is copied from there).

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 Jul 8, 2025
@akshaymankar akshaymankar marked this pull request as ready for review July 9, 2025 11:00
@akshaymankar akshaymankar requested review from a team as code owners July 9, 2025 11:00
Copy link
Contributor

@supersven supersven left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@akshaymankar akshaymankar merged commit 3b49b4a into develop Jul 10, 2025
7 checks passed
@akshaymankar akshaymankar deleted the dead-user-notifs-reconnect branch July 10, 2025 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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