Skip to content
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

Add metrics renaming changes to contrib #4

Closed
wants to merge 3 commits into from

Conversation

AzfaarQureshi
Copy link

Description

This PR makes the corresponding renaming changes from otel-python open-telemetry/opentelemetry-python#1372 in the contrib to fix failing tests as noted here: open-telemetry/opentelemetry-python#1372 (comment)

@AzfaarQureshi AzfaarQureshi force-pushed the add-metrics-renaming-changes-to-contrib branch from 8e41278 to 90aff44 Compare November 19, 2020 01:25
@@ -6,7 +6,7 @@ on:
- 'release/*'
pull_request:
env:
CORE_REPO_SHA: 47483865854c7adae7455f8441dab7f814f4ce2a
CORE_REPO_SHA: 3b813eb9921e709538dd1b07fa7a5f93600fbec1
Copy link
Author

Choose a reason for hiding this comment

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

The SHA needs to be updated every time in the contrib repo because it runs tests using the latest version of core

@AzfaarQureshi AzfaarQureshi force-pushed the add-metrics-renaming-changes-to-contrib branch 4 times, most recently from 1eb3f68 to 4aaa80c Compare November 19, 2020 01:48
@AzfaarQureshi AzfaarQureshi force-pushed the add-metrics-renaming-changes-to-contrib branch from 4aaa80c to 23de712 Compare November 19, 2020 01:51
Copy link

@alolita alolita left a comment

Choose a reason for hiding this comment

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

lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants