-
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
transform beat.cgroup.memory.mem.usage.bytes to Gauge #32652
transform beat.cgroup.memory.mem.usage.bytes to Gauge #32652
Conversation
💚 CLA has been signed |
1ecf76a
to
a5d514e
Compare
Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane) |
Changes look good, you'll need to sign the CLA and then we can kick the tests and get this merged. |
I'd bet I've signed this a couple of hours ago, but in order to reflect the changes, should we re-trigger ci? /test |
/test |
Yes, CLA check has passed now. Someone from Elastic has to trigger the first test run, which I've now done. We can ignore the linter errors here as they aren't caused by changes you've made (it always wants to lint the entire file, not just the changed lines). |
Merged, thanks! |
What does this PR do?
This transforms beat.cgroup.memory.mem.usage.bytes to Gauge as it can have a negative value
Why is it important?
it tries to eliminate
Non-zero metrics in the last 30s
errors by transformingbeat.cgroup.memory.mem.usage.bytes
to gaugeChecklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Author's Checklist
How to test this PR locally
Related issues
Fix: #31582
Use cases
Screenshots
Logs