Skip to content

Conversation

david-gang
Copy link
Contributor

Description

This is a continuation of: #3617 (comment)
We want to align with the semantic conventions. As specified there the data should be taken from get_stats

Type of change

How Has This Been Tested?

Changed the unit tests to mock get_stats

Does This PR Require a Core Repo Change?

  • No.

Checklist:

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

  • [ V] Followed the style guidelines of this project
  • Changelogs have been updated - not relevant
  • Unit tests have been added - not relevant
  • Documentation has been updated - not relevant

@emdneto emdneto requested a review from xrmx July 30, 2025 11:10
@emdneto emdneto added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Jul 30, 2025
@xrmx
Copy link
Contributor

xrmx commented Jul 30, 2025

Thanks for the quick turnaround!

@xrmx xrmx moved this to Easy to review / merge / close in @xrmx's Python PR digest Jul 30, 2025
@xrmx xrmx merged commit d2d136d into open-telemetry:main Jul 30, 2025
625 of 626 checks passed
@github-project-automation github-project-automation bot moved this from Easy to review / merge / close to Done in @xrmx's Python PR digest Jul 30, 2025
devmonkey22 pushed a commit to devmonkey22/opentelemetry-python-contrib that referenced this pull request Aug 5, 2025
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

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants