Skip to content
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

Stabilize "attributes" instrument advisory parameter #3785

Open
trask opened this issue Dec 7, 2023 · 7 comments
Open

Stabilize "attributes" instrument advisory parameter #3785

trask opened this issue Dec 7, 2023 · 7 comments
Labels
spec:metrics Related to the specification/metrics directory triage:accepted:ready Ready to be implemented. Small enough or uncontroversial enough to be implemented without sponsor

Comments

@trask
Copy link
Member

trask commented Dec 7, 2023

What are you trying to achieve?

Stability of Instrument advisory parameter: Attributes

Additional context.

Java Instrumentation is relying heavily on this feature, e.g. emitting HTTP span attributes on HTTP metrics and using this instrument advice to report only the expected attributes, and we would benefit from the feature being stable.

Has anyone else implemented/prototyped this advisory parameter yet?

cc @open-telemetry/go-approvers @open-telemetry/dotnet-approvers @open-telemetry/python-approvers @open-telemetry/javascript-approvers

@trask trask added the spec:metrics Related to the specification/metrics directory label Dec 7, 2023
@jack-berg jack-berg added the [label deprecated] triaged-accepted [label deprecated] Issue triaged and accepted by OTel community, can proceed with creating a PR label Dec 13, 2023
@jack-berg
Copy link
Member

Current status: waiting for additional implementations to collect feedback.

Known implementations:

  • Java

@open-telemetry/go-approvers @open-telemetry/dotnet-approvers @open-telemetry/python-approvers @open-telemetry/javascript-approvers Can you please comment on whether or not this is implemented in your respective language?

@cijothomas
Copy link
Member

OTel .NET has not implemented this. I just created an issue to track this : open-telemetry/opentelemetry-dotnet#5165

(Given .NET's special arrangement of maintaining the Metrics API in the .NET Runtime itself, the earliest we can do this is with next .NET release, which will be Nov 2024, with preview releases comings much earlier)

@pichlermarc
Copy link
Member

pichlermarc commented Dec 13, 2023

We currently do not have a prototype for this in the JavaScript SIG.
Edit: I also just created an issue to track this: open-telemetry/opentelemetry-js#4365

@trask
Copy link
Member Author

trask commented Dec 14, 2023

Linking related: #3790

@trask
Copy link
Member Author

trask commented Feb 9, 2024

pre-requisite for Go: #3803

@austinlparker austinlparker added triage:accepted:needs-sponsor Ready to be implemented, but does not yet have a specification sponsor and removed [label deprecated] triaged-accepted [label deprecated] Issue triaged and accepted by OTel community, can proceed with creating a PR labels Apr 30, 2024
@austinlparker austinlparker added triage:accepted:ready Ready to be implemented. Small enough or uncontroversial enough to be implemented without sponsor and removed triage:accepted:needs-sponsor Ready to be implemented, but does not yet have a specification sponsor labels Apr 30, 2024
@austinlparker austinlparker moved this to Spec - Accepted in 🔭 Main Backlog Jul 16, 2024
@kaylareopelle
Copy link

kaylareopelle commented Aug 23, 2024

Greetings from Rubyland!

I'm working on implementing advisory parameters for Ruby

I'm just planning to work on explicit bucket boundaries for now, but I'm curious about the attributes advisory parameter. How close are we to stability?

For context, we just released an early, experimental metrics API and SDK (OTLP exporter coming soon). The feature set is not complete, but we're working our way through the spec compliance checklist. Advisory parameters came up because I'd like to add metrics to our instrumentation.

If it seems like this feature will stabilize soon, it might be nice to have all instrumentation start with this parameter available.

@trask
Copy link
Member Author

trask commented Oct 25, 2024

How close are we to stability?

I think we still only have the 1 prototype (Java). I've added this to Monday's maintainer meeting. If we can get another language to implement it, and if Ruby can also implement it, that would give us enough prototypes (3) to propose stabilization.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec:metrics Related to the specification/metrics directory triage:accepted:ready Ready to be implemented. Small enough or uncontroversial enough to be implemented without sponsor
Projects
Status: Spec - Accepted
Development

No branches or pull requests

7 participants