-
Notifications
You must be signed in to change notification settings - Fork 3k
Pylint 3.3.6 updates [azure-ai-ml] #41614
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pylint 3.3.6 updates [azure-ai-ml] #41614
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates logging in several modules to improve clarity and consistency when handling exceptions with pylint 3.3.6.
- Standardizes exception logging by separating concise user‐facing messages and detailed debug logs.
- Removes exception interpolation from warning/error logs in favor of dedicated debug logging.
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
Show a summary per file
File | Description |
---|---|
sdk/ml/azure-ai-ml/azure/ai/ml/operations/_workspace_operations_base.py | Updates logging for duration errors by splitting warning and debug logs. |
sdk/ml/azure-ai-ml/azure/ai/ml/operations/_component_operations.py | Splits exception log messages into info/error and debug logs for more consistent logging. |
sdk/ml/azure-ai-ml/azure/ai/ml/entities/_job/job.py | Changes exception logging level from info to debug and adds an info log for failure. |
sdk/ml/azure-ai-ml/azure/ai/ml/_utils/_artifact_utils.py | Updates artifact tool path logging to separate warning and debug details. |
sdk/ml/azure-ai-ml/azure/ai/ml/_azure_environments.py | Refines logging messages by removing exception details from warnings and adding debug logs. |
sdk/ml/azure-ai-ml/azure/ai/ml/_arm_deployments/arm_deployment_executor.py | Updates polling exception logging to display exception type only in the debug log. |
Comments suppressed due to low confidence (2)
sdk/ml/azure-ai-ml/azure/ai/ml/entities/_job/job.py:337
- [nitpick] Check if lowering the logging level for exceptions might obscure error context during parsing issues; ensure that the operational logging configuration captures sufficient details.
module_logger.debug("Exception: %s.\n%s\nUnable to parse the job resource: %s.\n",
sdk/ml/azure-ai-ml/azure/ai/ml/_azure_environments.py:301
- [nitpick] Verify that removing the exception details from the warning log does not hinder troubleshooting connectivity or configuration issues in certain environments.
module_logger.warning("Error: Azure ML was unable to load cloud metadata from the metadata url.")
Description
Compatible with pylint 3.3.6

If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines