Skip to content

Log errors when policy change notifications fail#15525

Merged
michaelklishin merged 1 commit intomainfrom
da-queue-fed
Feb 21, 2026
Merged

Log errors when policy change notifications fail#15525
michaelklishin merged 1 commit intomainfrom
da-queue-fed

Conversation

@ansd
Copy link
Copy Markdown
Member

@ansd ansd commented Feb 20, 2026

Previously, exceptions during policy change notifications in rabbit_policy:update_matched_objects/3 were silently swallowed by bare catch expressions. This meant that if a decorator's policy_changed callback failed (e.g. due to a node being unreachable), the error was invisible in the logs.
The policy itself was persisted, but the side effects (such as starting or stopping federation links) never took place, requiring a node restart to take effect.

Previously, exceptions during policy change notifications in
rabbit_policy:update_matched_objects/3 were silently swallowed by bare
`catch` expressions. This meant that if a decorator's policy_changed
callback failed (e.g. due to a node being unreachable), the error was
invisible in the logs. The policy itself was persisted, but the
side effects (such as starting or stopping federation links) never
took place, requiring a node restart to take effect.
@michaelklishin michaelklishin added this to the 4.3.0 milestone Feb 21, 2026
@michaelklishin michaelklishin merged commit 209b47f into main Feb 21, 2026
576 of 577 checks passed
@michaelklishin michaelklishin deleted the da-queue-fed branch February 21, 2026 00:37
michaelklishin added a commit that referenced this pull request Feb 21, 2026
Log errors when policy change notifications fail (backport #15525)
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.

2 participants