jedcunningham commented on code in PR #47310: URL: https://github.com/apache/airflow/pull/47310#discussion_r1978006825
########## chart/values.yaml: ########## @@ -2541,7 +2541,7 @@ ports: statsdScrape: 9102 pgbouncer: 6543 pgbouncerScrape: 9127 - apiServer: 9091 + apiServer: 8080 Review Comment: I left this one separate from the existing `ports.airflowUI` config - matching it with the service name seemed better imo. ########## dev/breeze/doc/03_developer_tasks.rst: ########## @@ -152,8 +152,7 @@ database client: You can change the used host port numbers by setting appropriate environment variables: * ``SSH_PORT`` -* ``WEBSERVER_HOST_PORT`` - for Airflow 2 web UI when --use-airflow-version is used -* ``API_SERVER_HOST_PORT`` +* ``WEB_HOST_PORT`` - API server for Airflow 3, or webserver port for Airflow 2 when --use-airflow-version is used Review Comment: I chose this as a "least bad" option, so it's not specific to either Airflow version. I think `HTTP_HOST_PORT` would also be a good option. Happy to adjust if folks have strong opinions on this one. -- 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]
