potiuk commented on a change in pull request #19819:
URL: https://github.com/apache/airflow/pull/19819#discussion_r756924024
##########
File path:
docs/docker-stack/docker-examples/restricted/restricted_environments.sh
##########
@@ -22,28 +22,29 @@ AIRFLOW_SOURCES="$(cd "$(dirname
"${BASH_SOURCE[0]}")/../../../../" && pwd)"
cd "${AIRFLOW_SOURCES}"
# [START download]
+export AIRFLOW_VERSION="2.3.0.dev0"
rm docker-context-files/*.whl docker-context-files/*.tar.gz
docker-context-files/*.txt || true
curl -Lo "docker-context-files/constraints-3.7.txt" \
-
https://raw.githubusercontent.com/apache/airflow/constraints-2.3.0.dev0/constraints-3.7.txt
+
"https://raw.githubusercontent.com/apache/airflow/constraints-${AIRFLOW_VERSION}/constraints-3.7.txt";
# For Airflow pre 2.1 you need to use PIP 20.2.4 to install/download Airflow
packages.
pip install pip==20.2.4
pip download --dest docker-context-files \
--constraint docker-context-files/constraints-3.7.txt \
-
"apache-airflow[async,aws,azure,celery,dask,elasticsearch,gcp,kubernetes,postgres,redis,slack,ssh,statsd,virtualenv]==2.0.2"
+
"apache-airflow[async,aws,azure,celery,dask,elasticsearch,gcp,kubernetes,postgres,redis,slack,ssh,statsd,virtualenv]==${AIRFLOW_VERSION}"
Review comment:
Should be 2.2. in both cases. It was originally 2.0.2 in both cases, but
has been replaced by one of bulk version replacements apparently. (this example
was showing also the PIP version downgrade needed for pre 2.1.0 version - but
we should definitely drop it).
--
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]
