Add xpack data structure for Elasticsearch index_summary metricset#7102
Add xpack data structure for Elasticsearch index_summary metricset#7102jsoriano merged 2 commits intoelastic:masterfrom
Conversation
There was a problem hiding this comment.
don't use an underscore in package name
There was a problem hiding this comment.
Can we configure hound rules to ignore this? :D
There was a problem hiding this comment.
Would be nice. Not sure how hound exaclty works / is configured TBH ;-)
There was a problem hiding this comment.
I think this cannot be configured 😕 reasons: golang/lint#209
7622e1d to
ee0e680
Compare
* Add ID to node info and make an empty node ID possible
a68b4e1 to
6493bcd
Compare
| event.RootFields.Put("type", "indices_stats") | ||
| event.RootFields.Put("source_node", sourceNode) | ||
|
|
||
| event.Index = ".monitoring-es-6-mb" |
There was a problem hiding this comment.
This 6 is related to the version? Could we extract it from somewhere else so we don't forget to update it in 7.0? :)
There was a problem hiding this comment.
This is only temporary and is the same across all metricsets for now. We have to come up with something better in the long run. Would keep it for now.
| event.RootFields.Put("indices_stats._all", fields) | ||
| event.RootFields.Put("cluser_uuid", info.ClusterID) | ||
| event.RootFields.Put("timestamp", common.Time(time.Now())) | ||
| event.RootFields.Put("interval_ms", m.Module().Config().Period.Nanoseconds()/1000/1000) |
There was a problem hiding this comment.
m.Module().Config().Period/time.Millisecond
No description provided.