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

Add 4 missing duration and queue size metrics #27

Closed
wants to merge 1 commit into from

Conversation

rifelpet
Copy link

This adds the following metrics:

logstash_node_queue_size_bytes{pipeline="main"} 1.4235844e+07
logstash_node_pipeline_queue_push_duration_seconds_total{pipeline="main"} 382
logstash_node_plugin_queue_push_duration_seconds_total{pipeline="main",plugin="beats",plugin_id="beats",plugin_type="input"} 0
logstash_node_plugin_duration_seconds_total{pipeline="main",plugin="elasticsearch",plugin_id="es",plugin_type="output"} 67

This resolves #17

This adds the following metrics:
```
logstash_node_queue_size_bytes{pipeline="main"} 1.4235844e+07
logstash_node_pipeline_queue_push_duration_seconds_total{pipeline="main"} 382
logstash_node_plugin_queue_push_duration_seconds_total{pipeline="main",plugin="beats",plugin_id="beats",plugin_type="input"} 0
logstash_node_plugin_duration_seconds_total{pipeline="main",plugin="elasticsearch",plugin_id="es",plugin_type="output"} 67
```
@tivvit
Copy link

tivvit commented Jan 15, 2019

I think this should be merged, the metrics are missing

@tivvit
Copy link

tivvit commented Jan 15, 2019

I tested the code and it works

@danielmotaleite
Copy link

This repo is dead, use the below fork #35

@rifelpet
Copy link
Author

closing because the fork mentioned above includes this PR and we'll be switching to that.

@rifelpet rifelpet closed this Feb 18, 2020
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.

Add queue_size_in_bytes metrics
3 participants