-
Notifications
You must be signed in to change notification settings - Fork 203
[8.19] (backport #9257) [beatreceivers] Integrate beatsauthextension #9970
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
Conversation
* [beatreceivers] Integrate beatsauthextension * add test cases * final tests and this works * update * remove agent.port * mage otel:readme * address review comments * check is ssl.TLS is non-nil * address review comments * remove port * make notice for beatsauth * update beatsauthextension * beatsauth final test * final fix * add beatsauthextension test * Address comments * fix test * fix test * change comment * update beatsauthextension * Update internal/pkg/otel/translate/otelconfig.go Co-authored-by: Craig MacKenzie <craig.mackenzie@elastic.co> * fix ci * add a test that a unique extension is created per output * fix ci * add comment * add better test * fix test --------- Co-authored-by: Craig MacKenzie <craig.mackenzie@elastic.co> (cherry picked from commit 779fafd) # Conflicts: # internal/pkg/otel/manager/manager_test.go # internal/pkg/otel/translate/otelconfig.go # internal/pkg/otel/translate/otelconfig_test.go
|
Cherry-pick of 779fafd has failed: To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
|
Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane) |
|
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
|
@khushijain21 Can you merge target into the backport branch? That should fix the ci failures |
|
This pull request is now in conflicts. Could you fix it? 🙏 |
|
This pull request has not been merged yet. Could you please review and merge it @khushijain21? 🙏 |
|
This pull request is now in conflicts. Could you fix it? 🙏 |
|
This pull request has not been merged yet. Could you please review and merge it @khushijain21? 🙏 |
# Conflicts: # internal/pkg/otel/translate/otelconfig.go # internal/pkg/otel/translate/otelconfig_test.go
⏳ Build in-progress, with failures
Failed CI StepsHistory
|
|




What does this PR do?
This PR adds
beatsauthextensionto elasticsearch config translation layer and also propagates local logger to the translation logicWhy is it important?
It replaces es-exporter's
Rountripperwith beat's implementation of it. This is important for 1:1 behavioral compatibility between beats and beatreceiversChecklist
./changelog/fragmentsusing the changelog toolDisruptive User Impact
None
How to test this PR locally
Check E2E test
Related issues
This is an automatic backport of pull request #9257 done by [Mergify](https://mergify.com).