From e1f8de80a72fdf88d936a98908f1fd362d6529d3 Mon Sep 17 00:00:00 2001
From: Manuel <5673677+mtrezza@users.noreply.github.com>
Date: Fri, 29 Oct 2021 17:57:37 +0200
Subject: [PATCH] ci: fix docker build with node:lts-alpine (#7663)

---
 Dockerfile | 21 ++++++++++++++++-----
 1 file changed, 16 insertions(+), 5 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index 0749e8d8..a351d090 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -1,20 +1,30 @@
+############################################################
 # Build stage
+############################################################
 FROM node:lts-alpine as build
 
 RUN apk update; \
   apk add git;
 WORKDIR /tmp
+
+# Copy package.json first to benefit from layer caching
 COPY package*.json ./
 
-# Copy local dependencies for CI tests
-COPY spec/dependencies spec/dependencies
-
-RUN npm cache clean --force
-RUN npm ci
+# Copy src to have config files for install
 COPY . .
+
+# Clean npm cache; added to fix an issue with the install process
+RUN npm cache clean --force
+
+# Install all dependencies
+RUN npm ci
+
+# Run build steps
 RUN npm run build
 
+############################################################
 # Release stage
+############################################################
 FROM node:lts-alpine as release
 
 RUN apk update; \
@@ -26,6 +36,7 @@ WORKDIR /parse-server
 
 COPY package*.json ./
 
+# Clean npm cache; added to fix an issue with the install process
 RUN npm cache clean --force
 RUN npm ci --production --ignore-scripts