Crowiant commented on code in PR #45931:
URL: https://github.com/apache/airflow/pull/45931#discussion_r2627247913
##########
airflow-core/src/airflow/configuration.py:
##########
@@ -2301,9 +2288,8 @@ def ensure_secrets_loaded(
"""
# Check if the secrets_backend_list contains only 2 default backends.
- # Check if we are loading the backends for worker too by checking if the
default_backends is equal
- # to DEFAULT_SECRETS_SEARCH_PATH.
- if len(secrets_backend_list) == 2 or default_backends !=
DEFAULT_SECRETS_SEARCH_PATH:
+ # Check if we are loading the backends for worker too by checking if the
default_backends is not None
+ if len(secrets_backend_list) == 2 or default_backends is not None:
return initialize_secrets_backends(default_backends=default_backends)
return secrets_backend_list
Review Comment:
Hello @amoghrajesh ! Thank you for mentioning the PR, I updated the code.
Please check when you have time!
--
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]
