Skip to content

Commit 1febb2d

Browse files
committed
Split the installation steps
This should let the base build get cached.
1 parent c49e96f commit 1febb2d

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

canarytokens/Dockerfile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ WORKDIR /
1111
COPY requirements.txt ./
1212
# removing PIP deprecation banners
1313
ENV PIP_DISABLE_PIP_VERSION_CHECK=1
14-
# concatenating RUN commands into a single image layer
14+
# Keeping base and code installs on separate lines to try cache in Github Actions
1515
# installing PIP requirements from requirements.txt file
1616
RUN apt-get update && apt-get install -y --no-install-recommends ca-certificates \
1717
wget python-pip unzip osslsigncode gcc python-dev libreadline-dev \
@@ -21,8 +21,8 @@ RUN apt-get update && apt-get install -y --no-install-recommends ca-certificates
2121
pip install -U setuptools && \
2222
pip install --upgrade pip && \
2323
pip install --no-cache-dir -r requirements.txt && \
24-
pip install --no-cache-dir Twisted==19.10.0 && \
25-
wget -O master.zip https://github.com/thinkst/canarytokens/archive/refs/heads/master.zip && \
24+
pip install --no-cache-dir Twisted==19.10.0
25+
RUN wget -O master.zip https://github.com/thinkst/canarytokens/archive/refs/heads/master.zip && \
2626
unzip master.zip && \
2727
mv /canarytokens-master/* /srv && \
2828
rm -rf /canarytokens-master

0 commit comments

Comments
 (0)