Skip to content
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.

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

ARTEMIS-5250 Add sender state to the management metrics for federation #5458

Merged

Conversation

tabish121
Copy link
Contributor

Adds management and metrics to the outgoing AMQP senders created by a remote peer that has message federation configured from the local broker including a view for the remote policies that tracks number of sent messages for all producers. Also adds a top level federation management view that shows counts for incoming and outgoing messages for all federation consumer and producers.

Add a more declarative state handling API for initialization, start, stop and shutdown to federation resources and the broker connection API to make management resource create and cleanup more reliable and robust to avoid and leaked management objects when reconnection attempts stop or broker connections are manually stopped or are stopped by configuration updates or configuration remove.

Adds management and metrics to the outgoing AMQP senders created by a
remote peer that has message federation configured from the local broker
including a view for the remote policies that tracks number of sent
messages for all producers. Also adds a top level federation management
view that shows counts for incoming and outgoing messages for all
federation consumer and producers.

Add a more declarative state handling API for initialization, start,
stop and shutdown to federation resources and the broker connection API
to make management resource create and cleanup more reliable and robust
to avoid and leaked management objects when reconnection attempts stop
or broker connections are manually stopped or are stopped by
configuration updates or configuration remove.
@gemmellr gemmellr merged commit d130d6b into apache:main Jan 21, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants