Skip to content

Commit b81fe4b

Browse files
authored
Merge pull request aws#152 from awslabs/arpin_scikit_docker_restart
Fixed: build_and_push.sh to include docker restart command
2 parents 0febd3a + 4ffcb63 commit b81fe4b

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

advanced_functionality/scikit_bring_your_own/container/build_and_push.sh

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,13 @@ $(aws ecr get-login --region ${region} --no-include-email)
4343

4444
# Build the docker image locally with the image name and then push it to ECR
4545
# with the full name.
46+
47+
# On a SageMaker Notebook Instance, the docker daemon may need to be restarted in order
48+
# to detect your network configuration correctly. (This is a known issue.)
49+
if [ -d "/home/ec2-user/SageMaker" ]; then
50+
sudo service docker restart
51+
fi
52+
4653
docker build -t ${image} .
4754
docker tag ${image} ${fullname}
4855

0 commit comments

Comments
 (0)