Skip to content

Conversation

@LoisSotoLopez
Copy link
Contributor

@LoisSotoLopez LoisSotoLopez commented Jun 11, 2024

Proposed Changes

Removes the queue filtering for Rabbit's Prometheus metric collector since it wasn't being used as stated at #11315

Types of Changes

  • Bug fix (non-breaking change which fixes issue #NNNN)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause an observable behavior change in existing systems)
  • Documentation improvements (corrections, new content, etc)
  • Cosmetic change (whitespace, formatting, etc)
  • Build system and/or CI

Checklist

  • I have read the CONTRIBUTING.md document
  • I have signed the CA (see https://cla.pivotal.io/sign/rabbitmq)
  • I have added tests that prove my fix is effective or that my feature works
  • All tests pass locally with my changes
  • If relevant, I have added necessary documentation to https://github.com/rabbitmq/rabbitmq-website
  • If relevant, I have added this change to the first version(s) in release-notes that I expect to introduce it

Further Comments

Closes #11315

@michaelklishin michaelklishin changed the title fix: Remove queues filter for prometh. collector fix: Remove queues filter for Prometheus collector Jun 11, 2024
@michaelklishin michaelklishin added this to the 4.0.0 milestone Jun 11, 2024
@michaelklishin michaelklishin changed the title fix: Remove queues filter for Prometheus collector 4.x: remove queues filter for Prometheus collector Jun 11, 2024
@michaelklishin
Copy link
Collaborator

Actions fail because google-github-actions/auth does. I suspect it may have something to do with the fact that this is an external contribution (to the GitHub org), which always messes up some permission assumptions around Actions.

@michaelklishin
Copy link
Collaborator

Rebasing this PR will address a few CI failures.

Remove queue filters for prometheus_rabbitmq_core_metrics_collector

Closes rabbitmq#11315
@LoisSotoLopez LoisSotoLopez force-pushed the fix/11315-remove-filter-aggregated-queue-metrics-pattern branch from c03ef4f to 27a0b0f Compare June 13, 2024 09:10
@LoisSotoLopez
Copy link
Contributor Author

Did that rebase. Any other action I can help with to move this PR forward?

Copy link
Collaborator

@michaelklishin michaelklishin left a comment

Choose a reason for hiding this comment

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

I'd keep prometheus.filter_aggregated_queue_metrics_pattern. Even if it is not used, why break node startup for 4.0 when it can be easily avoided? The option can be removed in 4.1, 4.2 or something like that, that is, in a release with way fewer changes.

@LoisSotoLopez LoisSotoLopez force-pushed the fix/11315-remove-filter-aggregated-queue-metrics-pattern branch from 928fca1 to 30ecf6a Compare June 20, 2024 07:53
@michaelklishin michaelklishin merged commit 30ecf6a into rabbitmq:main Jun 22, 2024
@michaelklishin
Copy link
Collaborator

Thank you.

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.

prometheus.filter_aggregated_queue_metrics_pattern has no effect

3 participants