-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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 to k8s-v1.7 #1693
Upgrade to k8s-v1.7 #1693
Conversation
* Added a godep-update-k8s script for upgrading. $ KUBE_VERSION ./hack/godeps/godep-update-k8s.sh * Added client-go version ldflags to localkube and minikube. The version package is now split in both kubernetes and client-go, so we need to supply both ldflags. * Change cross builds to use BUILD_IN_DOCKER * Add a patch for kube-proxy The setup of kube-proxy recently changed, making the registering of the component config a fatal error instead of a warning. This patch reverts it to be a warning, which is a no-op for us anyways since we don't supply a config file.
The k8s components are registering their flags on localkube, polluting the flag namespace. A 'version' flag was added, which conflicts with a localkube flag. Changing the name of this flag so that there is no conflict.
Also switch out-of-order format strings in test output.
Codecov Report
@@ Coverage Diff @@
## master #1693 +/- ##
==========================================
- Coverage 38.77% 38.55% -0.22%
==========================================
Files 51 51
Lines 2664 2674 +10
==========================================
- Hits 1033 1031 -2
- Misses 1452 1463 +11
- Partials 179 180 +1
Continue to review full report at Codecov.
|
Was this just a rebase + merge from the 1.7 branch? |
Yup, small rebase conflict in the makefile, but that was easily resolved. |
No description provided.