potiuk commented on a change in pull request #11099:
URL: https://github.com/apache/airflow/pull/11099#discussion_r493454843
##########
File path: breeze
##########
@@ -2962,6 +2992,8 @@ breeze::setup_default_breeze_constants
initialization::initialize_common_environment
+initialization::intialize_airflow_variables
Review comment:
One thing here though. You will have to move this to inside of
```
initialization::initialize_common_environment
```
This is an "umbrella initialization function that performs all the
initialization together. all the other initializations are separated out for
clarity, but they are actually called from `initialize_common_environment`
The CI scripts also run the `initialize_common_environment` and since
the`intialize_airflow_variables` is not called there, you get the CI errors.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
