-
Notifications
You must be signed in to change notification settings - Fork 4.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
Bump etcd to v3.1.0 GA #12600
Bump etcd to v3.1.0 GA #12600
Conversation
9e9e1d5
to
e22997b
Compare
Just a flake, @liggitt as requested. See justification for the two hacks.
|
@@ -2960,7 +2960,7 @@ var _ grpc.ClientConn | |||
|
|||
// This is a compile-time assertion to ensure that this generated file | |||
// is compatible with the grpc package it is being compiled against. | |||
const _ = grpc.SupportPackageIsVersion3 | |||
const _ = grpc.SupportPackageIsVersion4 |
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.
how is this part of kube using a different version of grpc than us?
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.
Kube is still on etcd 3.1.0-rc1 track.
[test] |
Evaluated for origin test up to e22997b |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/13184/) (Base Commit: 6822a9b) |
[merge] |
As per Jordan's verbal OK and the desire to align with what we are going to ship sooner rather than later. |
Evaluated for origin merge up to e22997b |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/13184/) (Base Commit: fac9b78) (Image: devenv-rhel7_5757) |
Had to workaround GRPC incompatibilities with CRI and rkt, neither of which we are actually using in 1.5.
[test]