-
Notifications
You must be signed in to change notification settings - Fork 814
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
using go.etcd.io/etcd for etcd import path #430
Conversation
interesting that your ci is using an internal GOPROXY? is it managed by your team? |
Yup. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems very odd to include 2 separate versions of etcd. Any objection to waiting with this change until there's a new etcd release with the new names?
sure, np. I filed etcd-io/etcd#10083 to query about their schedules. |
Hi, we are switching to go module in master branch soon. But, maybe not in release branches. Do you need us use go module for release branch as well? |
(I am actually not a member of go-cloud) so quoting from:
I got this reply when I asked them to bump the version to 4.0.0 because import path change is not backward compatible. They refused because their version number is for the binary, not for the source code (client api). As a result, using release tags from the etcd git repo in go mod is probably incorrect. |
Can we close this PR and revisit once etcd has done a new release with the new paths (whatever they end up tagging it)? |
sure, whatever. again, no matter what version they use, it is not for go module import. the version is for the binary not the library. |
for #429