-
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
[Elastic Agent] Add stream.* to metrics and logs collected #18261
[Elastic Agent] Add stream.* to metrics and logs collected #18261
Conversation
This PR ensure that the stream.type, stream.dataset and stream.namespace are added to each logs or metrics collected by the Agent. Fixes: elastic#18203
Pinging @elastic/ingest-management (Team:Ingest Management) |
@@ -199,6 +211,18 @@ func (o *Operator) getMonitoringMetricbeatConfig(output interface{}) (map[string | |||
"period": "10s", | |||
"hosts": hosts, | |||
"index": "metrics-agent-default", | |||
"processors": []map[string]interface{}{ |
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.
a slice of map make me always cringe a little.
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.
Code LGTM, but did not run it locally.
This PR made me think if we should have a final pipeline that enriches events with these fields if they are missing based on the index name 🤔
This seems to be a good default, that would be a painless script? Maybe create an issue. |
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.
LGTM i will test tomorrow morning, if you need to backport let's get this in
💚 Build SucceededExpand to view the summary
Build stats
|
…8261) * [Elastic Agent] Add stream.* to metrics and logs collected This PR ensure that the stream.type, stream.dataset and stream.namespace are added to each logs or metrics collected by the Agent. Fixes: elastic#18203 (cherry picked from commit 7b87db5)
This PR ensure that the stream.type, stream.dataset and stream.namespace
are added to each logs or metrics collected by the Agent.
Fixes: #18203
What does this PR do?
Why is it important?
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Author's Checklist
How to test this PR locally
Related issues
Use cases
Screenshots
Logs