Skip to content

Add cache mounts to docker files #1452

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
arkid15r merged 13 commits into from
May 2, 2025
Merged

Add cache mounts to docker files #1452

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
2f892e6
Add cache mounts to docker files
ahmedxgouda May 1, 2025
74a03e6
Merge branch 'main' into feature/introduce-docker-cache-mounts
ahmedxgouda May 1, 2025
a70ee42
Fix docker e2e and unit test syntax
ahmedxgouda May 1, 2025
0b0871b
Apply coderabbit suggestion
ahmedxgouda May 1, 2025
a0813d4
Fix spelling in backend
ahmedxgouda May 1, 2025
4c3d763
Refactor Dockerfiles to use consistent cache mount for pnpm
ahmedxgouda May 1, 2025
d0c742f
Fix typo in cache mount path in docs Dockerfile.local
ahmedxgouda May 1, 2025
5023cc9
Fix typo in cache mount path in backend Dockerfile
ahmedxgouda May 1, 2025
eee5fc6
Add cache mount to cspell
ahmedxgouda May 1, 2025
c541322
Fix poetry cache
ahmedxgouda May 2, 2025
3be4c59
Add PNPM Home environement variable
ahmedxgouda May 2, 2025
bd1bcde
Merge branch 'main' into pr/ahmedxgouda/1452
arkid15r May 2, 2025
9d46437
Update code
arkid15r May 2, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions backend/docker/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ FROM python:3.13.3-alpine AS builder
RUN apk update && \
addgroup -S owasp && \
adduser -S -h /home/owasp -G owasp owasp && \
python -m pip install --no-cache-dir poetry
python -m pip install poetry

ENV POETRY_VIRTUALENVS_IN_PROJECT=true \
PYTHONUNBUFFERED=1
Expand All @@ -13,7 +13,8 @@ WORKDIR /home/owasp
USER owasp

COPY --chmod=444 --chown=owasp:owasp poetry.lock pyproject.toml ./
RUN poetry install --no-root --without dev --without test
RUN --mount=type=cache,target=/root/.cache/pypotery \
poetry install --no-root --without dev --without test
Comment thread
coderabbitai[bot] marked this conversation as resolved.
Outdated

COPY apps apps
COPY docker/entrypoint.sh entrypoint.sh
Expand All @@ -33,6 +34,7 @@ ENV PATH="/home/owasp/.venv/bin:$PATH" \

WORKDIR /home/owasp

COPY --from=builder --chmod=755 --chown=owasp:owasp /home/owasp/.venv /home/owasp/.venv
COPY --from=builder --chmod=555 --chown=owasp:owasp /home/owasp /home/owasp
RUN chmod +x /home/owasp/entrypoint.sh

Expand Down
8 changes: 5 additions & 3 deletions backend/docker/Dockerfile.local
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ SHELL ["/bin/sh", "-o", "pipefail", "-c"]
RUN apk update && apk upgrade && \
addgroup -S owasp && \
adduser -S -h /home/owasp -G owasp owasp && \
python -m pip install --no-cache-dir poetry
python -m pip install poetry

ENV POETRY_VIRTUALENVS_IN_PROJECT=true \
PYTHONUNBUFFERED=1
Expand All @@ -13,7 +13,8 @@ USER owasp
WORKDIR /home/owasp

COPY --chmod=444 --chown=owasp:owasp poetry.lock pyproject.toml ./
RUN poetry install --no-root --without dev --without test
RUN --mount=type=cache,target=/root/.cache/pypotery \
poetry install --no-root --without dev --without test
Comment thread
coderabbitai[bot] marked this conversation as resolved.
Outdated

FROM python:3.13.3-alpine

Expand All @@ -23,7 +24,7 @@ RUN apk update && \
apk add postgresql-client redis && \
addgroup -S owasp && \
adduser -S -h /home/owasp -G owasp owasp && \
python -m pip install --no-cache-dir poetry
python -m pip install poetry

ENV PATH="/home/owasp/.venv/bin:$PATH" \
PYTHONUNBUFFERED=1
Expand All @@ -33,4 +34,5 @@ EXPOSE 8000
USER owasp
WORKDIR /home/owasp

COPY --from=builder --chmod=755 --chown=owasp:owasp /home/owasp/.venv /home/owasp/.venv
COPY --from=builder --chmod=755 --chown=owasp:owasp /home/owasp /home/owasp
6 changes: 4 additions & 2 deletions backend/docker/Dockerfile.test
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ RUN addgroup -S owasp && \
adduser -S -h /home/owasp -G owasp owasp && \
mkdir -p /home/owasp && \
chown owasp:owasp /home/owasp && \
python -m pip install --no-cache-dir poetry
python -m pip install poetry

ENV FORCE_COLOR=1 \
POETRY_VIRTUALENVS_IN_PROJECT=true \
Expand All @@ -14,7 +14,8 @@ WORKDIR /home/owasp
USER owasp

COPY --chmod=444 poetry.lock pyproject.toml ./
RUN poetry install --no-root
RUN --mount=type=cache,target=/root/.cache/pypotery \
poetry install --no-root
Comment thread
coderabbitai[bot] marked this conversation as resolved.
Outdated

COPY .env.example .env.example
COPY apps apps
Expand All @@ -38,4 +39,5 @@ ENV FORCE_COLOR=1 \
WORKDIR /home/owasp
USER owasp

COPY --from=builder --chmod=755 --chown=owasp:owasp /home/owasp/.venv /home/owasp/.venv
COPY --from=builder --chmod=555 --chown=owasp:owasp /home/owasp /home/owasp
6 changes: 4 additions & 2 deletions docs/docker/Dockerfile.local
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ RUN addgroup -S owasp && \
adduser -S -h /home/owasp -G owasp owasp && \
mkdir -p /home/owasp && \
chown owasp:owasp /home/owasp && \
python -m pip install --no-cache-dir poetry
python -m pip install poetry

ENV FORCE_COLOR=1 \
POETRY_VIRTUALENVS_IN_PROJECT=true \
Expand All @@ -16,7 +16,8 @@ WORKDIR /home/owasp
USER owasp

COPY --chmod=444 --chown=owasp:owasp docs/poetry.lock docs/pyproject.toml mkdocs.yaml ./
RUN poetry install --no-root && \
RUN --mount=type=cache,target=/root/.cache/pypotery \
poetry install --no-root && \
Comment thread
coderabbitai[bot] marked this conversation as resolved.
Outdated
rm -rf docs/poetry.lock docs/pyproject.toml

FROM python:3.13.3-alpine
Expand All @@ -36,4 +37,5 @@ EXPOSE 8001
USER owasp
WORKDIR /home/owasp

COPY --from=builder --chmod=755 --chown=owasp:owasp /home/owasp/.venv /home/owasp/.venv
COPY --from=builder --chmod=555 --chown=owasp:owasp /home/owasp /home/owasp
4 changes: 3 additions & 1 deletion frontend/docker/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,9 @@ WORKDIR /app

RUN npm install --ignore-scripts -g pnpm
COPY --chmod=444 package.json pnpm-lock.yaml ./
RUN pnpm install --frozen-lockfile --ignore-scripts
RUN --mount=type=cache,target=/root/.pnpm-store \
--mount=type=cache,target=/home/owasp/node_modules \
pnpm install --frozen-lockfile --ignore-scripts
Comment thread
ahmedxgouda marked this conversation as resolved.
Outdated

COPY --chmod=444 .env .pnpmrc next.config.ts postcss.config.js tailwind.config.js tsconfig.json ./
COPY --chmod=555 public public
Expand Down
2 changes: 2 additions & 0 deletions frontend/docker/Dockerfile.e2e.test
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@ WORKDIR /app

COPY --chmod=444 package.json pnpm-lock.yaml ./
RUN npm install --ignore-scripts -g pnpm && \
--mount=type=cache,target=/root/.pnpm-store && \
--mount=type=cache,target=/app/node_modules && \
pnpm install --frozen-lockfile --ignore-scripts
Comment thread
coderabbitai[bot] marked this conversation as resolved.
Outdated

COPY __tests__/e2e __tests__/e2e
Expand Down
6 changes: 3 additions & 3 deletions frontend/docker/Dockerfile.local
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,9 @@ RUN apk update && \
WORKDIR /home/owasp

COPY --chmod=444 --chown=node:node package.json pnpm-lock.yaml ./
RUN pnpm install --frozen-lockfile --ignore-scripts
RUN --mount=type=cache,target=/root/.pnpm-store \
--mount=type=cache,target=/home/owasp/node_modules \
pnpm install --frozen-lockfile --ignore-scripts

FROM node:22-alpine

Expand All @@ -21,8 +23,6 @@ RUN apk update && \
chown -R node:node /home/owasp && \
npm install --ignore-scripts -g pnpm

COPY --from=builder --chmod=755 --chown=node:node /home/owasp/node_modules /home/owasp/node_modules

EXPOSE 3000

USER node
Expand Down
2 changes: 2 additions & 0 deletions frontend/docker/Dockerfile.unit.test
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@ WORKDIR /app

COPY --chmod=444 package.json pnpm-lock.yaml ./
RUN npm install --ignore-scripts -g pnpm && \
--mount=type=cache,target=/root/.pnpm-store && \
--mount=type=cache,target=/app/node_modules && \
pnpm install --frozen-lockfile --ignore-scripts && \
chown node:node /app
Comment thread
coderabbitai[bot] marked this conversation as resolved.
Outdated

Expand Down
Loading