Skip to content

Commit 4d790a8

Browse files
authored
Merge pull request #123 from awslabs/arpin_r_byo_markdown_ecr
Fixed: Markdown on ECR permissions to use
2 parents 2bacd29 + 8829623 commit 4d790a8

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

advanced_functionality/r_bring_your_own/r_bring_your_own.ipynb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@
8888
"source": [
8989
"### Permissions\n",
9090
"\n",
91-
"Running this notebook requires permissions in addition to the normal `SageMakerFullAccess` permissions. This is because we'll be creating a new repository in Amazon ECR. The easiest way to add these permissions is simply to add the managed policy `AmazonEC2ContainerRegistryPowerUser` to the role that you used to start your notebook instance. There's no need to start your notebook instance when you do this, the new permissions will be available immediately.\n",
91+
"Running this notebook requires permissions in addition to the normal `SageMakerFullAccess` permissions. This is because we'll be creating a new repository in Amazon ECR. The easiest way to add these permissions is simply to add the managed policy `AmazonEC2ContainerRegistryFullAccess` to the role that you used to start your notebook instance. There's no need to restart your notebook instance when you do this, the new permissions will be available immediately.\n",
9292
"\n",
9393
"---\n",
9494
"## Code\n",
@@ -227,9 +227,9 @@
227227
"\n",
228228
"\n",
229229
"#' Ping to show server is there\n",
230-
"#' @post /ping\n",
230+
"#' @get /ping\n",
231231
"function() {\n",
232-
" list(status='200', code='200')}\n",
232+
" return('')}\n",
233233
"\n",
234234
"\n",
235235
"#' Parse input and return the prediction from the model\n",
@@ -268,7 +268,7 @@
268268
"```Dockerfile\n",
269269
"FROM ubuntu:16.04\n",
270270
"\n",
271-
"MAINTAINER David Arpin <arpin@amazon.com>\n",
271+
"MAINTAINER Amazon SageMaker Examples <amazon-sagemaker-examples@amazon.com>\n",
272272
"\n",
273273
"RUN apt-get -y update && apt-get install -y --no-install-recommends \\\n",
274274
" wget \\\n",

0 commit comments

Comments
 (0)