Skip to content

Commit

Permalink
[Fix] revert --ssh-port to SSH_PORT (#187)
Browse files Browse the repository at this point in the history
*Issue #, if available:*

*Description of changes:*
fix the error of --ssh-port in Dockerfile.local.

By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice.

---------

Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: xiang song(charlie.song) <[email protected]>
  • Loading branch information
3 people authored and Ubuntu committed May 22, 2023
1 parent 6c31846 commit c31b35f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions docker/Dockerfile.local
Original file line number Diff line number Diff line change
Expand Up @@ -39,9 +39,9 @@ COPY code/training_scripts /graphstorm/training_scripts

# Set up SSH
RUN apt-get install -y openssh-client openssh-server
ENV --ssh-port=2222
ENV SSH_PORT=2222
RUN cat /etc/ssh/sshd_config > /tmp/sshd_config && \
sed "0,/^#Port 22/s//Port ${--ssh-port}/" /tmp/sshd_config > /etc/ssh/sshd_config
sed "0,/^#Port 22/s//Port ${SSH_PORT}/" /tmp/sshd_config > /etc/ssh/sshd_config
ENV SSHDIR $HOME/.ssh
RUN mkdir -p ${SSHDIR}
RUN ssh-keygen -t rsa -f ${SSHDIR}/id_rsa -N ''
Expand Down
2 changes: 1 addition & 1 deletion docker/build_docker_oss4local.sh
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ cp -r $GSF_HOME"training_scripts" $GSF_HOME"docker/code/training_scripts"
DOCKER_FULLNAME="${IMAGE_NAME}:${TAG}"

echo "Build a local docker image ${DOCKER_FULLNAME}"
docker build -f $GSF_HOME"docker/Dockerfile.local" . -t $DOCKER_FULLNAME
docker build --no-cache -f $GSF_HOME"docker/Dockerfile.local" . -t $DOCKER_FULLNAME

# remove the temporary code folder
rm -rf $GSF_HOME"docker/code"

0 comments on commit c31b35f

Please sign in to comment.