ashb commented on a change in pull request #4543: [AIRFLOW-3718] [WIP] 
Multi-layered version of the docker image
URL: https://github.com/apache/airflow/pull/4543#discussion_r262449409
 
 

 ##########
 File path: scripts/ci/kubernetes/docker/build.sh
 ##########
 @@ -34,7 +34,7 @@ fi
 echo "Airflow directory $AIRFLOW_ROOT"
 echo "Airflow Docker directory $DIRNAME"
 
-if [[ ${PYTHON_VERSION} == '3' ]]; then
+if [[ ${PYTHON_VERSION} == 3* ]]; then
 
 Review comment:
   One interesting thing on 3.5 vs 3.6 that I noticed - It's probably worth 
testing against 3.5.2 explicitly as that is the version of Python3 on Ubuntu 
16.04 (and maybe 18.04 too?) - one of our downstream deps had an issue with 
3.5.2, so testing that explicitly makes sense as it's a common deployment 
target.

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


With regards,
Apache Git Services

Reply via email to