-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
cgroups: accurate procfs parsing #6462
Conversation
|
/cc @dashpole |
Can you add a test using the output from the bug report? |
@dashpole done. |
@rockdrilla thanks for the contribution! Can you please add a changelog entry? |
Codecov ReportBase: 91.86% // Head: 91.86% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #6462 +/- ##
=======================================
Coverage 91.86% 91.86%
=======================================
Files 237 237
Lines 13620 13620
=======================================
Hits 12512 12512
Misses 879 879
Partials 229 229
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
@dmitryax I'm afraid I can't express that change in "human" words better than any of existing contributors. :) |
Got it, wait a minute. |
Just create a file in https://github.com/open-telemetry/opentelemetry-collector/tree/main/.chloggen with a few words like |
fixes #6389 Signed-off-by: Konstantin Demin <rockdrilla@gmail.com>
Description: split line into exactly 3 parts while parsing /proc/{pid}/cgroup.
Link to tracking Issue: #6389
Testing: None
Documentation: None