ashb commented on a change in pull request #6538: [AIRFLOW-5886] Selective 
copying of sources in Docker image.
URL: https://github.com/apache/airflow/pull/6538#discussion_r379436725
 
 

 ##########
 File path: Dockerfile
 ##########
 @@ -357,6 +341,29 @@ RUN chmod +x "${HOME}/run-tests-complete"
 
 RUN chmod +x "${HOME}/run-tests"
 
+# Copy selected subdirectories only
+COPY .github/ ${AIRFLOW_SOURCES}/.github/
+COPY dags/ ${AIRFLOW_SOURCES}/dags/
+COPY common/ ${AIRFLOW_SOURCES}/common/
+COPY licenses/ ${AIRFLOW_SOURCES}/licenses/
+COPY scripts/ci/in_container/ ${AIRFLOW_SOURCES}/scripts/ci/in_container/
+COPY docs/ ${AIRFLOW_SOURCES}/docs/
+COPY tests/ ${AIRFLOW_SOURCES}/tests/
+COPY airflow/ ${AIRFLOW_SOURCES}/airflow/
+COPY .coveragerc .rat-excludes .flake8 pylintrc LICENSE MANIFEST.in NOTICE 
CHANGELOG.txt \
+     .github .bash_completion .bash_completion.d run-tests run-tests-complete \
+     setup.cfg setup.py \
 
 Review comment:
   Not that it makes much difference, but is ther a reson we are copying 
setup.py here again?  We already copied it in on L 311 (in this diff, line 
number has changed since.)

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