Minor update to default prefix

Author: brightsparc

README.md

@@ -133,17 +133,26 @@ Then, click the **Add inline policy** link, switch to to the **JSON** tab, and p
         {
             "Effect": "Allow",
             "Action": [
-                "iam:AttachRolePolicy",
                 "iam:CreateRole",
-                "iam:GetRole",
-                "iam:PutRolePolicy",
                 "iam:PassRole",
-                "iam:DetachRolePolicy",
-                "iam:DeleteRolePolicy",
                 "iam:DeleteRole"
             ],
             "Resource": "arn:aws:iam::*:role/ab-testing-api-*"
         },
+        {
+            "Effect": "Allow",
+            "Action": [
+                "iam:GetRole",
+                "iam:AttachRolePolicy",
+                "iam:PutRolePolicy",
+                "iam:DetachRolePolicy",
+                "iam:DeleteRolePolicy"
+            ],
+            "Resource": [
+              "arn:aws:iam::*:role/ab-testing-api-*",
+              "arn:aws:iam::*:role/service-role/AmazonSageMakerServiceCatalogProductsLaunchRole"
+            ]
+        },
         {
             "Effect": "Allow",
             "Action": [
@@ -181,7 +190,7 @@ Follow are a list of context values that are provided in the `cdk.json`, which c
 |---------------------------|-----------------------------------------------------------------------------------------------------------------------------------------------------------------|------------------------------------|
 | `api_name`                | The API Gateway Name                                                                                                                                            | "ab-testing"                       |
 | `stage_name`              | The stage namespace for resource and API Gateway path                                                                                                           | "dev"                              |
-| `endpoint_prefix`         | A prefix to filter Amazon SageMaker endpoints the API can invoke.                                                                                               | ""                                 |
+| `endpoint_prefix`         | A prefix to filter Amazon SageMaker endpoints the API can invoke.                                                                                               | "sagemaker-"                       |
 | `api_lambda_memory`       | The [lambda memory](https://docs.aws.amazon.com/lambda/latest/dg/configuration-memory.html) allocation for API endpoint.                                        | 768                                |
 | `api_lambda_timeout`      | The lambda timeout for the API endpoint.                                                                                                                        | 10                                 |
 | `metrics_lambda_memory`   | The [lambda memory](https://docs.aws.amazon.com/lambda/latest/dg/configuration-memory.html) allocated for metrics processing Lambda                             | 768                                |
@@ -196,7 +205,7 @@ Follow are a list of context values that are provided in the `cdk.json`, which c
 Run the following command to deploy the API and testing infrastructure, optionally override context values.
 
 ```
-cdk deploy ab-testing-api -c endpoint_prefix=ab-testing-pipeline
+cdk deploy ab-testing-api -c endpoint_prefix=sagemaker-ab-testing-pipeline
 ```
 
 This stack will ask you to confirm any changes, and output the `ApiEndpoint` which you will provide to the A/B Testing sample notebook.

cdk.json

@@ -11,7 +11,7 @@
     "log_level": "INFO",
     "api_name": "ab-testing",
     "stage_name": "dev",
-    "endpoint_prefix": "",
+    "endpoint_prefix": "sagemaker-",
     "api_lambda_memory": 768,
     "api_lambda_timeout": 60,
     "metrics_lambda_memory": 768,

notebook/mab-reviews-helpfulness.ipynb

@@ -380,7 +380,7 @@
     "\n",
     "# Create SageMaker client with up to 10 retries\n",
     "config = Config(retries={\"max_attempts\": 10, \"mode\": \"standard\"})\n",
-    "sm_client = boto3.client('sagemaker')\n",
+    "sm_client = boto3.client('sagemaker', config=config)\n",
     "\n",
     "# Get the project name\n",
     "response = sm_client.describe_project(ProjectName=project_name)\n",
@@ -666,7 +666,7 @@
    "source": [
     "### Track Lineage\n",
     "\n",
-    "List lineage associated with the training job "
+    "List lineage associated with the training job."
    ]
   },
   {
@@ -712,6 +712,13 @@
     "associate_trial_component(training_job_arn, trial_name)"
    ]
   },
+  {
+   "cell_type": "markdown",
+   "metadata": {},
+   "source": [
+    "Then list the metrics for this experiment."
+   ]
+  },
   {
    "cell_type": "code",
    "execution_count": null,
@@ -747,10 +754,11 @@
    "source": [
     "# list all packages and select the latest one\n",
     "packages = sm_client.list_model_packages(ModelPackageGroupName=champion_model_group,\n",
-    "                                        SortBy='CreationTime', SortOrder='Descending')['ModelPackageSummaryList']\n",
+    "                                        SortBy='CreationTime', SortOrder='Descending',\n",
+    "                                        MaxResults=1)['ModelPackageSummaryList']\n",
     "\n",
-    "# Approve the top to models\n",
-    "for package in packages[:2]:\n",
+    "# Approve the top model\n",
+    "for package in packages:\n",
     "    latest_model_package_arn = package['ModelPackageArn']\n",
     "    model_package_version = latest_model_package_arn.split('/')[-1]\n",
     "    if package['ModelApprovalStatus'] == 'PendingManualApproval':\n",
@@ -1100,7 +1108,7 @@
     "\n",
     "### Invoke endpoint\n",
     "\n",
-    "Ensure that our endpoint is in service.  Set the stage_name here if not `dev`\n",
+    "Ensure that our endpoint is in service.  Set the `stage_name` here if not `dev`\n",
     "\n",
     "👇👇👇"
    ]