fix: switch FrameworkProcessor from bash to sh

/bin/sh is more portable than /bin/bash, as the latter may not be
present on some optimized, container-oriented Linux distributions
(e.g. Alpine). Improves compatibility for custom containers.

Author: athewsey

src/sagemaker/processing.py

@@ -1265,7 +1265,7 @@ class FeatureStoreOutput(ApiObject):
 class FrameworkProcessor(ScriptProcessor):
     """Handles Amazon SageMaker processing tasks for jobs using a machine learning framework."""
 
-    framework_entrypoint_command = ["/bin/bash"]
+    framework_entrypoint_command = ["/bin/sh"]
 
     # Added new (kw)args for estimator. The rest are from ScriptProcessor with same defaults.
     def __init__(
@@ -1664,15 +1664,15 @@ def _generate_framework_script(self, user_script: str) -> str:
         """
         return dedent(
             """\
-            #!/bin/bash
+            #!/bin/sh
 
             cd /opt/ml/processing/input/code/
             tar -xzf sourcedir.tar.gz
 
             # Exit on any error. SageMaker uses error code to mark failed job.
             set -e
 
-            if [[ -f 'requirements.txt' ]]; then
+            if [ -f 'requirements.txt' ]; then
                 # Some py3 containers has typing, which may breaks pip install
                 pip uninstall --yes typing
 

tests/unit/sagemaker/huggingface/test_processing.py

@@ -134,7 +134,7 @@ def _get_expected_args_modular_code(job_name, code_s3_uri=f"s3://{BUCKET_NAME}")
         "app_specification": {
             "ImageUri": CUSTOM_IMAGE_URI,
             "ContainerEntrypoint": [
-                "/bin/bash",
+                "/bin/sh",
                 "/opt/ml/processing/input/entrypoint/runproc.sh",
             ],
         },

tests/unit/test_processing.py

@@ -887,7 +887,7 @@ def _get_expected_args_modular_code(job_name, code_s3_uri=f"s3://{BUCKET_NAME}")
         "app_specification": {
             "ImageUri": CUSTOM_IMAGE_URI,
             "ContainerEntrypoint": [
-                "/bin/bash",
+                "/bin/sh",
                 "/opt/ml/processing/input/entrypoint/runproc.sh",
             ],
         },
@@ -1128,7 +1128,7 @@ def _get_expected_args_all_parameters_modular_code(
             "ImageUri": "012345678901.dkr.ecr.us-west-2.amazonaws.com/my-custom-image-uri",
             "ContainerArguments": ["--drop-columns", "'SelfEmployed'"],
             "ContainerEntrypoint": [
-                "/bin/bash",
+                "/bin/sh",
                 "/opt/ml/processing/input/entrypoint/runproc.sh",
             ],
         },