-
Notifications
You must be signed in to change notification settings - Fork 134
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.14](backport #4961) Introduce agent.monitoring.metrics_period #5003
Conversation
* feat: introduce agent.monitoring.metrics_period * doc: add changelog/fragments * fix: TestDiagnosticLocalConfig unit-test * doc: reword summary in changelog fragment (cherry picked from commit 6a45256) # Conflicts: # _meta/config/common.p2.yml.tmpl # _meta/config/common.reference.p2.yml.tmpl # _meta/config/elastic-agent.docker.yml.tmpl # elastic-agent.docker.yml # elastic-agent.reference.yml # elastic-agent.yml # internal/pkg/agent/application/monitoring/v1_monitor.go # internal/pkg/agent/application/monitoring/v1_monitor_test.go
Cherry-pick of 6a45256 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 |
Looks like we can ignore my message and that @pkoutsovasilis did the necessary adjustments. |
Quality Gate passedIssues Measures |
What does this PR do?
This PR introduces
agent.monitoring.metrics_period
fieldWhy is it important?
I consider this PR important as it allows each user to control the sampling period of monitoring metrics according to them needs
Checklist
- [ ] I have commented my code, particularly in hard-to-understand areas./changelog/fragments
using the changelog tool- [ ] I have added an integration test or an E2E testDisruptive User Impact
None
How to test this PR locally
Spawn an agent with setting the config
agent.monitoring.metrics_period
Related issues
This is an automatic backport of pull request #4961 done by [Mergify](https://mergify.com).