Skip to content

Cherry-pick from 6a1a74f, 16f7be0 - Add scope to metric assertions in tests#121

Closed
Cirilla-zmh wants to merge 2 commits intoalibaba:mainfrom
Cirilla-zmh:minghui/fix_metrics_scope
Closed

Cherry-pick from 6a1a74f, 16f7be0 - Add scope to metric assertions in tests#121
Cirilla-zmh wants to merge 2 commits intoalibaba:mainfrom
Cirilla-zmh:minghui/fix_metrics_scope

Conversation

@Cirilla-zmh
Copy link
Copy Markdown
Collaborator

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Does This PR Require a Core Repo Change?

  • No.

Checklist:

See contributing.md for styleguide, changelog guidelines, and more.

  • Followed the style guidelines of this project

anuraaga and others added 2 commits February 24, 2026 16:45
* Pass scope to get_sorted_metrics

* Lint

Change-Id: Icca2e627b6eb61733682d873cd111ca9af0f4bfb
Co-developed-by: Cursor <noreply@cursor.com>
* Add scope to metric assertions in more tests

* More

* Lint

---------

Change-Id: I8ec34407ac7b8094e6785e00163d8c70fdfd60fb
Co-developed-by: Cursor <noreply@cursor.com>
Co-authored-by: Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com>
@Cirilla-zmh Cirilla-zmh added Skip Changelog PRs that do not require a CHANGELOG.md entry synchronize upstream This is a synchronization PR from OpenTelemetry upstream. labels Feb 24, 2026
@Cirilla-zmh
Copy link
Copy Markdown
Collaborator Author

This work can be replaced by #125.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry synchronize upstream This is a synchronization PR from OpenTelemetry upstream.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants