From 4f1d9708d1afa0bd180d3e06757171762033fd4f Mon Sep 17 00:00:00 2001
From: Alexander Garagatyi <agaragatyi@codenvy.com>
Date: Mon, 29 Feb 2016 20:06:13 +0200
Subject: [PATCH] CHE-275: rename DOCKER_MACHINE_HOST to
 CHE_DOCKER_MACHINE_HOST Signed-off-by: Alexander Garagatyi
 <agaragatyi@codenvy.com>

---
 .../assembly-main/src/assembly/bin/che.sh     |  4 ++--
 ...va => LocalDockerInstanceRuntimeInfo.java} | 19 +++++++++++--------
 .../machine/local/LocalDockerModule.java      |  2 +-
 3 files changed, 14 insertions(+), 11 deletions(-)
 rename plugins/plugin-docker/che-plugin-docker-machine/src/main/java/org/eclipse/che/plugin/docker/machine/local/{LocalDockerInstanceMetadata.java => LocalDockerInstanceRuntimeInfo.java} (62%)

diff --git a/assembly/assembly-main/src/assembly/bin/che.sh b/assembly/assembly-main/src/assembly/bin/che.sh
index da9daf3cde..dd0bb7f91e 100644
--- a/assembly/assembly-main/src/assembly/bin/che.sh
+++ b/assembly/assembly-main/src/assembly/bin/che.sh
@@ -232,7 +232,7 @@ set_environment_variables () {
   fi
 
   if [[ "${CHE_IP}" != "" ]]; then
-    export DOCKER_MACHINE_HOST="${CHE_IP}"
+    export CHE_DOCKER_MACHINE_HOST="${CHE_IP}"
   fi
 
   #if [ "${WIN}" == "true" ] && [ ! -z "${JAVA_HOME}" ]; then
@@ -566,7 +566,7 @@ kill_and_launch_docker_che () {
   -v //home/user/che/lib:/home/user/che/lib-copy \
   -v //home/user/che/workspaces:/home/user/che/workspaces \
   -v //home/user/che/tomcat/temp/local-storage:/home/user/che/tomcat/temp/local-storage \
-  -e DOCKER_MACHINE_HOST=${DOCKER_PRINT_VALUE} --name ${CONTAINER} -d --net=host codenvy/che:${CHE_DOCKER_TAG} \
+  -e CHE_DOCKER_MACHINE_HOST=${DOCKER_PRINT_VALUE} --name ${CONTAINER} -d --net=host codenvy/che:${CHE_DOCKER_TAG} \
   bash -c "tail -f /dev/null" || DOCKER_EXIT=$? || true 
   set +x
 }
diff --git a/plugins/plugin-docker/che-plugin-docker-machine/src/main/java/org/eclipse/che/plugin/docker/machine/local/LocalDockerInstanceMetadata.java b/plugins/plugin-docker/che-plugin-docker-machine/src/main/java/org/eclipse/che/plugin/docker/machine/local/LocalDockerInstanceRuntimeInfo.java
similarity index 62%
rename from plugins/plugin-docker/che-plugin-docker-machine/src/main/java/org/eclipse/che/plugin/docker/machine/local/LocalDockerInstanceMetadata.java
rename to plugins/plugin-docker/che-plugin-docker-machine/src/main/java/org/eclipse/che/plugin/docker/machine/local/LocalDockerInstanceRuntimeInfo.java
index 015e0a5c84..f8b71d9e51 100644
--- a/plugins/plugin-docker/che-plugin-docker-machine/src/main/java/org/eclipse/che/plugin/docker/machine/local/LocalDockerInstanceMetadata.java
+++ b/plugins/plugin-docker/che-plugin-docker-machine/src/main/java/org/eclipse/che/plugin/docker/machine/local/LocalDockerInstanceRuntimeInfo.java
@@ -19,27 +19,30 @@ import org.eclipse.che.plugin.docker.machine.DockerInstanceRuntimeInfo;
 import javax.inject.Inject;
 import javax.inject.Named;
 
+import static com.google.common.base.MoreObjects.firstNonNull;
+
 /**
  * Gets predefined docker containers host for machine servers instead of evaluating it from docker configuration
  *
  * <p>Value of host can be retrieved from property ${code machine.docker.local_node_host} or
- * from environment variable {@code DOCKER_MACHINE_HOST}.<br>
+ * from environment variable {@code CHE_DOCKER_MACHINE_HOST}.<br>
  * Environment variable has lower priority.
  *
  * @author Alexander Garagatyi
  * @see org.eclipse.che.plugin.docker.machine.ServerConf
  */
-public class LocalDockerInstanceMetadata extends DockerInstanceRuntimeInfo {
+public class LocalDockerInstanceRuntimeInfo extends DockerInstanceRuntimeInfo {
     /**
      * Env variable that shows host (or IP) where docker machines are deployed
      */
-    public static final String DOCKER_MACHINE_HOST = "DOCKER_MACHINE_HOST";
+    public static final String CHE_DOCKER_MACHINE_HOST = "CHE_DOCKER_MACHINE_HOST";
 
     @Inject
-    public LocalDockerInstanceMetadata(@Assisted ContainerInfo containerInfo,
-                                       @Assisted String containerHost,
-                                       @Nullable @Named("machine.docker.local_node_host") String dockerNodeHost) {
-        super(containerInfo, dockerNodeHost != null ? dockerNodeHost :
-                             (System.getenv(DOCKER_MACHINE_HOST) != null ? System.getenv(DOCKER_MACHINE_HOST) : containerHost));
+    public LocalDockerInstanceRuntimeInfo(@Assisted ContainerInfo containerInfo,
+                                          @Assisted String containerHost,
+                                          @Nullable @Named("machine.docker.local_node_host") String dockerNodeHost) {
+        super(containerInfo, firstNonNull(firstNonNull(dockerNodeHost,
+                                                       System.getenv(CHE_DOCKER_MACHINE_HOST)),
+                                          containerHost));
     }
 }
diff --git a/plugins/plugin-docker/che-plugin-docker-machine/src/main/java/org/eclipse/che/plugin/docker/machine/local/LocalDockerModule.java b/plugins/plugin-docker/che-plugin-docker-machine/src/main/java/org/eclipse/che/plugin/docker/machine/local/LocalDockerModule.java
index 9ba7675a70..70e08d5cce 100644
--- a/plugins/plugin-docker/che-plugin-docker-machine/src/main/java/org/eclipse/che/plugin/docker/machine/local/LocalDockerModule.java
+++ b/plugins/plugin-docker/che-plugin-docker-machine/src/main/java/org/eclipse/che/plugin/docker/machine/local/LocalDockerModule.java
@@ -47,7 +47,7 @@ public class LocalDockerModule extends AbstractModule {
                         .implement(InstanceProcess.class, DockerProcess.class)
                         .implement(DockerNode.class, LocalDockerNode.class)
                         .implement(DockerInstanceRuntimeInfo.class,
-                                   org.eclipse.che.plugin.docker.machine.local.LocalDockerInstanceMetadata.class)
+                                   org.eclipse.che.plugin.docker.machine.local.LocalDockerInstanceRuntimeInfo.class)
                         .build(org.eclipse.che.plugin.docker.machine.DockerMachineFactory.class));