Skip to content

Bump to otel 1.12.0#204

Merged
tyroneyu merged 5 commits intomainfrom
bump-otel-1.12.0
Feb 6, 2023
Merged

Bump to otel 1.12.0#204
tyroneyu merged 5 commits intomainfrom
bump-otel-1.12.0

Conversation

@tyroneyu
Copy link
Copy Markdown
Contributor

@tyroneyu tyroneyu commented Feb 1, 2023

Description

Bump otel to 1.12.0

Comment thread go.mod
go.opentelemetry.io/otel/metric v0.31.0
go.opentelemetry.io/otel/sdk v1.10.0
go.opentelemetry.io/otel/trace v1.10.0
google.golang.org/grpc v1.49.0
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Need to bump metrics as well to the corresponding version: i.e go.opentelemetry.io/otel/metric I believe it's v0.35.0 https://github.com/open-telemetry/opentelemetry-go/releases/tag/v1.12.0

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks! that was the hint that I needed. I also had to bump up the contrib versions as well.

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 4, 2023

Codecov Report

Merging #204 (de7cfd5) into main (eded7db) will increase coverage by 0.13%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##             main     #204      +/-   ##
==========================================
+ Coverage   58.23%   58.37%   +0.13%     
==========================================
  Files          55       55              
  Lines        2229     2222       -7     
==========================================
- Hits         1298     1297       -1     
+ Misses        870      866       -4     
+ Partials       61       59       -2     
Impacted Files Coverage Δ
instrumentation/opentelemetry/metrics.go 0.00% <0.00%> (ø)
instrumentation/opentelemetry/init.go 82.29% <85.71%> (+1.52%) ⬆️
...atchspanprocessor/batch_span_processor.modified.go 15.67% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Comment thread instrumentation/opentelemetry/init.go
Comment thread .github/workflows/pr-test.yml Outdated
strategy:
matrix:
go: ["1.17", "1.18", "1.19"]
go: ["1.18", "1.19"]
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Add "1.20"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

For future reference, where did you read the 1.20 support? I used this link for reference to know 1.17 was gone. I did not see 1.20.
https://github.com/open-telemetry/opentelemetry-go

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

In the compatibility it says latest version - 2. https://github.com/open-telemetry/opentelemetry-go#compatibility

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

For posterity, otel hasn't updated their compatibiity chart nor their CI yet. It still only runs golang 1.18/1.19. However, golang 1.20 was released Feb 1st and otel points to https://go.dev/doc/devel/release#policy to say they support the last 2 golang versions.

It does not hurt to add this into our CI right now.

Done.

@tim-mwangi tim-mwangi requested a review from prodion23 February 6, 2023 22:01
@tyroneyu tyroneyu merged commit b433f7a into main Feb 6, 2023
@tyroneyu tyroneyu deleted the bump-otel-1.12.0 branch February 6, 2023 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants