jedcunningham commented on code in PR #32835:
URL: https://github.com/apache/airflow/pull/32835#discussion_r1274173817
##########
airflow/configuration.py:
##########
@@ -2011,6 +2011,13 @@ def
load_standard_airflow_configuration(airflow_config_parser: AirflowConfigPars
# there
AIRFLOW_HOME = airflow_config_parser.get("core", "airflow_home")
# type: ignore[assignment]
warnings.warn(msg, category=DeprecationWarning)
+ # Set the WEBSERVER_CONFIG variable
+ set_webserver_config(airflow_config_parser)
Review Comment:
Let's just do it here, not sure we really need another function for it.
We should also stop setting it in `write_webserver_configuration_if_needed`
as well, and just have that do the writing.
I think we can also undo the test change in
https://github.com/apache/airflow/commit/6362ba5ab45a38008814616df4e17717cc3726c3
then as well?
--
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]
