Merge pull request #95 from bcgov/ci/fluentbit

Fix NPM Cache directory permission and PID Signalling bugs
bcgov · Nov 9, 2023 · 818ca5e · 818ca5e
2 parents 1e23d36 + 99f2433
commit 818ca5e
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -18,7 +18,9 @@ RUN wget https://www2.gov.bc.ca/assets/gov/british-columbians-our-governments/se
     rm -rf ./BcSansFont_Print && \
     fc-cache -f
 
-# NPM Permission Fix (already present in base image)
+# NPM Permission Fix
+RUN mkdir -p /.npm
+RUN chown -R 1001:0 /.npm
 
 # Install Application
 COPY app ${APP_ROOT}
@@ -27,4 +29,4 @@ USER 1001
 RUN npm ci --omit=dev
 
 EXPOSE ${APP_PORT}
-CMD ["npm", "run", "start"]
+CMD ["node", "./bin/www"]
diff --git a/app/Dockerfile b/app/Dockerfile
@@ -18,7 +18,9 @@ RUN wget https://www2.gov.bc.ca/assets/gov/british-columbians-our-governments/se
     rm -rf ./BcSansFont_Print && \
     fc-cache -f
 
-# NPM Permission Fix (already present in base image)
+# NPM Permission Fix
+RUN mkdir -p /.npm
+RUN chown -R 1001:0 /.npm
 
 # Install Application
 COPY . .
@@ -27,4 +29,4 @@ USER 1001
 RUN npm ci --omit=dev
 
 EXPOSE ${APP_PORT}
-CMD ["npm", "run", "start"]
+CMD ["node", "./bin/www"]