From d46c97f64a5d6f609f416f313fe3e11f04d87505 Mon Sep 17 00:00:00 2001 From: Davide Bortolami Date: Sun, 8 Mar 2020 16:24:07 +0000 Subject: [PATCH 1/4] Enumerate node packages --- build/frappe-nginx/Dockerfile | 4 ++++ build/frappe-nginx/v11.Dockerfile | 4 ++++ build/frappe-nginx/v12.Dockerfile | 4 ++++ build/frappe-socketio/Dockerfile | 3 +++ build/frappe-socketio/v11.Dockerfile | 3 +++ build/frappe-socketio/v12.Dockerfile | 3 +++ 6 files changed, 21 insertions(+) diff --git a/build/frappe-nginx/Dockerfile b/build/frappe-nginx/Dockerfile index 03b40883..425b3fe0 100644 --- a/build/frappe-nginx/Dockerfile +++ b/build/frappe-nginx/Dockerfile @@ -31,6 +31,10 @@ COPY build/frappe-nginx/docker-entrypoint.sh / RUN apt-get update && apt-get install -y rsync && apt-get clean +RUN node --version +RUN npm --version +RUN yarn --version + VOLUME [ "/assets" ] ENTRYPOINT ["/docker-entrypoint.sh"] diff --git a/build/frappe-nginx/v11.Dockerfile b/build/frappe-nginx/v11.Dockerfile index cfeb2c3d..f6590101 100644 --- a/build/frappe-nginx/v11.Dockerfile +++ b/build/frappe-nginx/v11.Dockerfile @@ -31,6 +31,10 @@ COPY build/frappe-nginx/docker-entrypoint.sh / RUN apt-get update && apt-get install -y rsync && apt-get clean +RUN node --version +RUN npm --version +RUN yarn --version + VOLUME [ "/assets" ] ENTRYPOINT ["/docker-entrypoint.sh"] diff --git a/build/frappe-nginx/v12.Dockerfile b/build/frappe-nginx/v12.Dockerfile index 4994ad28..bb9b0e49 100644 --- a/build/frappe-nginx/v12.Dockerfile +++ b/build/frappe-nginx/v12.Dockerfile @@ -31,6 +31,10 @@ COPY build/frappe-nginx/docker-entrypoint.sh / RUN apt-get update && apt-get install -y rsync && apt-get clean +RUN node --version +RUN npm --version +RUN yarn --version + VOLUME [ "/assets" ] ENTRYPOINT ["/docker-entrypoint.sh"] diff --git a/build/frappe-socketio/Dockerfile b/build/frappe-socketio/Dockerfile index a74b0d3e..fcb6c0fb 100644 --- a/build/frappe-socketio/Dockerfile +++ b/build/frappe-socketio/Dockerfile @@ -30,5 +30,8 @@ RUN ln -s /usr/local/bin/docker-entrypoint.sh / # backwards compat WORKDIR /home/frappe/frappe-bench/sites +RUN node --version +RUN npm --version + ENTRYPOINT ["docker-entrypoint.sh"] CMD ["start"] diff --git a/build/frappe-socketio/v11.Dockerfile b/build/frappe-socketio/v11.Dockerfile index fcaac8ac..49732267 100644 --- a/build/frappe-socketio/v11.Dockerfile +++ b/build/frappe-socketio/v11.Dockerfile @@ -30,5 +30,8 @@ RUN ln -s /usr/local/bin/docker-entrypoint.sh / # backwards compat WORKDIR /home/frappe/frappe-bench/sites +RUN node --version +RUN npm --version + ENTRYPOINT ["docker-entrypoint.sh"] CMD ["start"] diff --git a/build/frappe-socketio/v12.Dockerfile b/build/frappe-socketio/v12.Dockerfile index adf536bd..618712e9 100644 --- a/build/frappe-socketio/v12.Dockerfile +++ b/build/frappe-socketio/v12.Dockerfile @@ -30,5 +30,8 @@ RUN ln -s /usr/local/bin/docker-entrypoint.sh / # backwards compat WORKDIR /home/frappe/frappe-bench/sites +RUN node --version +RUN npm --version + ENTRYPOINT ["docker-entrypoint.sh"] CMD ["start"] From b36513f8fd8badfdd14d6269df1212655941324c Mon Sep 17 00:00:00 2001 From: Davide Bortolami Date: Mon, 9 Mar 2020 13:29:58 +0000 Subject: [PATCH 2/4] fix image build fail --- build/frappe-nginx/Dockerfile | 8 ++++---- build/frappe-nginx/v11.Dockerfile | 8 ++++---- build/frappe-nginx/v12.Dockerfile | 8 ++++---- 3 files changed, 12 insertions(+), 12 deletions(-) diff --git a/build/frappe-nginx/Dockerfile b/build/frappe-nginx/Dockerfile index 425b3fe0..36168b9c 100644 --- a/build/frappe-nginx/Dockerfile +++ b/build/frappe-nginx/Dockerfile @@ -23,6 +23,10 @@ RUN git clone --depth 1 https://github.com/frappe/bench /tmp/bench \ RUN cp -R /home/frappe/frappe-bench/apps/frappe/frappe/public/* /home/frappe/frappe-bench/sites/assets/frappe \ && cp -R /home/frappe/frappe-bench/apps/frappe/node_modules /home/frappe/frappe-bench/sites/assets/frappe/ +RUN node --version +RUN npm --version +RUN yarn --version + FROM nginx:latest COPY --from=0 /home/frappe/frappe-bench/sites /var/www/html/ COPY --from=0 /var/www/error_pages /var/www/ @@ -31,10 +35,6 @@ COPY build/frappe-nginx/docker-entrypoint.sh / RUN apt-get update && apt-get install -y rsync && apt-get clean -RUN node --version -RUN npm --version -RUN yarn --version - VOLUME [ "/assets" ] ENTRYPOINT ["/docker-entrypoint.sh"] diff --git a/build/frappe-nginx/v11.Dockerfile b/build/frappe-nginx/v11.Dockerfile index f6590101..4c97dccc 100644 --- a/build/frappe-nginx/v11.Dockerfile +++ b/build/frappe-nginx/v11.Dockerfile @@ -23,6 +23,10 @@ RUN git clone --depth 1 https://github.com/frappe/bench /tmp/bench \ RUN cp -R /home/frappe/frappe-bench/apps/frappe/frappe/public/* /home/frappe/frappe-bench/sites/assets/frappe \ && cp -R /home/frappe/frappe-bench/apps/frappe/node_modules /home/frappe/frappe-bench/sites/assets/frappe/ +RUN node --version +RUN npm --version +RUN yarn --version + FROM nginx:latest COPY --from=0 /home/frappe/frappe-bench/sites /var/www/html/ COPY --from=0 /var/www/error_pages /var/www/ @@ -31,10 +35,6 @@ COPY build/frappe-nginx/docker-entrypoint.sh / RUN apt-get update && apt-get install -y rsync && apt-get clean -RUN node --version -RUN npm --version -RUN yarn --version - VOLUME [ "/assets" ] ENTRYPOINT ["/docker-entrypoint.sh"] diff --git a/build/frappe-nginx/v12.Dockerfile b/build/frappe-nginx/v12.Dockerfile index bb9b0e49..869b45bc 100644 --- a/build/frappe-nginx/v12.Dockerfile +++ b/build/frappe-nginx/v12.Dockerfile @@ -23,6 +23,10 @@ RUN git clone --depth 1 https://github.com/frappe/bench /tmp/bench \ RUN cp -R /home/frappe/frappe-bench/apps/frappe/frappe/public/* /home/frappe/frappe-bench/sites/assets/frappe \ && cp -R /home/frappe/frappe-bench/apps/frappe/node_modules /home/frappe/frappe-bench/sites/assets/frappe/ +RUN node --version +RUN npm --version +RUN yarn --version + FROM nginx:latest COPY --from=0 /home/frappe/frappe-bench/sites /var/www/html/ COPY --from=0 /var/www/error_pages /var/www/ @@ -31,10 +35,6 @@ COPY build/frappe-nginx/docker-entrypoint.sh / RUN apt-get update && apt-get install -y rsync && apt-get clean -RUN node --version -RUN npm --version -RUN yarn --version - VOLUME [ "/assets" ] ENTRYPOINT ["/docker-entrypoint.sh"] From 104f3e67e23a05388b119a925e01f1cb19daaf53 Mon Sep 17 00:00:00 2001 From: Davide Bortolami Date: Mon, 9 Mar 2020 13:39:28 +0000 Subject: [PATCH 3/4] =?UTF-8?q?reduce=20layers=20=E2=80=A8?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- build/frappe-socketio/Dockerfile | 7 +++---- build/frappe-socketio/v11.Dockerfile | 7 +++---- build/frappe-socketio/v12.Dockerfile | 7 +++---- 3 files changed, 9 insertions(+), 12 deletions(-) diff --git a/build/frappe-socketio/Dockerfile b/build/frappe-socketio/Dockerfile index fcb6c0fb..da9462c1 100644 --- a/build/frappe-socketio/Dockerfile +++ b/build/frappe-socketio/Dockerfile @@ -19,7 +19,9 @@ RUN cd /home/frappe/frappe-bench/apps/frappe \ --output /home/frappe/frappe-bench/apps/frappe/node_utils.js RUN cd /home/frappe/frappe-bench/apps/frappe \ - && npm install --only=production + && npm install --only=production \ + && node --version \ + && npm --version COPY build/frappe-socketio/health.js /home/frappe/frappe-bench/apps/frappe/health.js RUN chown -R frappe:frappe /home/frappe @@ -30,8 +32,5 @@ RUN ln -s /usr/local/bin/docker-entrypoint.sh / # backwards compat WORKDIR /home/frappe/frappe-bench/sites -RUN node --version -RUN npm --version - ENTRYPOINT ["docker-entrypoint.sh"] CMD ["start"] diff --git a/build/frappe-socketio/v11.Dockerfile b/build/frappe-socketio/v11.Dockerfile index 49732267..ed3885fc 100644 --- a/build/frappe-socketio/v11.Dockerfile +++ b/build/frappe-socketio/v11.Dockerfile @@ -19,7 +19,9 @@ RUN cd /home/frappe/frappe-bench/apps/frappe \ --output /home/frappe/frappe-bench/apps/frappe/node_utils.js RUN cd /home/frappe/frappe-bench/apps/frappe \ - && npm install --only=production + && npm install --only=production \ + && node --version \ + && npm --version COPY build/frappe-socketio/health.js /home/frappe/frappe-bench/apps/frappe/health.js RUN chown -R frappe:frappe /home/frappe @@ -30,8 +32,5 @@ RUN ln -s /usr/local/bin/docker-entrypoint.sh / # backwards compat WORKDIR /home/frappe/frappe-bench/sites -RUN node --version -RUN npm --version - ENTRYPOINT ["docker-entrypoint.sh"] CMD ["start"] diff --git a/build/frappe-socketio/v12.Dockerfile b/build/frappe-socketio/v12.Dockerfile index 618712e9..e0cf42b8 100644 --- a/build/frappe-socketio/v12.Dockerfile +++ b/build/frappe-socketio/v12.Dockerfile @@ -19,7 +19,9 @@ RUN cd /home/frappe/frappe-bench/apps/frappe \ --output /home/frappe/frappe-bench/apps/frappe/node_utils.js RUN cd /home/frappe/frappe-bench/apps/frappe \ - && npm install --only=production + && npm install --only=production \ + && node --version \ + && npm --version COPY build/frappe-socketio/health.js /home/frappe/frappe-bench/apps/frappe/health.js RUN chown -R frappe:frappe /home/frappe @@ -30,8 +32,5 @@ RUN ln -s /usr/local/bin/docker-entrypoint.sh / # backwards compat WORKDIR /home/frappe/frappe-bench/sites -RUN node --version -RUN npm --version - ENTRYPOINT ["docker-entrypoint.sh"] CMD ["start"] From 1e9cdacf1a0b1e87fc9d995a118d5b7df01201bd Mon Sep 17 00:00:00 2001 From: Davide Bortolami Date: Mon, 9 Mar 2020 23:37:25 +0000 Subject: [PATCH 4/4] Reduce layers --- build/frappe-nginx/Dockerfile | 9 ++++----- build/frappe-nginx/v11.Dockerfile | 9 ++++----- build/frappe-nginx/v12.Dockerfile | 9 ++++----- 3 files changed, 12 insertions(+), 15 deletions(-) diff --git a/build/frappe-nginx/Dockerfile b/build/frappe-nginx/Dockerfile index 36168b9c..64df31b7 100644 --- a/build/frappe-nginx/Dockerfile +++ b/build/frappe-nginx/Dockerfile @@ -14,7 +14,10 @@ RUN cd /home/frappe/frappe-bench/apps/frappe \ && yarn \ && yarn run production \ && rm -fr node_modules \ - && yarn install --production=true + && yarn install --production=true \ + && node --version \ + && npm --version \ + && yarn --version RUN git clone --depth 1 https://github.com/frappe/bench /tmp/bench \ && mkdir -p /var/www/error_pages \ @@ -23,10 +26,6 @@ RUN git clone --depth 1 https://github.com/frappe/bench /tmp/bench \ RUN cp -R /home/frappe/frappe-bench/apps/frappe/frappe/public/* /home/frappe/frappe-bench/sites/assets/frappe \ && cp -R /home/frappe/frappe-bench/apps/frappe/node_modules /home/frappe/frappe-bench/sites/assets/frappe/ -RUN node --version -RUN npm --version -RUN yarn --version - FROM nginx:latest COPY --from=0 /home/frappe/frappe-bench/sites /var/www/html/ COPY --from=0 /var/www/error_pages /var/www/ diff --git a/build/frappe-nginx/v11.Dockerfile b/build/frappe-nginx/v11.Dockerfile index 4c97dccc..e44e7663 100644 --- a/build/frappe-nginx/v11.Dockerfile +++ b/build/frappe-nginx/v11.Dockerfile @@ -14,7 +14,10 @@ RUN cd /home/frappe/frappe-bench/apps/frappe \ && yarn \ && yarn run production \ && rm -fr node_modules \ - && yarn install --production=true + && yarn install --production=true \ + && node --version \ + && npm --version \ + && yarn --version RUN git clone --depth 1 https://github.com/frappe/bench /tmp/bench \ && mkdir -p /var/www/error_pages \ @@ -23,10 +26,6 @@ RUN git clone --depth 1 https://github.com/frappe/bench /tmp/bench \ RUN cp -R /home/frappe/frappe-bench/apps/frappe/frappe/public/* /home/frappe/frappe-bench/sites/assets/frappe \ && cp -R /home/frappe/frappe-bench/apps/frappe/node_modules /home/frappe/frappe-bench/sites/assets/frappe/ -RUN node --version -RUN npm --version -RUN yarn --version - FROM nginx:latest COPY --from=0 /home/frappe/frappe-bench/sites /var/www/html/ COPY --from=0 /var/www/error_pages /var/www/ diff --git a/build/frappe-nginx/v12.Dockerfile b/build/frappe-nginx/v12.Dockerfile index 869b45bc..29adfead 100644 --- a/build/frappe-nginx/v12.Dockerfile +++ b/build/frappe-nginx/v12.Dockerfile @@ -14,7 +14,10 @@ RUN cd /home/frappe/frappe-bench/apps/frappe \ && yarn \ && yarn run production \ && rm -fr node_modules \ - && yarn install --production=true + && yarn install --production=true \ + && node --version \ + && npm --version \ + && yarn --version RUN git clone --depth 1 https://github.com/frappe/bench /tmp/bench \ && mkdir -p /var/www/error_pages \ @@ -23,10 +26,6 @@ RUN git clone --depth 1 https://github.com/frappe/bench /tmp/bench \ RUN cp -R /home/frappe/frappe-bench/apps/frappe/frappe/public/* /home/frappe/frappe-bench/sites/assets/frappe \ && cp -R /home/frappe/frappe-bench/apps/frappe/node_modules /home/frappe/frappe-bench/sites/assets/frappe/ -RUN node --version -RUN npm --version -RUN yarn --version - FROM nginx:latest COPY --from=0 /home/frappe/frappe-bench/sites /var/www/html/ COPY --from=0 /var/www/error_pages /var/www/