-
Notifications
You must be signed in to change notification settings - Fork 614
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
Update OS process metrics name according to semconv #2662
Comments
@emdneto May i take this one |
@arunk1988 Sure. Please open a PR! |
@emdneto I am still working on this and made some progress. how does this snippet look, please advise. def test_process_metrics_instrument(self):
|
@arunk1988 Could you please open a PR? It's difficult to answer without seeing the changes |
@emdneto can you please review the draft and let me know your inputs |
@arunk1988 I understand you're still working on the PR. If it's ready, please mark as "Ready for Review" |
Signed-off-by: KarthikeyanB, Arun <[email protected]>
@emdneto Please review, i have already added your recommendation to this |
What problem do you want to solve?
Some OS process metrics are being captured in system-metrics instrumentations with a
runtime
prefix. Following this, it seems we should decide if we set the metric name independent of runtime.Describe the solution you'd like
OS process metrics following the semantic conventions and no runtime prefix in system-metrics-instrumentation
Describe alternatives you've considered
No response
Additional Context
A result of a discussion #2652 (comment)
Would you like to implement a fix?
None
The text was updated successfully, but these errors were encountered: