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

Nodejs express - optimize copy for buildah #680

Merged
merged 12 commits into from
Mar 3, 2020
24 changes: 13 additions & 11 deletions incubator/nodejs-express/image/project/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -7,16 +7,23 @@ RUN apt-get update \
&& apt-get clean \
&& echo 'Finished installing dependencies'

# Copying individual files/folders as buildah 1.9.0 does not honour .dockerignore
COPY package*.json /project/
COPY *.js /project/
COPY user-app /project/user-app
# Removing node_modules as we can not make assumptions about file structure of user-app
RUN rm -rf /project/user-app/node_modules && mkdir -p /project/user-app/node_modules

# Install stack dependencies
WORKDIR /project
COPY ./package*.json ./
RUN npm install --production

# Install user-app dependencies
WORKDIR /project/user-app
COPY ./user-app/package*.json ./
RUN npm install --production
WORKDIR /project/user-app/node_modules

# Creating a tar to work around poor copy performance when using buildah 1.9.0
RUN cd / && tar czf project.tgz project

# Copy the dependencies into a slim Node docker image
FROM node:12-slim
Expand All @@ -27,14 +34,9 @@ RUN apt-get update \
&& apt-get clean \
&& echo 'Finished installing dependencies'

# Copy the project
COPY --chown=node:node . /project
neeraj-laad marked this conversation as resolved.
Show resolved Hide resolved

# Copy all dependencies
COPY --chown=node:node --from=0 /project/node_modules /project/node_modules
COPY --chown=node:node --from=0 /project/user-app/node_modules /project/user-app/node_modules

WORKDIR /project
# Copy project with dependencies
COPY --chown=node:node --from=0 /project.tgz /
RUN tar xf project.tgz && chown -R node:node /project && rm project.tgz

ENV NODE_PATH=/project/user-app/node_modules

Expand Down
2 changes: 1 addition & 1 deletion incubator/nodejs-express/image/project/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "nodejs-express",
"version": "0.4.2",
"version": "0.4.3",
"description": "Node.js Express Stack",
"license": "Apache-2.0",
"repository": {
Expand Down
2 changes: 1 addition & 1 deletion incubator/nodejs-express/stack.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
name: Node.js Express
version: 0.4.2
version: 0.4.3
description: Express web framework for Node.js
license: Apache-2.0
language: nodejs
Expand Down