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_addons - Adding variable with default for kubectl bin. #9616

Merged
merged 1 commit into from Jun 16, 2015

Conversation

jeffbean
Copy link
Contributor

Added a way to specify the kubectl path. Issue seen using the CentOS yum version of Kubernetes installs the kubectl in /usr/bin.

Fixes issue #9599

@k8s-bot
Copy link

k8s-bot commented Jun 10, 2015

Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist")

If this message is too spammy, please complain to ixdy.

@j3ffml
Copy link
Contributor

j3ffml commented Jun 15, 2015

lgtm.

@j3ffml
Copy link
Contributor

j3ffml commented Jun 15, 2015

ok to test

@j3ffml j3ffml added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 15, 2015
@k8s-bot
Copy link

k8s-bot commented Jun 15, 2015

GCE e2e build/test passed for commit 99f64a8.

fabioy added a commit that referenced this pull request Jun 16, 2015
kube_addons - Adding variable with default for kubectl bin.
@fabioy fabioy merged commit 9b3e9a5 into kubernetes:master Jun 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants