Skip to content

Add support for Logstash 7 'queue' stats from the Pipelines API#9080

Merged
ivorybilled merged 3 commits intomasterfrom
logstash7QueueSupport
Apr 6, 2021
Merged

Add support for Logstash 7 'queue' stats from the Pipelines API#9080
ivorybilled merged 3 commits intomasterfrom
logstash7QueueSupport

Conversation

@ivorybilled
Copy link
Copy Markdown
Contributor

This PR is meant to satisfy #7629, the Logstash plugin was unable to parse some stats for 'queue' when querying pipelines, so we'll add those to the model and if the newer ones exist, use those.

It also seems that 'events' was renamed to 'event_count' so that had to be handled as well.

Copy link
Copy Markdown
Contributor

@telegraf-tiger telegraf-tiger bot left a comment

Choose a reason for hiding this comment

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

🤝 ✅ CLA has been signed. Thank you!

@telegraf-tiger telegraf-tiger bot added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Apr 1, 2021
Copy link
Copy Markdown
Contributor

@telegraf-tiger telegraf-tiger bot left a comment

Choose a reason for hiding this comment

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

@ivorybilled ivorybilled merged commit 868befc into master Apr 6, 2021
@ivorybilled ivorybilled deleted the logstash7QueueSupport branch April 6, 2021 21:33
jblesener pushed a commit to jblesener/telegraf that referenced this pull request Apr 18, 2021
…uxdata#9080)

* LAdd support for logstash 7 'queue' stats for its pipelines stats API

* appease the linter

* Update samples_logstash7.go
arstercz pushed a commit to arstercz/telegraf that referenced this pull request Aug 28, 2025
…uxdata#9080)

* LAdd support for logstash 7 'queue' stats for its pipelines stats API

* appease the linter

* Update samples_logstash7.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants