Skip to content

[Logstash] Add Health Report API output#772

Merged
pickypg merged 1 commit intoelastic:mainfrom
lucabelluccini:patch-6
Dec 19, 2024
Merged

[Logstash] Add Health Report API output#772
pickypg merged 1 commit intoelastic:mainfrom
lucabelluccini:patch-6

Conversation

@lucabelluccini
Copy link
Copy Markdown
Contributor

Following elastic/logstash#16520 and consequent https://www.elastic.co/guide/en/logstash/current/health-report-pipeline-flow-worker-utilization.html, I'm adding this API.

FYI @robbavey / @jsvd - I'm adding this API to the Support Diagnostic tool.
Is there any other way to leverage it's output? Should we add something to the Flow visualizer or our Logstash Visualizer?

Checklist

  • I have verified that the APIs in this pull request do not return sensitive data

versions:
"> 0.0.0": "/_node/plugins"

logstash_health_report:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we sort this so it's in a predictable spot?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Should I sort the entire list of APIs?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ah, no. I hadn't realized Logstash had not been sorted properly. I'll merge this, then do it all.

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.

2 participants