-
-
Notifications
You must be signed in to change notification settings - Fork 12.8k
kubernetes-cli 1.9.4 #25378
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
kubernetes-cli 1.9.4 #25378
Conversation
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.
I'm not eager to allow the use of go@*
formulae as dependencies in Homebrew/homebrew-core as that has not be needed at all so far. What is the blocker to using the latest Go?
The homebrew compile failure is here: #25312 |
Can we cherry-pick the fix into Homebrew's Go 1.10? |
I don't think that alone will work, as the homebrew compile failure is on a go dependency in the release branch that would need to be bumped along side the upgrade. |
That is a shame. |
Yeah.. I just tugged on the thread, and the dependency that is failing to compile is actually a dependency of dependency (cfssl). So it's unfortunately a chain reaction that's preventing it. From an upstream perspective, we pin our go versions in our build container system that we use to generate the release binaries for this reason. We've also seen large performance regressions in the past as well. So using go1.9 is the most "right" from our side. I can understand that not being the preference from the homebrew side, however. |
there is no patch for cfssl? |
There is, but it hasn't been regression tested in the 1.9 release branch, or on master. We're in the kubernetes 1.10 freeze window right now (to be released in the next two weeks), so we can't upgrade it on master, let alone on a release branch. The upstream fix path as I'd see it:
|
I don't like this precedent we're setting right now at all, but OK. |
Thanks @cblecker! 🚀 |
I completely understand, and appreciate the exception in spite of it. Hopefully this is an extremely short lived state. Not being able to be compiled on the latest go version causes all sorts of bad issues. |
brew install --build-from-source <formula>
, where<formula>
is the name of the formula you're submitting?brew audit --strict <formula>
(after doingbrew install <formula>
)?Change the build dependency for kubernetes-cli to use [email protected]. We don't support go1.10 yet for kubectl.
fixes #25312.