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

Add docker image to grab the ip used by docker #1891

Merged
merged 1 commit into from
Jul 21, 2016
Merged

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Jul 21, 2016

What does this PR do?

Introduce docker image to get ip used by docker

What issues does this PR fix or reference?

Previous Behavior

hardcoded in the launcher script but it could be done within a docker image as well to share it with other docker images

New Behavior

New 'codenvy/che-ip' docker image

Tests written?

No

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

Change-Id: Id781e34c99f14c87909197b99969d645268e62d3
Signed-off-by: Florent BENOIT <[email protected]>
@codenvy-ci
Copy link

@TylerJewell
Copy link

lgtm

@benoitf benoitf mentioned this pull request Jul 21, 2016
20 tasks
@benoitf benoitf added kind/enhancement A feature request - must adhere to the feature request template. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. labels Jul 21, 2016
@TylerJewell TylerJewell merged commit 3d9060f into master Jul 21, 2016
@TylerJewell TylerJewell added this to the 4.6.0 milestone Jul 22, 2016
@benoitf benoitf deleted the che-ip-docker branch July 25, 2016 07:42
@bmicklea bmicklea mentioned this pull request Jul 26, 2016
64 tasks
ShimonBenYair pushed a commit to ShimonBenYair/che that referenced this pull request Aug 4, 2016
Change-Id: Id781e34c99f14c87909197b99969d645268e62d3
Signed-off-by: Florent BENOIT <[email protected]>
@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 Jan 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A feature request - must adhere to the feature request template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants