Merge pull request #250 from python-discord/ldap-deps-group

Ldap deps group

Dockerfile

@@ -6,17 +6,22 @@ FROM --platform=linux/amd64 ghcr.io/owl-corp/python-poetry-base:$python_version
 RUN apt-get update && apt-get install --no-install-recommends -y libldap2-dev libsasl2-dev gcc && apt autoclean && rm -rf /var/lib/apt/lists/*
 
 COPY pyproject.toml poetry.lock ./
-RUN poetry install --without dev --no-root
+
+# Only the LDAP group has deps that requires gcc, so only build that for now
+RUN poetry install --only ldap --no-root
 
 FROM --platform=linux/amd64 ghcr.io/owl-corp/python-poetry-base:$python_version
 
 RUN apt-get update && apt-get install --no-install-recommends -y libmagickwand-dev && rm -rf /var/lib/apt/lists/*
 
-# Install project dependencies
 WORKDIR /app
+
+# Copy the pre-built LDAP group from the build stage that required gcc
 COPY --from=wheel-builder /opt/poetry/cache /opt/poetry/cache
+
+# Install the rest of the dependencies
 COPY pyproject.toml poetry.lock ./
-RUN poetry install --without dev --no-root
+RUN poetry install --without dev --with ldap --no-root
 
 # Set Git SHA environment variable for Sentry
 ARG git_sha="development"

pyproject.toml

@@ -30,6 +30,8 @@ pre-commit = "3.7.1"
 ruff = "0.5.4"
 taskipy = "1.13.0"
 
+[tool.poetry.group.ldap]
+optional = true
 [tool.poetry.group.ldap.dependencies]
 bonsai = "^1.5.3"