-
Notifications
You must be signed in to change notification settings - Fork 38.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
Fix local-cluster-up on Mac and discovery on all systems #38389
Fix local-cluster-up on Mac and discovery on all systems #38389
Conversation
sttts
commented
Dec 8, 2016
- use cfssl on the host, not from Docker. Solves a number of permission problem with selinux and Mac
- fix discovery startup
c356a58
to
c5b6628
Compare
roll in #38359 too |
Actually, lets just make the whole thing switchable on |
@@ -147,6 +147,14 @@ function test_docker { | |||
fi | |||
} | |||
|
|||
function test_cfssl { |
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.
Should be test_cfssl_installed
@@ -147,6 +147,14 @@ function test_docker { | |||
fi | |||
} | |||
|
|||
function test_cfssl { | |||
if ! cfssl &>/dev/null || ! cfssljson &>/dev/null; then |
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.
cfssljson
seems to hang for me (probably because it waits for something in stdin). How about using which
or something else to verify the binary exists?
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.
Fixed already.
c5b6628
to
49e9247
Compare
49e9247
to
983fc20
Compare
LGTM, unbreaks hack/local-up-cluster.sh on master |
@deads2k do you want something more here? |
this looks fine. |
Automatic merge from submit-queue |