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

Metricbeat: RabbitMQ/Queue disk reads and writes count keys made optional #6887

Merged
merged 1 commit into from
Apr 18, 2018

Conversation

dolftax
Copy link
Contributor

@dolftax dolftax commented Apr 17, 2018

disk_reads and disk_writes keys of rabbitmq management plugin
aren't available at times, throwing error.

Signed-off-by: Jaipradeesh [email protected]

…onal

disk_reads and disk_writes keys of rabbitmq management plugin
aren't available at times, throwing error.

Signed-off-by: Jaipradeesh <[email protected]>
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@ruflin
Copy link
Contributor

ruflin commented Apr 18, 2018

jenkins, test it

@ruflin
Copy link
Contributor

ruflin commented Apr 18, 2018

@jaipradeesh Thanks for the PR, changes LGTM. I wonder what is the reason these metrics are sometimes not available?

@dolftax
Copy link
Contributor Author

dolftax commented Apr 18, 2018

I assume its because of the transient state of messages (or) the memory consumed by RabbitMQ isn't greater than 40% of machine's memory. Thus, paging to disk never happens.

https://www.rabbitmq.com/memory.html

@ruflin ruflin merged commit c733577 into elastic:master Apr 18, 2018
@ruflin
Copy link
Contributor

ruflin commented Apr 18, 2018

@jaipradeesh Perfect, exactly the explanation I was looking for, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants