fix naming and messaging

Author: gwang111

src/sagemaker/jumpstart/model.py

@@ -665,7 +665,7 @@ def deploy(
         managed_instance_scaling: Optional[str] = None,
         endpoint_type: EndpointType = EndpointType.MODEL_BASED,
         routing_config: Optional[Dict[str, Any]] = None,
-        model_access_configs: Optional[List[ModelAccessConfig]] = None,
+        model_access_configs: Optional[Dict[str, ModelAccessConfig]] = None,
     ) -> PredictorBase:
         """Creates endpoint by calling base ``Model`` class `deploy` method.
 
@@ -766,7 +766,7 @@ def deploy(
                 ModelAccessConfig, provide a `{ "model_id", ModelAccessConfig(accept_eula=True) }`
                 to indicate whether model terms of use have been accepted. The `accept_eula` value
                 must be explicitly defined as `True` in order to accept the end-user license
-                agreement (EULA) that some. (Default: None)
+                agreement (EULA) that some models require. (Default: None)
 
         Raises:
             MarketplaceModelSubscriptionError: If the caller is not subscribed to the model.

src/sagemaker/jumpstart/utils.py

@@ -558,12 +558,12 @@ def get_eula_message(model_specs: JumpStartModelSpecs, region: str) -> str:
     """Returns EULA message to display if one is available, else empty string."""
     if model_specs.hosting_eula_key is None:
         return ""
-    return format_eula_message_template(
+    return get_formatted_eula_message_template(
         model_id=model_specs.model_id, region=region, hosting_eula_key=model_specs.hosting_eula_key
     )
 
 
-def format_eula_message_template(model_id: str, region: str, hosting_eula_key: str):
+def get_formatted_eula_message_template(model_id: str, region: str, hosting_eula_key: str) -> str:
     """Returns a formatted EULA message."""
     return (
         f"Model '{model_id}' requires accepting end-user license agreement (EULA). "
@@ -1562,13 +1562,13 @@ def _add_model_access_configs_to_model_data_sources(
                 raise ValueError(
                     eula_message_template.format(
                         model_source="Additional " if model_data_source.get("ChannelName") else "",
-                        base_eula_message=format_eula_message_template(
+                        base_eula_message=get_formatted_eula_message_template(
                             model_id=model_id, region=region, hosting_eula_key=hosting_eula_key
                         ),
                         model_access_configs_message=(
-                            " Please add a ModelAccessConfig entry:"
+                            "Please add a ModelAccessConfig entry:"
                             f" {model_access_config_entry} "
-                            "to model_access_configs to acknowledge the EULA."
+                            "to model_access_configs to accept the EULA."
                         ),
                     )
                 )