Skip to content

[receiver/prometheus] Add otel_scope_[attribute]#41958

Closed
heliapb wants to merge 2 commits into
open-telemetry:mainfrom
heliapb:feat/add_otel_scope_attributes
Closed

[receiver/prometheus] Add otel_scope_[attribute]#41958
heliapb wants to merge 2 commits into
open-telemetry:mainfrom
heliapb:feat/add_otel_scope_attributes

Conversation

@heliapb
Copy link
Copy Markdown

@heliapb heliapb commented Aug 12, 2025

Description

Add otel_scope_[attribute], from opentelemetry-go/5947

Link to tracking issue

Fixes #41502

Testing

Documentation

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented Aug 12, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@heliapb heliapb changed the title feat: Add otel_scope_[attribute] [receiver/prometheus] Add otel_scope_[attribute] Aug 13, 2025
@heliapb heliapb force-pushed the feat/add_otel_scope_attributes branch 4 times, most recently from 7118529 to e6b4966 Compare August 13, 2025 17:05
Signed-off-by: Hélia Barroso <helia_barroso@hotmail.com>
@heliapb heliapb force-pushed the feat/add_otel_scope_attributes branch from e6b4966 to 75d2231 Compare August 13, 2025 17:06
@heliapb heliapb marked this pull request as ready for review August 13, 2025 17:06
@heliapb heliapb requested review from a team, ArthurSens and dashpole as code owners August 13, 2025 17:06
Signed-off-by: Hélia Barroso <helia_barroso@hotmail.com>
@ArthurSens
Copy link
Copy Markdown
Member

ArthurSens commented Aug 13, 2025

Hey Helia, thanks for offering help here! The change is not as easy as it looks. Besides parsing scope attributes from labels prefixed with otel_scope_, we also want to deprecate the old behavior where scope attributes were parsed from the metric otel_scope_info. We can't just remove a functionality like that, so this also needs to be done through feature gates.

I happen to have a PR open already, do you want to take a look, try it out and maybe review? :)
#40060

I totally forgot to link the PR with the issue, sorry about that

@heliapb
Copy link
Copy Markdown
Author

heliapb commented Aug 13, 2025

Hi @ArthurSens sure thing, I totally forgot to check the existing prs as well, my bad. I'll close this one

@heliapb heliapb closed this Aug 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[receiver/prometheus] Add support for otel_scope_<attribute-name> labels

3 participants