You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by zh...@apache.org on 2017/09/06 03:27:58 UTC

[bookkeeper] branch master updated: ISSUE #496: some docker images don't need port forwarding

This is an automated email from the ASF dual-hosted git repository.

zhaijia pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git


The following commit(s) were added to refs/heads/master by this push:
     new 96a45e2  ISSUE #496: some docker images don't need port forwarding
96a45e2 is described below

commit 96a45e2a5733c91d89f9e91755ac3f42fdcea4d4
Author: Sijie Guo <si...@apache.org>
AuthorDate: Wed Sep 6 11:27:45 2017 +0800

    ISSUE #496: some docker images don't need port forwarding
    
    Descriptions of the changes in this PR:
    
    some docker images don't need port forwarding. remove port forwarding from them.
    
    Author: Sijie Guo <si...@apache.org>
    
    Reviewers: Jia Zhai <None>
    
    This closes #497 from sijie/remove_docker_port, closes #496
---
 dev/docker/ci.sh  | 1 -
 dev/docker/run.sh | 1 -
 site/docker/ci.sh | 1 -
 3 files changed, 3 deletions(-)

diff --git a/dev/docker/ci.sh b/dev/docker/ci.sh
index bb268f8..faacea3 100755
--- a/dev/docker/ci.sh
+++ b/dev/docker/ci.sh
@@ -56,7 +56,6 @@ docker run \
   -u "${USER}" \
   -v "${BOOKKEEPER_ROOT}:${BOOKKEEPER_ROOT}" \
   -v "${LOCAL_HOME}:/home/${USER_NAME}" \
-  -p 4000:4000 \
   -e MAVEN_OPTS='-Xmx4g -Xms2g' \
   ${IMAGE_NAME}-${USER_NAME} \
   bash -c "mvn clean apache-rat:check package findbugs:check"
diff --git a/dev/docker/run.sh b/dev/docker/run.sh
index dbca46c..235cba7 100755
--- a/dev/docker/run.sh
+++ b/dev/docker/run.sh
@@ -65,7 +65,6 @@ docker run -i -t \
   -u "${USER}" \
   -v "${BOOKKEEPER_ROOT}:${BOOKKEEPER_ROOT}" \
   -v "${LOCAL_HOME}:/home/${USER_NAME}" \
-  -p 4000:4000 \
   ${IMAGE_NAME}-${USER_NAME} \
   bash -c "${CMD}"
 
diff --git a/site/docker/ci.sh b/site/docker/ci.sh
index 262e2b3..44c4e23 100755
--- a/site/docker/ci.sh
+++ b/site/docker/ci.sh
@@ -56,7 +56,6 @@ docker run \
   -u "${USER}" \
   -v "${BOOKKEEPER_DOC_ROOT}:${BOOKKEEPER_DOC_ROOT}" \
   -v "${LOCAL_HOME}:/home/${USER_NAME}" \
-  -p 4000:4000 \
   -e JEKYLL_ENV='production' \
   ${IMAGE_NAME}-${USER_NAME} \
   bash -c "make setup && make apache"

-- 
To stop receiving notification emails like this one, please contact
['"commits@bookkeeper.apache.org" <co...@bookkeeper.apache.org>'].