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

[3.5] etcdserver: bump OpenTelemetry to 1.0.1 and gRPC to 1.41.0 #14312

Merged
merged 5 commits into from Aug 8, 2022

Conversation

ahrtr
Copy link
Member

@ahrtr ahrtr commented Aug 4, 2022

Fix #14311 by cherry picking #13361 to 3.5.

Signed-off-by: Benjamin Wang wachao@vmware.com

Please read https://github.com/etcd-io/etcd/blob/main/CONTRIBUTING.md#contribution-flow.

@ahrtr ahrtr marked this pull request as draft August 4, 2022 22:08
@ahrtr ahrtr force-pushed the 3.5_bump_otl branch 7 times, most recently from 28b114c to 18a87dd Compare August 4, 2022 23:57
@ahrtr ahrtr changed the title etcdserver: bump OpenTelemetry to 1.0.1 [3.5] etcdserver: bump OpenTelemetry to 1.0.1 Aug 5, 2022
@ahrtr ahrtr changed the title [3.5] etcdserver: bump OpenTelemetry to 1.0.1 [3.5] etcdserver: bump OpenTelemetry to 1.0.1 and gRPC to 1.41.0 Aug 5, 2022
@ahrtr ahrtr marked this pull request as ready for review August 5, 2022 04:46
@ahrtr
Copy link
Member Author

ahrtr commented Aug 5, 2022

cc @spzala @ptabor @serathius Please take a look, thx.

cc @nikhita @dims

Signed-off-by: Benjamin Wang <wachao@vmware.com>
Signed-off-by: Benjamin Wang <wachao@vmware.com>
Upgrade grpc to 1.41.0;
Run ./script/fix.sh to fix all related issue.

Signed-off-by: Benjamin Wang <wachao@vmware.com>
Refer to etcd-io@a0bdfc4

Signed-off-by: Benjamin Wang <wachao@vmware.com>
Refer to etcd-io#14318

Signed-off-by: Benjamin Wang <wachao@vmware.com>
@ahrtr
Copy link
Member Author

ahrtr commented Aug 6, 2022

cc @dashpole

Just rebased the PR to merge @damemi 's PR #14318

Copy link
Member

@spzala spzala left a comment

Choose a reason for hiding this comment

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

lgtm
Thanks, @ahrtr !! Please squash commits before merging.

@ahrtr
Copy link
Member Author

ahrtr commented Aug 8, 2022

lgtm Thanks, @ahrtr !! Please squash commits before merging.

Thanks @spzala . I intentionally created a separate commit for each different kind of change for better review and future archaeology (debug).

@ahrtr
Copy link
Member Author

ahrtr commented Aug 8, 2022

Thanks @dashpole for the review as well.

@ahrtr ahrtr merged commit 9e95685 into etcd-io:release-3.5 Aug 8, 2022
@spzala
Copy link
Member

spzala commented Aug 8, 2022

lgtm Thanks, @ahrtr !! Please squash commits before merging.

Thanks @spzala . I intentionally created a separate commit for each different kind of change for better review and future archaeology (debug).

Ok, cool. Thanks @ahrtr !!

@zmt zmt mentioned this pull request Aug 12, 2022
16 tasks
zmt added a commit to zmt/m3 that referenced this pull request Aug 13, 2022
zmt added a commit to zmt/m3 that referenced this pull request Aug 13, 2022
See etcd-io/etcd#14312 for details.

Signed-off-by: Zack Train <ztrain@uber.com>
zmt added a commit to zmt/m3 that referenced this pull request Aug 13, 2022
See etcd-io/etcd#14312 for details.

Signed-off-by: Zack Train <ztrain@uber.com>
zmt added a commit to zmt/m3 that referenced this pull request Aug 13, 2022
See etcd-io/etcd#14312 for details.

Signed-off-by: Zack Train <ztrain@uber.com>
zmt added a commit to zmt/m3 that referenced this pull request Aug 13, 2022
See etcd-io/etcd#14312 for details of the needed
PR. Incidentally remove the go.etcd.io/etcd/client/v2 replace directive
since M3 is already on go 1.17 and release-3.5 branch indirectly pulls
in v2.305.4 which does exhibit the problem with sigs.k8s.io/json:
```
% go mod why sigs.k8s.io/json
(main module does not need package sigs.k8s.io/json)
```

Signed-off-by: Zack Train <ztrain@uber.com>
zmt added a commit to zmt/m3 that referenced this pull request Aug 13, 2022
See etcd-io/etcd#14312 for details of the needed
PR. Incidentally remove the go.etcd.io/etcd/client/v2 replace directive
since M3 is already on go 1.17 and release-3.5 branch indirectly pulls
in v2.305.4 which does exhibit the problem with sigs.k8s.io/json:
```
% go mod why sigs.k8s.io/json
(main module does not need package sigs.k8s.io/json)
```

Signed-off-by: Zack Train <ztrain@uber.com>
zmt added a commit to zmt/m3 that referenced this pull request Aug 13, 2022
See etcd-io/etcd#14312 for details of the needed
PR. Rewrite integration package imports from the 3.6.0.alpha.0 path to
the 3.5.5 path. Incidentally remove the go.etcd.io/etcd/client/v2 replace directive
since M3 is already on go 1.17 and release-3.5 branch indirectly pulls
in v2.305.4 which does exhibit the problem with sigs.k8s.io/json:
```
% go mod why sigs.k8s.io/json
(main module does not need package sigs.k8s.io/json)
```

Signed-off-by: Zack Train <ztrain@uber.com>
andrewmains12 pushed a commit to m3db/m3 that referenced this pull request Aug 16, 2022
* Downgrade etcd to release-3.5 branch with otel fix.

See etcd-io/etcd#14312 for details of the needed
PR. Rewrite integration package imports from the 3.6.0.alpha.0 path to
the 3.5.5 path. Incidentally remove the go.etcd.io/etcd/client/v2 replace directive
since M3 is already on go 1.17 and release-3.5 branch indirectly pulls
in v2.305.4 which does exhibit the problem with sigs.k8s.io/json:
```
% go mod why sigs.k8s.io/json
(main module does not need package sigs.k8s.io/json)
```

Signed-off-by: Zack Train <ztrain@uber.com>

* Fix etcd/integration tests to use V3.

Signed-off-by: Zack Train <ztrain@uber.com>

* Missed edit for ClusterV3.

Signed-off-by: Zack Train <ztrain@uber.com>

Signed-off-by: Zack Train <ztrain@uber.com>
@chaochn47 chaochn47 mentioned this pull request Oct 13, 2023
24 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants