Skip to content

Update ra_metrics family for Ra counters #2286

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mkuratczyk
Copy link
Contributor

PR rabbitmq/rabbitmq-server#13895 switches from exposing ra_metrics ETS table values as Prometheus metrics, to exposing Ra counters from seshat. There's many more metrics available with this change.
A few metrics are the same but have a slightly different name. Since the old name was incorrect, we don't maintain compatibility but use this opportunity to fix the name:

These metrics lost the _log part, since they are not log related:

  • rabbitmq_detailed_raft_log_snapshot_index -> rabbitmq_detailed_raft_commit_index
  • rabbitmq_detailed_raft_log_last_applied_index -> rabbitmq_detailed_raft_last_applied_index
  • rabbitmq_detailed_raft_log_commit_index -> rabbitmq_detailed_raft_commit_index

rabbitmq_detailed_raft_term_total is now rabbitmq_detailed_raft_term. It was a bug that led to adding the _total suffix.

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.

1 participant