From 2ad8c4919e069cb5048e0284d737cd7e15803a34 Mon Sep 17 00:00:00 2001 From: Aakcht Date: Fri, 10 Dec 2021 00:33:31 +0300 Subject: [PATCH] Log provider import errors as debug warnings (#20172) (cherry picked from commit 82102e656b10b12d7b93af2dfc9dd1de2a4ce708) --- airflow/providers_manager.py | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/airflow/providers_manager.py b/airflow/providers_manager.py index 02a79e7f8025a..796fc70af6f05 100644 --- a/airflow/providers_manager.py +++ b/airflow/providers_manager.py @@ -148,6 +148,16 @@ def _sanity_check(provider_package: str, class_name: str) -> bool: return False try: import_string(class_name) + except ImportError as e: + # When there is an ImportError we turn it into debug warnings as this is + # an expected case when only some providers are installed + log.debug( + "Exception when importing '%s' from '%s' package: %s", + class_name, + provider_package, + e, + ) + return False except Exception as e: log.warning( "Exception when importing '%s' from '%s' package: %s", @@ -649,16 +659,6 @@ def _import_hook( field_behaviours = hook_class.get_ui_field_behaviour() if field_behaviours: self._add_customized_fields(package_name, hook_class, field_behaviours) - except ImportError as e: - # When there is an ImportError we turn it into debug warnings as this is - # an expected case when only some providers are installed - log.debug( - "Exception when importing '%s' from '%s' package: %s", - hook_class_name, - package_name, - e, - ) - return None except Exception as e: log.warning( "Exception when importing '%s' from '%s' package: %s",