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

kube-up: fix gcloud version check #73

Conversation

danielnorberg
Copy link
Contributor

e2e-test fails with

ERROR: (gcloud) unrecognized arguments: --next-hop-instance-zone us-central1-b

and

$ gcloud version | grep compute | cut -f 2 -d ' '
2014.06.11

@jbeda
Copy link
Contributor

jbeda commented Jun 12, 2014

Thanks for this PR. The new flags are going to land at some point, but the version isn't known yet. I have another PR to fix up with the known good version but I'm going to switch back to our non-prerelease tool (gcutil) in a future PR.

I'm accepting this in spirit but will push the right fix in quickly to stop breakage. Thanks!

@jbeda jbeda closed this Jun 12, 2014
@proppy
Copy link
Contributor

proppy commented Jun 12, 2014

FYI, the PR @jbeda is referring to is #78

feiskyer added a commit to feiskyer/kubernetes that referenced this pull request Feb 23, 2016
vishh pushed a commit to vishh/kubernetes that referenced this pull request Apr 6, 2016
lazypower pushed a commit to lazypower/kubernetes that referenced this pull request Nov 3, 2016
Updating the action, and templates to get current with k8s upstream.
xingzhou pushed a commit to xingzhou/kubernetes that referenced this pull request Dec 15, 2016
euank pushed a commit to euank/kubernetes that referenced this pull request Jan 20, 2017
iaguis pushed a commit to kinvolk/kubernetes that referenced this pull request Feb 6, 2018
whypro pushed a commit to whypro/kubernetes that referenced this pull request Dec 13, 2018
ry4nz added a commit to ry4nz/kubernetes that referenced this pull request Mar 26, 2019
kukushal pushed a commit to gridgain-solutions/kubernetes that referenced this pull request Jan 19, 2021
kukushal pushed a commit to gridgain-solutions/kubernetes that referenced this pull request Jan 19, 2021
kukushal pushed a commit to gridgain-solutions/kubernetes that referenced this pull request Jan 19, 2021
kukushal pushed a commit to gridgain-solutions/kubernetes that referenced this pull request Jan 21, 2021
kukushal pushed a commit to gridgain-solutions/kubernetes that referenced this pull request Jan 21, 2021
kukushal pushed a commit to gridgain-solutions/kubernetes that referenced this pull request Jan 21, 2021
kukushal pushed a commit to gridgain-solutions/kubernetes that referenced this pull request Jan 21, 2021
kukushal pushed a commit to gridgain-solutions/kubernetes that referenced this pull request Jan 22, 2021
kukushal pushed a commit to gridgain-solutions/kubernetes that referenced this pull request Jan 22, 2021
kukushal pushed a commit to gridgain-solutions/kubernetes that referenced this pull request Jan 22, 2021
kukushal pushed a commit to gridgain-solutions/kubernetes that referenced this pull request Jan 26, 2021
b3atlesfan pushed a commit to b3atlesfan/kubernetes that referenced this pull request Feb 5, 2021
Some environments may assign multiple IP addresses
on a single interface. It's better to use a global
address to a link-local one (which was probably used
for bootstrapping).

Fixes kubernetes#73
sttts pushed a commit to sttts/kubernetes that referenced this pull request Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants