chore: address PR comments

Author: evakravi

src/sagemaker/utils.py

@@ -41,6 +41,7 @@
 from sagemaker.config.config_utils import (
     _log_sagemaker_config_single_substitution,
     _log_sagemaker_config_merge,
+    get_sagemaker_config_logger,
 )
 from sagemaker.session_settings import SessionSettings
 from sagemaker.workflow import is_pipeline_variable, is_pipeline_parameter_string
@@ -1099,11 +1100,13 @@ def get_sagemaker_config_value(sagemaker_session, key):
         return None
 
     if (
-        sagemaker_session is not None
-        and sagemaker_session.settings is not None
+        sagemaker_session.settings is not None
         and sagemaker_session.settings.ignore_sagemaker_config
     ):
-        logger.info("Ignoring defaults config.")
+        get_sagemaker_config_logger().info(
+            "Ignoring defaults config because "
+            "`ignore_sagemaker_config` was set in Session settings."
+        )
         return None
 
     if sagemaker_session.sagemaker_config:

tests/unit/test_utils.py

@@ -1177,8 +1177,14 @@ def test_retry_with_backoff(patched_sleep):
     assert retry_with_backoff(callable_func, 2) == func_return_val
 
 
-@patch("sagemaker.utils.logger.info")
-def test_resolve_value_from_config(mock_info_logger):
+@patch("sagemaker.utils.get_sagemaker_config_logger")
+def test_resolve_value_from_config(mock_get_sagemaker_config_logger):
+
+    mock_config_logger = Mock()
+    mock_get_sagemaker_config_logger.return_value = mock_config_logger
+
+    mock_info_logger = Mock()
+    mock_config_logger.info = mock_info_logger
     # using a shorter name for inside the test
     sagemaker_session = MagicMock()
     sagemaker_session.settings.ignore_sagemaker_config = False
@@ -1235,12 +1241,16 @@ def test_resolve_value_from_config(mock_info_logger):
 
     sagemaker_session.settings.ignore_sagemaker_config = True
     assert resolve_value_from_config("blah", config_key_path, None, sagemaker_session) == "blah"
-    mock_info_logger.assert_called_once_with("Ignoring defaults config.")
+    mock_info_logger.assert_called_once_with(
+        "Ignoring defaults config because `ignore_sagemaker_config` was set in Session settings."
+    )
 
     mock_info_logger.reset_mock()
 
     assert resolve_value_from_config(None, config_key_path, "blah", sagemaker_session) == "blah"
-    mock_info_logger.assert_called_once_with("Ignoring defaults config.")
+    mock_info_logger.assert_called_once_with(
+        "Ignoring defaults config because `ignore_sagemaker_config` was set in Session settings."
+    )
 
 
 @patch("jsonschema.validate")