[GitHub] [airflow] boredland commented on pull request #20429: fix(standalone): remove hardcoded webserver-port

[GitBox <gi...@apache.org>]

boredland commented on pull request #20429:
URL: https://github.com/apache/airflow/pull/20429#issuecomment-998186862


   > add to docs how to override port
   
   as we spoke about what I as a user would have expected: I found the configuration variables page and expected them to "just work". So from my perspective a hint to that page is even more than enough. Let me know if you'd like to here more.
   
   For people stumbling upon this as long as it isn't merged/released and like me are using standalone in a docker context, you can extend from the Docker like to to apply this tiny patch:
   
   ```Dockerfile
   FROM apache/airflow:2.2.2-python3.9
   
   ## All airflow.cfg options can be set via environment variables
   ## See: https://airflow.apache.org/docs/apache-airflow/stable/configurations-ref.html
   ENV AIRFLOW__WEBSERVER__WEB_SERVER_PORT=${PORT:-5000}
   
   COPY app.json ./
   
   # TODO: remove when https://github.com/apache/airflow/pull/20429 is merged and released
   RUN sed -i -e 's/, "--port", "8080"//g' /home/airflow/.local/lib/python3.9/site-packages/airflow/cli/commands/standalone_command.py
   
   CMD ["airflow", "standalone"]
   ```


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org