kaxil edited a comment on pull request #14099:
URL: https://github.com/apache/airflow/pull/14099#issuecomment-774151382


   > Actually we should revert it it was as intended. AIRFLOW_INSTALL_VERSION 
was supposed to be the "==2.0.0rc2" in this case (
   > 
   > This is on purpose done this way, so that you could for example specify 
"<3.0.0" if you want to use it in your CI pipeline to build lastest 2.* image.
   > 
   > See the docs: 
https://github.com/apache/airflow/blob/master/docs/apache-airflow/production-deployment.rst
 and plenty of examples.
   > 
   > AIRFLOW_INSTALL_VERSION |   | Optional - might be used for package 
installation of different Airflow version for example"==2.0.0". For 
consistency, you should also set`AIRFLOW_VERSION` to the same value 
AIRFLOW_VERSION is embedded as label in the image created.
   
   In that case, we need to still fix it somewhere else coz the dockerbuild 
step here: 
https://github.com/apache/airflow/blob/master/dev/README_RELEASE_AIRFLOW.md#optional---manually-prepare-production-docker-image
 fails with "pip install failure" coz it used `pip install 
apache-airflow[amazon,google]2.0.0rc2
   ` @potiuk 


----------------------------------------------------------------
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]


Reply via email to