fix: Make 'role' as Optional for ModelQualityMonitor and DefaultModelMonitor, and fixed PROCESSING_CONFIG_PATH (aws#849)

Co-authored-by: Balaji Sankar <[email protected]>

Author: 2 people

src/sagemaker/model_monitor/clarify_model_monitoring.py

@@ -40,7 +40,7 @@ class ClarifyModelMonitor(mm.ModelMonitor):
 
     def __init__(
         self,
-        role,
+        role=None,
         instance_count=1,
         instance_type="ml.m5.xlarge",
         volume_size_in_gb=30,

src/sagemaker/model_monitor/model_monitoring.py

@@ -1565,7 +1565,7 @@ class DefaultModelMonitor(ModelMonitor):
 
     def __init__(
         self,
-        role,
+        role=None,
         instance_count=1,
         instance_type="ml.m5.xlarge",
         volume_size_in_gb=30,
@@ -2632,7 +2632,7 @@ class ModelQualityMonitor(ModelMonitor):
 
     def __init__(
         self,
-        role,
+        role=None,
         instance_count=1,
         instance_type="ml.m5.xlarge",
         volume_size_in_gb=30,

src/sagemaker/session.py

@@ -242,9 +242,7 @@ def _simple_path(*args: str):
 PROCESSING_JOB_SECURITY_GROUP_IDS_PATH = _simple_path(
     PROCESSING_JOB_VPC_CONFIG_PATH, SECURITY_GROUP_IDS
 )
-PROCESSING_OUTPUT_CONFIG_PATH = _simple_path(
-    SAGEMAKER, PROCESSING_JOB, PROCESSING_OUTPUT_CONFIG, KMS_KEY_ID
-)
+PROCESSING_OUTPUT_CONFIG_PATH = _simple_path(SAGEMAKER, PROCESSING_JOB, PROCESSING_OUTPUT_CONFIG)
 PROCESSING_JOB_KMS_KEY_ID_PATH = _simple_path(
     SAGEMAKER, PROCESSING_JOB, PROCESSING_OUTPUT_CONFIG, KMS_KEY_ID
 )