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

rabbitmq plugin: make deprecated heathcheck optional #7823

Closed
pabigot opened this issue Jul 11, 2020 · 3 comments
Closed

rabbitmq plugin: make deprecated heathcheck optional #7823

pabigot opened this issue Jul 11, 2020 · 3 comments

Comments

@pabigot
Copy link
Contributor

pabigot commented Jul 11, 2020

The latest RabbitMQ servers issue a diagnostic each time the HTTP management API is used to invoke a node health check from the rabbitmq telegraf plugin. This makes the rabbitmq server logs very noisy.

Relevant telegraf.conf:

# Reads metrics from RabbitMQ servers via the Management Plugin
[[inputs.rabbitmq]]
  # No non-default configuration

System info:

Telegraf 1.14.5 monitoring rabbitmq 3.8.5 using the rabbitmq:3-management docker image

Steps to reproduce:

Enable inputs.rabbitmq with RabbitMQ 3.8.5 and monitor the logs


Expected behavior:

No complaints in the rabbitmq logs due to use of the rabbitmq monitoring plugin.

Actual behavior:

Messages like:

2020-07-11 15:56:20.023 [warning] <0.26026.0> rabbitmqctl node_health_check and its HTTP API counterpart are DEPRECATED. See https://www.rabbitmq.com/monitoring.html#health-checks for replacement options.

appear at the telegraf check rate of 0.1 Hz.

Additional info:

See: rabbitmq/rabbitmq-cli#426

@ssoroka
Copy link
Contributor

ssoroka commented Jul 29, 2020

If it's deprecated, maybe we should just stop collecting this?

pabigot added a commit to pabigot/telegraf that referenced this issue Jul 29, 2020
pabigot added a commit to pabigot/telegraf that referenced this issue Jul 29, 2020
@pabigot
Copy link
Contributor Author

pabigot commented Jul 29, 2020

If it's deprecated, maybe we should just stop collecting this?

I think #7920 handles this: manual run with telegraf --test removes the diagnostic on my system.

@ssoroka
Copy link
Contributor

ssoroka commented Aug 10, 2020

Looks like this is already resolved as of #7920

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

No branches or pull requests

2 participants