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

[etcd] Downgrade to release-3.5 branch with otel fix. #4139

Merged
merged 3 commits into from
Aug 16, 2022

Conversation

zmt
Copy link
Contributor

@zmt zmt commented Aug 15, 2022

See etcd-io/etcd#14312 for details of the needed
PR in etcd. 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

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 andrewmains12 self-requested a review August 15, 2022 19:29
Signed-off-by: Zack Train <ztrain@uber.com>
Signed-off-by: Zack Train <ztrain@uber.com>
@zmt
Copy link
Contributor Author

zmt commented Aug 15, 2022

@andrewmains12 Please let me know if I should re-do this against a non-master branch.

@andrewmains12
Copy link
Contributor

Going to merge this temporarily into an Uber specific branch; will revive the PR against master shortly.

@andrewmains12 andrewmains12 changed the base branch from master to uber/20220816-grpc_update August 16, 2022 23:11
@andrewmains12 andrewmains12 merged commit 64acc90 into m3db:uber/20220816-grpc_update Aug 16, 2022
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