Skip to content
This repository has been archived by the owner on Oct 16, 2023. It is now read-only.

Commit

Permalink
Don't empty the /tmp directory on docker image build
Browse files Browse the repository at this point in the history
We should not empty the `/tmp` directory when the docker image is built. The base image `mojdigital/wordpress-base` includes a file called `/tmp/keeptmp` which is used to mitigate a bug whereby `yas3fs` self-destructs by deleting the entire `/tmp` directory. This is a known issue which has been reported to the developers of `yas3fs`: danilop/yas3fs#150

The bug here is that we were emptying the `/tmp` directory at build time, thus disabling the `yas3fs` bug workaround.
  • Loading branch information
Ollie Treend authored and jsmcgd committed Jul 9, 2018
1 parent d7e3c82 commit d7a9890
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ FROM mojdigital/wordpress-base:latest
RUN curl -sL https://deb.nodesource.com/setup_8.x | bash - && \
DEBIAN_FRONTEND=noninteractive apt-get install -y nodejs && \
apt-get clean && \
rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/* /init
rm -rf /var/lib/apt/lists/* /var/tmp/* /init

ADD . /bedrock

Expand Down

0 comments on commit d7a9890

Please sign in to comment.