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

Upgrade etcd to 3.5 #1067

Closed

Conversation

sagikazarmark
Copy link
Contributor

As of 3.5 etcd properly uses go modules.

Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com>
@peterbourgon
Copy link
Member

Hooray!

@sagikazarmark
Copy link
Contributor Author

Sadly, it's still alpha though, hence the draft PR. But if you are okay with alpha versions, I can mark it as ready.

@peterbourgon
Copy link
Member

@sagikazarmark Is 3.4 impossible?

@sagikazarmark
Copy link
Contributor Author

@peterbourgon 3.4 won't receive proper go modules support. Furthermore, 3.5 introduces separate client modules which means less dependencies (we don't have to rely on the server stuff which probably has more deps).

@peterbourgon
Copy link
Member

Issues keep piling up related to this dependency. Can we justifiably use https://github.com/etcd-io/etcd/releases/tag/v3.5.0-beta.3?

@sagikazarmark
Copy link
Contributor Author

TBH I wouldn't recommend that. At least there were quite a few fixes to the release pipeline, so I'd probably wait until it's more stable.

The stable should be close IMHO.

Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com>
@sagikazarmark sagikazarmark marked this pull request as ready for review May 28, 2021 19:36
@sagikazarmark
Copy link
Contributor Author

Looks like the packaging issues were sorted out.

Comment on lines -52 to +55
go.etcd.io/etcd v0.0.0-20191023171146-3cf2f69b5738
go.etcd.io/etcd/client/pkg/v3 v3.5.0-beta.4
go.etcd.io/etcd/client/v2 v2.305.0-beta.4
go.etcd.io/etcd/client/v3 v3.5.0-beta.4
Copy link
Member

Choose a reason for hiding this comment

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

I can't say I understand why all of these are here. Maybe a quick summary?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

go.etcd.io/etcd/client/v2: used by sd/etcd (v2 client became a separate module)
go.etcd.io/etcd/client/v3: used by sd/etcdv3
go.etcd.io/etcd/client/pkg/v3: some common client code has been extracted to separate a pkg module (go-kit uses the transport package from there)

etcd decided to slice up the project into smaller modules.

Does that help?

peterbourgon added a commit that referenced this pull request May 30, 2021
peterbourgon added a commit that referenced this pull request May 30, 2021
peterbourgon added a commit that referenced this pull request May 30, 2021
@peterbourgon
Copy link
Member

#1097

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