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:
[email protected]


With regards,
Apache Git Services

Reply via email to