-
Notifications
You must be signed in to change notification settings - Fork 144
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
[8.x](backport #5492) Add apm trace sampling #5582
Conversation
Cherry-pick of b284e56 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-control-plane (Team:Elastic-Agent-Control-Plane) |
This pull request has not been merged yet. Could you please review and merge it @pchila? 🙏 |
* Update elastic-agent-client dependency to 7.16.0 * Add support for APM sampling rate config * fixup! Add support for APM sampling rate config * Add unit tests for APM config triggering component changes * make notice (cherry picked from commit b284e56) # Conflicts: # go.mod # go.sum
86ecafd
to
0659a63
Compare
Quality Gate failedFailed conditions |
What does this PR do?
This PR adds the possibility to specify apm sample rate in elastic agent APM configuration
Why is it important?
Specifying a sample rate allows controlling the ingestion rate, reducing overhead and storage requirements when self-monitoring is active
Checklist
[ ] I have made corresponding changes to the documentation./changelog/fragments
using the changelog toolDisruptive User Impact
How to test this PR locally
Related issues
Questions to ask yourself
This is an automatic backport of pull request #5492 done by [Mergify](https://mergify.com).