open-telemetry / opentelemetry-specification

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

Attribute limits not supported in Metrics #1830

Open jtmalinowski opened 3 years ago

jtmalinowski commented 3 years ago

What are you trying to achieve?

As open-telemetry/opentelemetry-specification#1130 is nearing completion we're aiming to support Attribute limits in all models, which have Attributes. However, Metrics cause additional difficulties, because truncation and/or deletion of their attributes could affect their identity, which requires further discussion.

Additional context.

Some notes from a comment on open-telemetry/opentelemetry-specification#1130:

And some more ideas from another comment in open-telemetry/opentelemetry-specification#1130:

jtmalinowski commented 3 years ago

I'm happy to own this one and drive the discussion, unless someone working on Metrics would like to own it.

manolama commented 3 years ago

HI! Starting to look into supporting OTel but I'm worried about this idea of truncating data. The limits make absolute sense and being configurable is great. But to avoid user confusion, avoid the duplicated identity issue, and the problems of how to truncate UTF-8 from https://github.com/open-telemetry/opentelemetry-specification/issues/504, wouldn't it be better to drop the errant data and notify the user? E.g. have the APIs return a response code, 0 for good, non-zero for a data error. Then use the built-in SDK telemetry to track the errors. I think truncation will lead to a world of hurt.

reyang commented 3 years ago

Two updates: