Apply default processors to events from the monitoring receiver#12717
Apply default processors to events from the monitoring receiver#12717faec merged 13 commits intoelastic:mainfrom
Conversation
|
This pull request is now in conflicts. Could you fix it? 🙏 |
|
This pull request does not have a backport label. Could you fix it @faec? 🙏
|
|
Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane) |
|
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
|
Can we have a test for this? Ideally you could change one of the |
swiatekm
left a comment
There was a problem hiding this comment.
I agree that an integration test should check this.
|
@mauri870 I couldn't find a practical way to do an integration test that used the OTel manager config code directly, so I added a unit test that makes sure the expected processor config is there and then separately an integration test that makes sure that including a processor with that config template produces the expected event fields. |
⏳ Build in-progress, with failures
Failed CI StepsHistory
cc @faec |
…rs-for-otel-monitoring
…onitoring receiver (#12832) * Apply default processors to events from the monitoring receiver (#12717) Include the `beat` processor in the OTel Collector and use it to apply default processors (most notably `add_host_metadata`) to monitoring events generated by collector internal telemetry. See #12577 (cherry picked from commit 27d1218) # Conflicts: # internal/pkg/otel/translate/otelconfig.go * fix merge --------- Co-authored-by: Fae Charlton <fae.charlton@elastic.co>
Include the
beatprocessor in the OTel Collector and use it to apply default processors (most notablyadd_host_metadata) to monitoring events generated by collector internal telemetry. See #12577Checklist
./changelog/fragmentsusing the changelog toolHow to test this PR locally
With this PR, any ingested monitoring metrics with
component.idofelasticsearch-defaultorelasticsearch-monitoringshould include the fields from standard processors, most notably the hostname.