Merge pull request #3 from 1ambda/fix/remove-startzeppelinsh

mfelgamal · web-flow · commit cc8493fe047a · 2016-12-13T12:34:39.000+02:00
fix: Remove start-zeppelin.sh
diff --git a/docs/install/docker.md b/docs/install/docker.md
@@ -51,7 +51,7 @@ docker run --rm -it -p 7077:7077 -p 8080:8080 ${DOCKER_USERNAME}/zeppelin-releas
 
 * Run zeppelin inside docker:
 ```
-start-zeppelin.sh start
+/usr/local/zeppelin/bin/zeppelin.sh
 ```
 
 * To Run Zeppelin in daemon mode
@@ -62,7 +62,7 @@ docker run -p 7077:7077 -p 8080:8080 --privileged=true -v $PWD/logs:/logs -v $PW
 -e ZEPPELIN_NOTEBOOK_DIR='/notebook' \
 -e ZEPPELIN_LOG_DIR='/logs' \
 -d ${DOCKER_USERNAME}/zeppelin-release:<release-version> \
- bash -c "start-zeppelin.sh restart && while true; do sleep 3; done"
+/usr/local/zeppelin/bin/zeppelin.sh
 ```
 
 
diff --git a/scripts/docker/zeppelin-base/Dockerfile b/scripts/docker/zeppelin-base/Dockerfile
@@ -36,9 +36,6 @@ RUN apk add --update bash curl openjdk7-jre wget ca-certificates python build-ba
 RUN wget -O /usr/local/bin/dumb-init https://github.com/Yelp/dumb-init/releases/download/v1.1.3/dumb-init_1.1.3_amd64
 RUN chmod +x /usr/local/bin/dumb-init
 
-ADD start-zeppelin.sh /usr/bin/start-zeppelin.sh
-RUN chmod a+x /usr/bin/start-zeppelin.sh
-
 # ports for zeppelin
 EXPOSE 8080 7077
 
diff --git a/scripts/docker/zeppelin-base/start-zeppelin.sh b/scripts/docker/zeppelin-base/start-zeppelin.sh
