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

Update the otelgocql to use the latest otel pkg #4199

Merged
merged 1 commit into from Aug 17, 2023

Conversation

MrAlias
Copy link
Contributor

@MrAlias MrAlias commented Aug 15, 2023

The skd/metric/aggregation package was deprecated in favor of using the added aggregation types in sdk/metric. This updates the otelgocql example to use the new types.

Note, this is the only change needed to bring contrib into a compile-able state with commit d78820e9050cd63daebdb4b82202f10d9c2b66e3 of go.opentelemetry.io/otel*.

The `skd/metric/aggregation` package was deprecated in favor of using
the added aggregation types in `sdk/metric`. This updates the otelgocql
example to use the new types.

Note, this is the only change needed to bring contrib into a
compile-able state with commit d78820e9050cd63daebdb4b82202f10d9c2b66e3
of go.opentelemetry.io/otel*.
@MrAlias MrAlias added Skip Changelog Allow PR to succeed without requiring an addition to the CHANGELOG instrumentation: otelgocql labels Aug 15, 2023
@MrAlias MrAlias requested a review from a team as a code owner August 15, 2023 22:42
@codecov
Copy link

codecov bot commented Aug 15, 2023

Codecov Report

Merging #4199 (4711dad) into main (e12a8ba) will increase coverage by 1.7%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #4199     +/-   ##
=======================================
+ Coverage   79.4%   81.2%   +1.7%     
=======================================
  Files        166     168      +2     
  Lines      10360   10406     +46     
=======================================
+ Hits        8230    8451    +221     
+ Misses      1996    1808    -188     
- Partials     134     147     +13     

see 7 files with indirect coverage changes

@MadVikingGod MadVikingGod merged commit e3f3e7a into open-telemetry:main Aug 17, 2023
26 checks passed
@MrAlias MrAlias added this to the v0.43.0 milestone Aug 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
instrumentation: otelgocql Skip Changelog Allow PR to succeed without requiring an addition to the CHANGELOG
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants