Skip to content

Support including scope metadata as metric labels#16878

Merged
ArthurSens merged 2 commits intomainfrom
promote-scope-attributes
Jul 18, 2025
Merged

Support including scope metadata as metric labels#16878
ArthurSens merged 2 commits intomainfrom
promote-scope-attributes

Conversation

@ArthurSens
Copy link
Member

Reintroduce #16730 and #16760, which were reverted in #16842.

After further discussions in open-telemetry/semantic-conventions#2489, a scope shouldn't have attributes called name, version, or schema_url, so the collision should not happen.

Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
@ArthurSens
Copy link
Member Author

I'm taking the liberty to merge with a single approval since I'm just re-introducing something that was approved by others before!

@ArthurSens ArthurSens merged commit 3602785 into main Jul 18, 2025
45 checks passed
@ArthurSens ArthurSens deleted the promote-scope-attributes branch July 18, 2025 16:44
tcp13equals2 pushed a commit to tcp13equals2/prometheus that referenced this pull request Aug 18, 2025
* Support including scope metadata as metric labels

Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>

* Ensure Scope Name, Version and Schema URL aren't overriden by attributes

Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>

---------

Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
Signed-off-by: Andrew Hall <andrew.hall@grafana.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants