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

using go.etcd.io/etcd for etcd import path #430

Closed
wants to merge 1 commit into from

Conversation

h8liu
Copy link
Contributor

@h8liu h8liu commented Sep 11, 2018

for #429

@googlebot googlebot added the cla: yes Google CLA has been signed! label Sep 11, 2018
@h8liu
Copy link
Contributor Author

h8liu commented Sep 11, 2018

interesting that your ci is using an internal GOPROXY? is it managed by your team?

@vangent
Copy link
Contributor

vangent commented Sep 11, 2018

interesting that your ci is using an internal GOPROXY? is it managed by your team?

Yup.

@vangent vangent self-requested a review September 11, 2018 21:54
@vangent vangent self-assigned this Sep 11, 2018
Copy link
Contributor

@vangent vangent left a 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?

@h8liu
Copy link
Contributor Author

h8liu commented Sep 11, 2018

sure, np. I filed etcd-io/etcd#10083 to query about their schedules.

@gyuho
Copy link

gyuho commented Sep 11, 2018

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?

@h8liu
Copy link
Contributor Author

h8liu commented Sep 13, 2018

(I am actually not a member of go-cloud)

so quoting from:

etcd-io/etcd#10083

etcd is a binary program not a library. We have been talking about how to separate the client library versioning from the etcd binary versioning, but have not found a good way to do it.

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.

@vangent
Copy link
Contributor

vangent commented Sep 15, 2018

Can we close this PR and revisit once etcd has done a new release with the new paths (whatever they end up tagging it)?

@vangent vangent added the blocked Blocked on a different issue label Sep 15, 2018
@h8liu
Copy link
Contributor Author

h8liu commented Sep 18, 2018

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.

@h8liu h8liu closed this Sep 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Blocked on a different issue cla: yes Google CLA has been signed!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants