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

Fall back to unix socket on Linux if DOCKER_HOST is malformed #454

Merged
1 commit merged into from
Feb 22, 2016

Conversation

ghost
Copy link

@ghost ghost commented Feb 22, 2016

_1 Upvote_ Use of DEFAULT_DOCKER_MACHINE_URI on Linux makes little sense when DOCKER_HOST is malformed, so to us better to use unix socket in this case.

@@ -44,7 +44,7 @@
/**
* Default ip of docker host (Linux system).
*/
protected static final String DEFAULT_LINUX_DOCKER_HOST_IP = "172.17.42.1";
protected static final String DEFAULT_LINUX_DOCKER_HOST_IP = "172.17.42.1";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extra space

@garagatyi
Copy link

LGTM

@garagatyi
Copy link

@skabashnyuk ?

@skabashnyuk
Copy link
Contributor

ок

@benoitf
Copy link
Contributor

benoitf commented Feb 22, 2016

+1 by squashing the two commits

Signed-off-by: Eugene Ivantsov <eivantsov@codenvy.com>
@ghost ghost force-pushed the use_unix_socket branch from 31a91c2 to 12a3268 Compare February 22, 2016 08:55
ghost pushed a commit that referenced this pull request Feb 22, 2016
Fall back to unix socket on Linux if DOCKER_HOST is malformed
@ghost ghost merged commit 4ef98ae into master Feb 22, 2016
@ghost ghost deleted the use_unix_socket branch February 22, 2016 08:56
Katka92 pushed a commit to Katka92/che that referenced this pull request Jun 27, 2018
)

* Remove outdated token provider plugin

* Remove unused token store plugin
This pull request was closed.
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.

3 participants