Skip to content

Switch runtime image from Alpine to Debian slim#20

Merged
Kaiohz merged 1 commit into
mainfrom
BRIC-18/multiplexed-content-stream
May 6, 2026
Merged

Switch runtime image from Alpine to Debian slim#20
Kaiohz merged 1 commit into
mainfrom
BRIC-18/multiplexed-content-stream

Conversation

@Kaiohz
Copy link
Copy Markdown
Contributor

@Kaiohz Kaiohz commented May 6, 2026

Avoid libc incompatibility issues with compiled wheels by using debian:bookworm-slim instead of Alpine, matching the builder stage base image. Also update package manager commands and user creation syntax for the Debian-based runtime.

Avoid libc incompatibility issues with compiled wheels by using
debian:bookworm-slim instead of Alpine, matching the builder stage
base image. Also update package manager commands and user creation
syntax for the Debian-based runtime.
@Kaiohz Kaiohz merged commit ea8bdcf into main May 6, 2026
1 check passed
@Kaiohz
Copy link
Copy Markdown
Contributor Author

Kaiohz commented May 6, 2026

📊 Code Review

Score: 8/10

✅ Points positifs

  • Raison solide : Le switch de Alpine (musl) vers Debian slim (glibc) résout les problèmes de compatibilité avec les compiled wheels Python. C'est une bonne décision technique.

  • Cohérence : Le runtime utilise maintenant la même base que le builder (python:3.11-slim-bookworm), ce qui garantit que les wheels compilées dans le builder fonctionneront dans le runtime.

  • Sécurité : Non-root user toujours présent, upgrade des packages système, cleanup des apt lists.

  • Minimal : slim-bookworm reste léger (~120MB vs ~50MB Alpine, mais acceptable pour la compatibilité glibc).

💡 Suggestions mineures

  1. Optimisation apt : Ajouter --no-install-recommends pour minimiser la taille :

    RUN apt-get update && apt-get upgrade -y --no-install-recommends && rm -rf /var/lib/apt/lists/*
  2. Sécurité : Considérer apt-get dist-upgrade au lieu de upgrade pour les mises à jour de kernel et packages critiques.

Verdict

PR prête à merger. Les suggestions sont mineures et peuvent être adressées dans une PR suivante si nécessaire.

🎯 Approved - Le changement est justifié et bien implémenté.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant