beat/kibana module - add {module}.elasticsearch.cluster.id property #29577
Conversation
klacabane
added
Team:Integrations
|
Pinging @elastic/stack-monitoring (Stack monitoring) |
|
Pinging @elastic/integrations (Team:Integrations) |
botelastic
bot
added
needs_team
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
💚 Flaky test reportTests succeeded. 🤖 GitHub commentsTo re-run your PR in the CI, just comment with:
|
| "running": c.Int("running"), | ||
| "starts": c.Int("starts"), | ||
| "stops": c.Int("stops"), | ||
| "config": c.Dict("config", s.Schema{ |
There was a problem hiding this comment.
You'll need to add those fields in _meta/fields.yml
klacabane
changed the title
|
It's possible I've got something mis-wired here, but not getting any beats monitoring data yet
|
|
@matschaffer did your ES build include these mappings change ? What type of beat did you monitor and did it output to ES ? |
…29577) * add elasticsearch.cluster.id property to beats state * add elasticsearch.cluster.id property to beats stats * fix test * lint * add missing reloads property * dont break earlier assumptions * remove trailing comma * update beats.stats fields * add updated fields doc * beat: update cluster.id path * kibana: update cluster.id path * mage fmt update * fix fields path * add missing map initialization * add changelog entry * fix pr number (cherry picked from commit 90dd169)
…h.cluster.id property (#29608) * beat/kibana module - add {module}.elasticsearch.cluster.id property (#29577) * add elasticsearch.cluster.id property to beats state * add elasticsearch.cluster.id property to beats stats * fix test * lint * add missing reloads property * dont break earlier assumptions * remove trailing comma * update beats.stats fields * add updated fields doc * beat: update cluster.id path * kibana: update cluster.id path * mage fmt update * fix fields path * add missing map initialization * add changelog entry * fix pr number (cherry picked from commit 90dd169) * remove non-backported changeset entries Co-authored-by: Kevin Lacabane <[email protected]>
Summary
This change adds the
beat.elasticsearch.cluster.idproperty to bothstateandstatsbeat's metricsets, which is required for the SM UI to retrieve beats that outputs their events to a selected cluster.It also reverts this change as we should not be writing to other root module properties from a given module. This requires alias changes in elasticsearch to be functional.
The cluster ID was already available in the
statemetricset so it was just a matter of adding that property to the right path. For thestatsmetricset I reused the 7.16 xpack logic.Testing
elasticsearch-xpackmodule enabled