ffinfo commented on a change in pull request #4483: [AIRFLOW-3673] Add official 
dockerfile
URL: https://github.com/apache/airflow/pull/4483#discussion_r247308769
 
 

 ##########
 File path: Dockerfile
 ##########
 @@ -0,0 +1,25 @@
+FROM python:3.6-slim
+COPY setup.* /opt/airflow/
+COPY airflow /opt/airflow/airflow
+
+ARG AIRFLOW_HOME=/usr/local/airflow
+ARG AIRFLOW_DEPS="all"
+ARG PYTHON_DEPS=""
+ARG buildDeps="freetds-dev libkrb5-dev libsasl2-dev libssl-dev libffi-dev 
libpq-dev git"
+ARG APT_DEPS="$buildDeps libsasl2-dev freetds-bin build-essential 
default-libmysqlclient-dev apt-utils curl rsync netcat locales"
 
 Review comment:
   I did bring it already down to 1 Gb now, before it was 1,6 Gb because of the 
non slim python image.
   I dit this on purpose because I like that the default image does support 
everything. But I did make it with ARG so that it would be possible to generate 
multiple images from the same docker file in case a smaller version is also 
required

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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