-
Notifications
You must be signed in to change notification settings - Fork 893
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 semconv for hardware metrics to use direction
attributes
#2942
Update semconv for hardware metrics to use direction
attributes
#2942
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a changelog entry!
@jack-berg CHANGELOG entry added! |
This seems like a straightforward change to comply with the other metrics we already have cc @jsuereth |
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
Since everybody approved this, I guess it's time to hit that green Merge button 😉 |
@bertysentry Agreed. Sorry, can you merge again? :( |
@carlosalberto Merge done, all checks have passed, we're good to go! Thank you! |
Fixes #2805
Changes
Updated semantic conventions for hardware metrics (
specification\metrics\semantic_conventions\hardware-metrics.md
) to use thedirection
attributes (and follow updated general semantic conventions).hw.gpu.io.receive
,hw.gpu.io.transmit
==>hw.gpu.io
hw.network.io.receive
,hw.network.io.transmit
==>hw.network.io
hw.network.packets.receive
,hw.network.packets.transmit
==>hw.network.packets