beats
beats copied to clipboard
[test][metricbeat] Fix system cpuinfo metricset tests on non-amd64 platforms
Proposed commit message
[test][metricbeat] Fix system cpuinfo metricset tests on non-amd64 platforms
Checklist
- [x ] My code follows the style guidelines of this project
- [ x] 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 in
CHANGELOG.next.asciidocorCHANGELOG-developer.next.asciidoc.~~
Disruptive User Impact
None
Author's Checklist
- [ ]
How to test this PR locally
Part of existing python tests
Related issues
- Closes #34244
Use cases
Screenshots
Logs
:robot: GitHub comments
Expand to view the GitHub comments
Just comment with:
rundocs-build: Re-trigger the docs validation. (use unformatted text in the comment!)
This pull request does not have a backport label. If this is a bug or security fix, could you label this PR @jonathan-albrecht-ibm? 🙏. For such, you'll need to label your PR with:
- The upcoming major version of the Elastic Stack
- The upcoming minor version of the Elastic Stack (if you're not pushing a breaking change)
To fixup this pull request, you need to add the backport labels for the needed branches, such as:
backport-8./dis the label to automatically backport to the8./dbranch./dis the digitbackport-active-allis the label that automatically backports to all active branches.backport-active-8is the label that automatically backports to all active minor branches for the 8 major.backport-active-9is the label that automatically backports to all active minor branches for the 9 major.
Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)
/test
@jonathan-albrecht-ibm Thanks! Can you run make check and commit the changes to the python file? It looks like CI is probably failing on some formatting requirements.
Thanks @mauri870 and @faec. I have fixed the formatting.
/test
Thanks!
Thanks for reviewing @mauri870 @andrzej-stencel @faec !