Address review feedback: restore debug log, remove sensitive parameters from warning logs

Co-authored-by: kingernupur <[email protected]>

Author: Copilot

sdk/ml/azure-ai-ml/azure/ai/ml/_arm_deployments/arm_deployment_executor.py

@@ -120,6 +120,7 @@ def deploy_resource(
             else:
                 return poller
         except Exception as ex:
+            module_logger.debug("Polling hit an exception")
             raise ex
 
         if error is not None:

sdk/ml/azure-ai-ml/azure/ai/ml/_azure_environments.py

@@ -298,13 +298,12 @@ def _get_clouds_by_metadata_url(metadata_url: str) -> Dict[str, Dict[str, str]]:
             return cli_cloud_dict
     except Exception as ex:  # pylint: disable=broad-except
         module_logger.warning(
-            "Error: Azure ML was unable to load cloud metadata from the url specified by %s. "
+            "Error: Azure ML was unable to load cloud metadata from the specified URL. "
             "This may be due to a misconfiguration of networking controls. Azure Machine Learning Python "
             "SDK requires outbound access to Azure Resource Manager. Please contact your networking team "
             "to configure outbound access to Azure Resource Manager on both Network Security Group and "
             "Firewall. For more details on required configurations, see "
-            "https://learn.microsoft.com/azure/machine-learning/how-to-access-azureml-behind-firewall.",
-            metadata_url,
+            "https://learn.microsoft.com/azure/machine-learning/how-to-access-azureml-behind-firewall."
         )
         module_logger.debug("Exception details: %s", ex)
         return {}

sdk/ml/azure-ai-ml/azure/ai/ml/entities/_job/job.py

@@ -334,14 +334,12 @@ def _from_rest_object(  # pylint: disable=too-many-return-statements
             raise ex
         except Exception as ex:
             error_message = json.dumps(obj.as_dict(), indent=2) if obj else None
-            module_logger.info(
-                "Unable to parse the job resource: %s",
-                error_message,
-            )
+            module_logger.info("Unable to parse the job resource")
             module_logger.debug(
-                "Exception: %s.\n%s",
+                "Exception: %s.\n%s\nUnable to parse the job resource: %s.",
                 ex,
                 traceback.format_exc(),
+                error_message,
             )
             raise JobParsingError(
                 message=str(ex),