[CONFIG] [Docker] [make] new "production" phase at the end.

Author: Gonzalo Diaz

.github/workflows/docker-image.yml

@@ -19,7 +19,11 @@ jobs:
       - uses: actions/checkout@a5ac7e51b41094c92402da3b24376905380afc29 # v4
       - name: Build the Docker image
         run: make compose/rebuild
-      - name: Run test in Docker image
+      - name: Lint in Docker image
+        run: make compose/lint
+      - name: Test in Docker image
+        run: make compose/test
+      - name: Run in Docker image
         run: make compose/run
       - name: Tag Docker image
         run: >

Dockerfile

@@ -67,14 +67,20 @@ RUN ls -alh
 # CMD []
 WORKDIR ${WORKDIR}
 ###############################################################################
-FROM development AS builder
+FROM base AS builder
 
 ENV WORKDIR=/app
 WORKDIR ${WORKDIR}
 
-RUN make clean && npm ci --verbose --omit-dev
+COPY ./src ${WORKDIR}/src
+COPY ./package.json ${WORKDIR}/package.json
+COPY ./package-lock.json ${WORKDIR}/package-lock.json
+COPY ./Makefile ${WORKDIR}/
 
-# CMD []
+RUN rm -vfr node_modules && npm ci --verbose --omit=dev
+RUN npm run build
+
+CMD ["ls", "-alh"]
 
 ###############################################################################
 ### In testing stage, can't use USER, due permissions issue
@@ -100,21 +106,22 @@ CMD ["make", "test"]
 ## in the production phase, "good practices" such as
 ## WORKSPACE and USER are maintained
 ##
-FROM builder AS production
+FROM base AS production
 
 ENV LOG_LEVEL=info
 ENV BRUTEFORCE=false
 ENV WORKDIR=/app
 WORKDIR ${WORKDIR}
 
-# TODO find a way to exclude /src/ and exclude *.test.js files
+COPY --from=builder /app/dist ${WORKDIR}/dist
+COPY --from=builder /app/node_modules ${WORKDIR}/node_modules
+
+COPY ./Makefile ${WORKDIR}/
+COPY ./package.json ${WORKDIR}/package.json
+COPY ./package-lock.json ${WORKDIR}/package-lock.json
+COPY ./Makefile ${WORKDIR}/
 
-COPY ./.babelrc /app/.babelrc
-COPY ./.eslintrc /app/.eslintrc
-COPY ./.prettierrc /app/.prettierrc
-COPY ./jest.config.js /app/jest.config.js
-COPY --from=builder /app/node_modules /app/node_modules
 RUN ls -alh
 
 USER node
-CMD ["npm", "run", "test"]
+CMD ["ls", "-alh"]

Makefile

@@ -52,6 +52,9 @@ clean:
 	mkdir -p ./coverage
 	touch ./coverage/.gitkeep
 
+build: dependencies
+	${NPM} run build
+
 dependencies:
 	@echo "################################################################################"
 	@echo "## Dependencies: ###############################################################"
@@ -94,10 +97,12 @@ upgrade: update
 compose/build: env
 	docker-compose --profile lint build
 	docker-compose --profile testing build
+	docker-compose --profile production build
 
 compose/rebuild: env
 	docker-compose --profile lint build --no-cache
 	docker-compose --profile testing build --no-cache
+	docker-compose --profile production build --no-cache
 
 compose/lint/markdown: compose/build
 	docker-compose --profile lint build
@@ -114,7 +119,13 @@ compose/test/static: compose/build
 
 compose/lint: compose/lint/markdown compose/lint/yaml compose/test/styling compose/test/static
 
+compose/test: compose/build
+	docker-compose --profile testing run --rm algorithm-exercises-js-test make test
+
 compose/run: compose/build
-	docker-compose --profile testing run --rm algorithm-exercises-js make test
+	docker-compose --profile production run --rm algorithm-exercises-js make run
 
 all: env dependencies test
+
+run:
+	ls -alh

compose.yaml

@@ -1,8 +1,8 @@
 ---
 
 services:
-  algorithm-exercises-js:
-    image: algorithm-exercises-js:latest
+  algorithm-exercises-js-test:
+    image: algorithm-exercises-js:test
     build:
       context: .
       target: testing
@@ -37,13 +37,13 @@ services:
       - ./:/app
     profiles: ["development"]
 
-  algorithm-exercises-js-prod:
-    image: algorithm-exercises-js:prod
+  algorithm-exercises-js:
+    image: algorithm-exercises-js:latest
     build:
       context: .
       target: production
-    volumes:
-      - ./:/app
+    # volumes:
+    #   - ./:/app
     profiles: ["production"]
 
 ## REFERENCES: