You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/07/23 17:06:16 UTC

[GitHub] [airflow] baryluk commented on a change in pull request #17195: Be verbose about failure to import airflow_local_settings

baryluk commented on a change in pull request #17195:
URL: https://github.com/apache/airflow/pull/17195#discussion_r675711455



##########
File path: airflow/settings.py
##########
@@ -432,7 +432,12 @@ def import_local_settings():
 
         log.info("Loaded airflow_local_settings from %s .", airflow_local_settings.__file__)
     except ImportError:
-        log.debug("Failed to import airflow_local_settings.", exc_info=True)
+        log.warning("Failed to import airflow_local_settings.", exc_info=True)
+    except ModuleNotFoundError as e:

Review comment:
       Done. Also made it critical and re-raise the exception.
   
   This is no different that other possible errors, like `SyntaxError`.
   




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org