You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/04/09 17:29:15 UTC

[GitHub] [airflow] potiuk commented on a change in pull request #8228: Expose Airflow Webserver Port in Production Docker Image

potiuk commented on a change in pull request #8228: Expose Airflow Webserver Port in Production Docker Image
URL: https://github.com/apache/airflow/pull/8228#discussion_r406362501
 
 

 ##########
 File path: scripts/ci/_utils.sh
 ##########
 @@ -1419,6 +1419,7 @@ function prepare_prod_build() {
                 "--build-arg" "CONSTRAINT_REQUIREMENTS=https://raw.githubusercontent.com/apache/airflow/v1-10-test/requirements/requirements-python${PYTHON_MAJOR_MINOR_VERSION}.txt"
                 # TODO We should uncomment it just before we merge this change to master/v1_10_test
 
 Review comment:
   This is already corrected in #8234. Also we should adapt it slightly after we release 1.10.10 (for releases < 1.10.10 it should be pointing to 1.10.10 and for >= 1.10.10 it should point to ${AIRFLOW_VERSION}

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services