Merge pull request #73 from geoadmin/bug-pb-887-docker-build-failure

PB 887 Fix dockerfile to use pipenv sync instead of install

Dockerfile

@@ -6,9 +6,8 @@ RUN groupadd -r geoadmin && useradd -u 1000 -r -s /bin/false -g geoadmin geoadmi
 RUN pip3 install pipenv \
     && pipenv --version
 
-COPY Pipfile* /tmp/
-RUN cd /tmp && \
-    pipenv install --system --deploy --ignore-pipfile
+COPY Pipfile.lock /tmp/
+RUN cd /tmp && pipenv sync
 
 WORKDIR /app
 COPY --chown=geoadmin:geoadmin ./ /app/

README.md

@@ -44,7 +44,7 @@ See also [Git Flow - Versioning](https://github.com/geoadmin/doc-guidelines/blob
 
 ### Make Dependencies
 
-The **Make** targets assume you have **python3.11**, **pipenv**, **bash**, **curl**, **tar**, **docker** and **docker-compose** installed.
+The **Make** targets assume you have **python3.11**, **pipenv**, **bash**, **curl**, **tar**, **docker** and **docker-compose-plugin** installed.
 
 ### Setting up to work
 
@@ -75,7 +75,7 @@ The other services that are used (DynamoDB local and [MinIO](https://www.min.io)
 Starting DynamoDB local and MinIO is done with a simple
 
 ```bash
-docker-compose up
+docker compose up
 ```
 
 in the source root folder. Make sure to run `make dev` before to ensure the necessary folders `.volumes/*` are in place. These folders are mounted in the services and allow data persistency over restarts of the containers.

docker-compose.yml

@@ -1,4 +1,3 @@
-version: "3.4"
 services:
   s3:
     image: minio/minio