opentelemetry-js icon indicating copy to clipboard operation
opentelemetry-js copied to clipboard

feat(instr-grpc): update semconv to 1.27

Open JamieDanielson opened this issue 1 year ago • 0 comments
trafficstars

Which problem is this PR solving?

  • updates #4897

Short description of the changes

  • use new ATTR_* for stable http/net attributes
  • start adding logic for opt-in options

Question: Should we be using ATTR_* from incubating, or keep using SEMATTRS_*? Asked on http server PR

TODO

  • OTEL_SEMCONV_STABILITY_OPT_IN
  • ATTR_NET_PEER_NAME => ATTR_SERVER_ADDRESS || ATTR_CLIENT_ADDRESS from stable
  • ATTR_NET_PEER_PORT => ATTR_SERVER_PORT || ATTR_CLIENT_PORT from stable

Type of change

Please delete options that are not relevant.

  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] This change requires a documentation update

How Has This Been Tested?

  • [ ] unit tests

Checklist:

  • [ ] Followed the style guidelines of this project
  • [ ] Unit tests have been added
  • [ ] Documentation has been updated

JamieDanielson avatar Sep 18 '24 21:09 JamieDanielson