-
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
Remove libbeat/metric
and use github.com/elastic/elastic-agent-system-metrics
instead
#31615
Conversation
This pull request is now in conflicts. Could you fix it? 🙏
|
💔 Tests Failed
Expand to view the summary
Build stats
Test stats 🧪
Test errorsExpand to view the tests failures
|
quick thing: The stuff in |
This pull request is now in conflicts. Could you fix it? 🙏
|
libbeat/metric
and use github.com/elastic/elastic-agent-system-metrics
instead
Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane) |
As usual I just spot checked this, we'll have to rely on CI to find any unexpected problems. |
I will fix crosscompiling issues in |
@kvch just to make sure I'm on the same page, is the plan to migrate + remove the metrics code in |
Yes. If that works for you. |
This pull request is now in conflicts. Could you fix it? 🙏
|
This pull request is now in conflicts. Could you fix it? 🙏
|
💚 Flaky test reportTests succeeded. 🤖 GitHub commentsTo re-run your PR in the CI, just comment with:
|
…tem-metrics` instead (#31615)
What does this PR do?
This PR removes the folder
libbeat/metric
and imports the same functionality fromgithub.com/elastic/elastic-agent-system-metrics
.I am not fixing linter issues.
Why is it important?
Avoid maintaining the same code in different places.
Checklist
- [ ] I have commented my code, particularly in hard-to-understand areas- [ ] I have made corresponding changes to the documentation- [ ] I have made corresponding change to the default configuration files- [ ] I have added tests that prove my fix is effective or that my feature works- [ ] I have added an entry inCHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.