Skip to content

Commit

Permalink
Merge pull request #152 from awslabs/arpin_scikit_docker_restart
Browse files Browse the repository at this point in the history
Fixed: build_and_push.sh to include docker restart command
  • Loading branch information
djarpin authored Dec 19, 2017
2 parents 0febd3a + 4ffcb63 commit b81fe4b
Showing 1 changed file with 7 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,13 @@ $(aws ecr get-login --region ${region} --no-include-email)

# Build the docker image locally with the image name and then push it to ECR
# with the full name.

# On a SageMaker Notebook Instance, the docker daemon may need to be restarted in order
# to detect your network configuration correctly. (This is a known issue.)
if [ -d "/home/ec2-user/SageMaker" ]; then
sudo service docker restart
fi

docker build -t ${image} .
docker tag ${image} ${fullname}

Expand Down

0 comments on commit b81fe4b

Please sign in to comment.