search

open-telemetry / opentelemetry-specification

Specifications for OpenTelemetry
https://opentelemetry.io
Apache License 2.0
3.72k stars 888 forks source link

Add schema translation to remove metric attributes that are newer than the user's pinned schema version #3288

Open trask opened 1 year ago

trask commented 1 year ago

From Semantic Conventions Stability:

Adding attributes to existing metrics is considered to be a breaking change.

and from @Oberon00's https://github.com/open-telemetry/opentelemetry-specification/issues/3253#issuecomment-1448120021:

This being a breaking change sounds like a serious problem. Can't there be a schema entry for new metric attributes so that they can be transparently removed, if this is really a problem?

trask commented 1 year ago

Marking this as an HTTP semconv stability blocker for now. We don't need to solve it before stability, but just knowing that it's solvable would allow us to postpone some other issues until after stability, e.g. adding an "http client error type" metric attribute as discussed in #3253