ashb commented on code in PR #57212:
URL: https://github.com/apache/airflow/pull/57212#discussion_r2461995482
##########
airflow-core/src/airflow/settings.py:
##########
@@ -700,17 +700,18 @@ def initialize():
configure_adapters()
# The webservers import this file from models.py with the default settings.
- if not os.environ.get("PYTHON_OPERATORS_VIRTUAL_ENV_MODE", None):
- is_worker = os.environ.get("_AIRFLOW__REEXECUTED_PROCESS") == "1"
- if not is_worker:
- configure_orm()
- configure_action_logging()
-
# Configure secrets masker before masking secrets
_configure_secrets_masker()
- # mask the sensitive_config_values
- conf.mask_secrets()
+ is_worker = os.environ.get("_AIRFLOW__REEXECUTED_PROCESS") == "1"
+ if not os.environ.get("PYTHON_OPERATORS_VIRTUAL_ENV_MODE", None) and not
is_worker:
+ configure_orm()
+
+ # mask the sensitive_config_values
+ conf.mask_secrets()
+ configure_action_logging()
Review Comment:
This is re-ordered, so that when this is used in the virtualenv script, we
don't log things before the logging is configured (resulting in "odd" output)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
