open-telemetry / opentelemetry-specification

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

Remove span name from stability guarantees #3995

Closed lmolkova closed 5 months ago

lmolkova commented 5 months ago

Changes

The context: we'd like to update span name for HTTP client spans to include HTTP route/operation name when it's available. Today span name includes only method (e.g. GET and could instead become more useful - GET users/{user_id}). See https://github.com/open-telemetry/semantic-conventions/pull/675 and https://github.com/open-telemetry/semantic-conventions/issues/923 for more details.

However, even in the general case, span name is not something dashboards and alerts should rely on:

Span name should be fine in queries like "top N longest" or "top N with highest error rate" - then, when span names become more descriptive, such queries would produce even better results.

github-actions[bot] commented 5 months ago

This PR was marked stale due to lack of activity. It will be closed in 7 days.