Skip to content

Revert "Merge pull request #1702 from stackhpc/update-rabbitmq-rule" #1811

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

Merged
merged 1 commit into from
Aug 4, 2025

Conversation

jackhodgkiss
Copy link
Contributor

This reverts commit c9c3eba, reversing changes made to ddd4daa.

The metric erlang_vm_dist_node_state should not be compared against the size of the RabbitMQ cluster as the values indicate the current state of the link with 3 implying the connection is up.

This reverts commit c9c3eba, reversing
changes made to ddd4daa.

The metric `erlang_vm_dist_node_state` should not be compared against
the size of the `RabbitMQ` cluster as the values indicate the current
state of the link with `3` implying the connection is `up`.
Copy link
Member

@priteau priteau left a comment

Choose a reason for hiding this comment

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

Good catch!

@Alex-Welsh Alex-Welsh enabled auto-merge (rebase) August 4, 2025 07:21
@Alex-Welsh Alex-Welsh merged commit 015ffb7 into stackhpc/2025.1 Aug 4, 2025
40 of 41 checks passed
@Alex-Welsh Alex-Welsh deleted the revert-rabbitmq-broken-rule branch August 4, 2025 08:12
@jackhodgkiss
Copy link
Contributor Author

/cherry-pick stackhpc/2024.1

Copy link

Cherry-pick failed with Merge error 015ffb70f2d1ae257d2a95cc11c9ec18069e478d into temp-cherry-pick-a572b2-stackhpc/2024.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
monitoring All things related to observability & telemetry size: xs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants