21367: Prefer COPY to ADD
authorBrett Smith <brett.smith@curii.com>
Wed, 10 Jan 2024 22:02:05 +0000 (17:02 -0500)
committerBrett Smith <brett.smith@curii.com>
Wed, 10 Jan 2024 22:02:05 +0000 (17:02 -0500)
This avoids ADD's auto-extract behavior and helps keep the image a
little cleaner.

Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>

sdk/dev-jobs.dockerfile

index dfc839fe00bd3f115f247ec2efaa464dbb4e0ad5..0169b947066df7a8e37f2224c5c3e5f5548621f3 100644 (file)
@@ -16,10 +16,13 @@ MAINTAINER Arvados Package Maintainers <packaging@arvados.org>
 RUN DEBIAN_FRONTEND=noninteractive apt-get update -q && apt-get install -qy --no-install-recommends \
     git python3-dev python3-venv libcurl4-gnutls-dev libgnutls28-dev nodejs build-essential
 
-ADD * /usr/local/src/
 RUN python3 -m venv /opt/arvados-py
 ENV PATH=/opt/arvados-py/bin:/usr/local/bin:/usr/bin:/bin
 RUN python3 -m pip install --no-cache-dir setuptools wheel
+
+# The build script sets up our build context with all the Python source
+# packages to install.
+COPY . /usr/local/src/
 # Run a-c-r afterward to check for a successful install.
 RUN python3 -m pip install --no-cache-dir /usr/local/src/* && arvados-cwl-runner --version