-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
vendor: update etcd to 3.4.14 #14238
Conversation
This version fixes etcd error "auth token invalid after watch reconnects" etcd-io/etcd#11954. Signed-off-by: André Martins <andre@cilium.io>
test-me-please |
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.
🚢
@@ -73,7 +73,7 @@ require ( | |||
github.com/stretchr/testify v1.6.1 | |||
github.com/vishvananda/netlink v1.1.1-0.20200603190939-5a869a71f0cb | |||
github.com/vishvananda/netns v0.0.0-20191106174202-0a2b9b5464df | |||
go.etcd.io/etcd v0.5.0-alpha.5.0.20200824191128-ae9734ed278b | |||
go.etcd.io/etcd v0.5.0-alpha.5.0.20201125193152-8a03d2e9614b |
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.
Isn't it possible to go get
the version rather than the git sha so we'd have go.etcd.io/etcd v3.4.14
? Looking at this line, it seems we're importing a very alpha version although we're not and we can't tell which version it is at first glance.
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.
@rolinh no, go get
will complain, I usually copy-paste the commit sha into the repository to confirm which version is it.
retest-gke |
restart-gke |
retest-gke |
This version fixes etcd error "auth token invalid after
watch reconnects". etcd-io/etcd#11954
Signed-off-by: André Martins andre@cilium.io