fix: prepare container def accept_eula=False

Author: evakravi

src/sagemaker/model.py

@@ -648,7 +648,9 @@ def prepare_container_def(
             self.repacked_model_data or self.model_data,
             deploy_env,
             image_config=self.image_config,
-            accept_eula=accept_eula or getattr(self, "accept_eula", None),
+            accept_eula=accept_eula
+            if accept_eula is not None
+            else getattr(self, "accept_eula", None),
         )
 
     def is_repack(self) -> bool:

tests/unit/sagemaker/model/test_model.py

@@ -337,6 +337,21 @@ def test_create_sagemaker_model_with_eula(prepare_container_def, sagemaker_sessi
     )
 
 
+@patch("sagemaker.model.Model.prepare_container_def")
+def test_create_sagemaker_model_with_eula_false(prepare_container_def, sagemaker_session):
+    model = Model(MODEL_IMAGE, MODEL_DATA, name=MODEL_NAME, sagemaker_session=sagemaker_session)
+
+    accelerator_type = "ml.eia.medium"
+    model.create(INSTANCE_TYPE, accelerator_type=accelerator_type, accept_eula=False)
+
+    prepare_container_def.assert_called_with(
+        INSTANCE_TYPE,
+        accelerator_type=accelerator_type,
+        serverless_inference_config=None,
+        accept_eula=False,
+    )
+
+
 @patch("sagemaker.model.Model.prepare_container_def")
 def test_create_sagemaker_model_tags(prepare_container_def, sagemaker_session):
     container_def = {"Image": MODEL_IMAGE, "Environment": {}, "ModelDataUrl": MODEL_DATA}