-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Omit monitoring object from logstash_stats.logstash object #16198
Conversation
Pinging @elastic/stack-monitoring (Stack monitoring) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functionally, LGTM!
59da7a2
to
cdbf46b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
left a minor. Beyond this it lgtm codewise.
39a97bd
to
127ea63
Compare
Travis CI is green and Jenkins CI failures are unrelated. Merging. |
…6198) * Omit monitoring object from logstash_stats.logstash object * Removing unnecessary struct tag info * Adding CHANGELOG entry * Adding explanatory comment for nodeInfo
What does this PR do?
It removes a redundant field from Logstash monitoring data being indexed by Metricbeat for Stack Monitoring.
Why is it important?
It brings parity between Metricbeat collection and internal collection for Logstash monitoring data, specifically documents of
type:logstash_stats
.Checklist
[ ] I have commented my code, particularly in hard-to-understand areas[ ] I have made corresponding changes to the documentation[ ] I have made corresponding change to the default configuration files[ ] I have added tests that prove my fix is effective or that my feature worksThere are existing tests (stack monitoring parity tests) that are currently failing and should start passing once this PR is merged.How to test this PR locally
Install Logstash 7.6.0 or higher.
Configure Logstash to set the override cluster UUID. Edit
config/logstash.yml
and setmonitoring.cluster_uuid: foobar
.Build Metricbeat with this PR.
Enable the Logstash module for Stack Monitoring.
Run Metricbeat, outputting events to the Console so we can inspect them easily.
Check that events with
"type": "logstash_stats"
or"type": "logstash_state"
both have a top-level"cluster_uuid": "foobar"
field in them.Also check that events with
"type": "logstash_stats"
has a"logstash_stats.logstash"
field which is an object. Check that this object does not contain a"monitoring"
field.Related issues