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

stats/otel: upgrade grpc version that contains the experimental/stats package #7545

Merged
merged 4 commits into from
Aug 29, 2024

Conversation

arvindbr8
Copy link
Member

Building stats/otel was broken on its own because this package depends on grpc/experimental/stats which is not released yet. Our tests passed because of the replace directive magic.

RELEASE NOTES: none

@arvindbr8 arvindbr8 added this to the 1.67 Release milestone Aug 21, 2024
@arvindbr8 arvindbr8 requested a review from dfawley August 21, 2024 22:21
Copy link

codecov bot commented Aug 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.84%. Comparing base (9ab8b62) to head (07f84df).
Report is 13 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7545      +/-   ##
==========================================
- Coverage   81.93%   81.84%   -0.10%     
==========================================
  Files         362      361       -1     
  Lines       27831    27825       -6     
==========================================
- Hits        22803    22773      -30     
- Misses       3832     3854      +22     
- Partials     1196     1198       +2     

see 25 files with indirect coverage changes

@arvindbr8 arvindbr8 assigned arvindbr8 and unassigned dfawley Aug 21, 2024
@arvindbr8 arvindbr8 assigned dfawley and unassigned arvindbr8 Aug 22, 2024
@@ -12,7 +12,7 @@ require (
go.opentelemetry.io/otel/metric v1.28.0
go.opentelemetry.io/otel/sdk v1.28.0
go.opentelemetry.io/otel/sdk/metric v1.28.0
google.golang.org/grpc v1.65.0
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't we use 1.66.0?

Copy link
Member Author

Choose a reason for hiding this comment

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

Done, 1.66.0 was not available at the time.

@dfawley dfawley assigned arvindbr8 and unassigned dfawley Aug 28, 2024
@arvindbr8 arvindbr8 assigned dfawley and unassigned arvindbr8 Aug 28, 2024
@arvindbr8 arvindbr8 changed the title stats/otel: upgrade grpc in go.mod to use version that contains the experimental/stats stats/otel: upgrade grpc version that contains the experimental/stats package Aug 29, 2024
@dfawley dfawley assigned arvindbr8 and unassigned dfawley Aug 29, 2024
@arvindbr8 arvindbr8 merged commit 845f62c into grpc:master Aug 29, 2024
14 checks passed
@arvindbr8 arvindbr8 deleted the fix_otel_go_mod branch August 29, 2024 21:34
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.

2 participants