Skip to content

Commit 5a2b3f0

Browse files
committed
Updates: BYO examples to remove docker restart
1 parent e2f6338 commit 5a2b3f0

File tree

3 files changed

+3
-18
lines changed

3 files changed

+3
-18
lines changed

advanced_functionality/r_bring_your_own/r_bring_your_own.ipynb

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -328,12 +328,6 @@
328328
"# Get the login command from ECR and execute it directly\n",
329329
"$(aws ecr get-login --region ${region} --no-include-email)\n",
330330
"\n",
331-
"# On a SageMaker Notebook Instance, the docker daemon may need to be restarted in order\n",
332-
"# to detect your network configuration correctly. (This is a known issue.)\n",
333-
"if [ -d \"/home/ec2-user/SageMaker\" ]; then\n",
334-
" sudo service docker restart\n",
335-
"fi\n",
336-
"\n",
337331
"# Build the docker image locally with the image name and then push it to ECR\n",
338332
"# with the full name.\n",
339333
"docker build -t ${algorithm_name} .\n",

advanced_functionality/scikit_bring_your_own/container/build_and_push.sh

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,9 @@ then
1313
exit 1
1414
fi
1515

16+
chmod +x decision_trees/train
17+
chmod +x decision_trees/serve
18+
1619
# Get the account number associated with the current IAM credentials
1720
account=$(aws sts get-caller-identity --query Account --output text)
1821

@@ -44,12 +47,6 @@ $(aws ecr get-login --region ${region} --no-include-email)
4447
# Build the docker image locally with the image name and then push it to ECR
4548
# with the full name.
4649

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-
5350
docker build -t ${image} .
5451
docker tag ${image} ${fullname}
5552

advanced_functionality/scikit_bring_your_own/scikit_bring_your_own.ipynb

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -271,12 +271,6 @@
271271
"# Build the docker image locally with the image name and then push it to ECR\n",
272272
"# with the full name.\n",
273273
"\n",
274-
"# On a SageMaker Notebook Instance, the docker daemon may need to be restarted in order\n",
275-
"# to detect your network configuration correctly. (This is a known issue.)\n",
276-
"if [ -d \"/home/ec2-user/SageMaker\" ]; then\n",
277-
" sudo service docker restart\n",
278-
"fi\n",
279-
"\n",
280274
"docker build -t ${algorithm_name} .\n",
281275
"docker tag ${algorithm_name} ${fullname}\n",
282276
"\n",

0 commit comments

Comments
 (0)