Skip to content

Update system metrics callbacks to accept CallbackOptions#1084

Merged
srikanthccv merged 3 commits intoopen-telemetry:mainfrom
aabmass:callbackoptions
May 10, 2022
Merged

Update system metrics callbacks to accept CallbackOptions#1084
srikanthccv merged 3 commits intoopen-telemetry:mainfrom
aabmass:callbackoptions

Conversation

@aabmass
Copy link
Copy Markdown
Member

@aabmass aabmass commented May 9, 2022

Description

This updates contrib for the core repo PR open-telemetry/opentelemetry-python#2664

Type of change

Please delete options that are not relevant.

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

How Has This Been Tested?

Existing tests pass

Does This PR Require a Core Repo Change?

Checklist:

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

  • Followed the style guidelines of this project

@aabmass aabmass added the Skip Changelog PRs that do not require a CHANGELOG.md entry label May 9, 2022
@aabmass aabmass force-pushed the callbackoptions branch from d43719a to 5d1d93d Compare May 9, 2022 16:50
@aabmass aabmass marked this pull request as ready for review May 9, 2022 17:18
@aabmass aabmass requested a review from a team May 9, 2022 17:18
Copy link
Copy Markdown
Member

@srikanthccv srikanthccv left a comment

Choose a reason for hiding this comment

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

it's always the cache issue :)

@srikanthccv srikanthccv merged commit 3e67893 into open-telemetry:main May 10, 2022
@aabmass aabmass deleted the callbackoptions branch May 11, 2022 14:21
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

None yet

Development

Successfully merging this pull request may close these issues.

4 participants