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

Use internal IP if specified. Else on Linux it is not matching the expecting interface #5275

Merged
merged 1 commit into from
Jun 4, 2017

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Jun 2, 2017

What does this PR do?

Routes created for Traefik were not containing the same ip than the hosts returned by the /api/workspace runtime links, so the routes didn't match and all traffic was routed to the default route : workspace master

What issues does this PR fix or reference?

#5240

Changelog

Use CHE_IP system provoperty if defined.

Change-Id: I62c766a3ef8dd1dac6831a5fbc986e555b14586b
Signed-off-by: Florent BENOIT [email protected]

…pecting interface name.

Change-Id: I62c766a3ef8dd1dac6831a5fbc986e555b14586b
Signed-off-by: Florent BENOIT <[email protected]>
@benoitf benoitf added kind/bug Outline of a bug - must adhere to the bug report template. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. labels Jun 2, 2017
@benoitf benoitf self-assigned this Jun 2, 2017
@benoitf benoitf requested review from skabashnyuk, slemeur and a user June 2, 2017 15:16
@codenvy-ci
Copy link

@benoitf benoitf merged commit 9b6546c into master Jun 4, 2017
@benoitf benoitf deleted the fix-single-port-linux-internalip branch June 4, 2017 08:29
@benoitf benoitf removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Jun 4, 2017
@benoitf benoitf added this to the 5.12.0 milestone Jun 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants