Skip to content

[spanmetricsconnector] Add instrumentation scope #38001

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

Merged
merged 2 commits into from
Mar 27, 2025

Conversation

iblancasa
Copy link
Contributor

Description

Added the instrumentation scope to the metrics generated by the span metrics connector.

Link to tracking issue

Fixes #23662

Testing

Added a new unit test to cover this feature.

Documentation

Some notes to the README.md file explaining the new config files.

@iblancasa iblancasa requested a review from a team as a code owner February 18, 2025 10:48
@iblancasa iblancasa requested a review from dehaansa February 18, 2025 10:48
@iblancasa iblancasa changed the title Add instrumentation scope to spanmetrics connector metrics [spanmetricsconnector] Add instrumentation scope Feb 18, 2025
@iblancasa
Copy link
Contributor Author

cc @portertech @Frapschen

@iblancasa
Copy link
Contributor Author

cc @portertech @Frapschen can you take a look?

@atoulme atoulme marked this pull request as draft March 19, 2025 18:21
@iblancasa iblancasa marked this pull request as ready for review March 21, 2025 11:01
@atoulme
Copy link
Contributor

atoulme commented Mar 23, 2025

@Frapschen @portertech please review as codeowners.

Copy link
Contributor

@Frapschen Frapschen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@iblancasa
Copy link
Contributor Author

@open-telemetry/android-maintainers this was approved by the code owner. Could we merge it?

@atoulme atoulme merged commit 4079d3f into open-telemetry:main Mar 27, 2025
171 checks passed
@github-actions github-actions bot added this to the next release milestone Mar 27, 2025
Fiery-Fenix pushed a commit to Fiery-Fenix/opentelemetry-collector-contrib that referenced this pull request Apr 24, 2025
#### Description
Added the instrumentation scope to the metrics generated by the span
metrics connector.

#### Link to tracking issue
Fixes open-telemetry#23662

#### Testing
Added a new unit test to cover this feature.

#### Documentation
Some notes to the `README.md` file explaining the new config files.

Signed-off-by: Israel Blancas <[email protected]>
songy23 pushed a commit that referenced this pull request Apr 24, 2025
I would like to become a code owner of spanmetrics connector. I have
been using it in my current and previous companies.

Also, I contributed some times to this component:

#39084

#38945

#38001

#34485

Signed-off-by: Israel Blancas <[email protected]>
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.

Add option to add instrumentation scope name as a dimension
4 participants