Skip to content

Conversation

@ilyam8
Copy link
Member

@ilyam8 ilyam8 commented Dec 5, 2025

Summary

Older versions have only rabbit_version, current versions have rabbitmq_version and rabbit_version. We don't really need this check.

Test Plan
Additional Information
For users: How does this change affect me?

Summary by cubic

Removed the rabbitmq_version check in the RabbitMQ collector to avoid errors with older brokers that only provide rabbit_version. This improves compatibility across versions without changing metrics collection.

Written for commit 34f54a4. Summary will update automatically on new commits.

@github-actions github-actions bot added area/collectors Everything related to data collection collectors/go.d area/go labels Dec 5, 2025
@ilyam8 ilyam8 enabled auto-merge (squash) December 5, 2025 13:27
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

@ilyam8 ilyam8 merged commit b921c2e into netdata:master Dec 5, 2025
117 checks passed
@ilyam8 ilyam8 deleted the fix-rabbitmq-ver-check branch December 5, 2025 16:31
@stelfrag stelfrag mentioned this pull request Dec 6, 2025
stelfrag pushed a commit to stelfrag/netdata that referenced this pull request Dec 6, 2025
Ferroin pushed a commit that referenced this pull request Dec 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/collectors Everything related to data collection area/go collectors/go.d

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants