Conversation
dfayd0
added
server
There was a problem hiding this comment.
Globalement tout est good. Attention juste pour le changement de nom de la section dans le README. Et wtf la fin du Dockerfile.
Sinon, j'ai checke un peu des bests practices de la containerization d'app Rust et il y a moyen de substantiellement écourté le temps de build de ton image avec cargo-chef et encore plus avec d'autres: https://depot.dev/blog/rust-dockerfile-best-practices
| && rm -rf /var/lib/apt/lists/* | ||
|
|
||
| COPY Cargo.toml Cargo.lock ./ | ||
| COPY local ./local |
There was a problem hiding this comment.
Je crois pas que tu ais besoin de local dans la compilation.
There was a problem hiding this comment.
pas sur de comprendre ce comment
|
Sinon tu peux rajouter le prefix FROM docker.io/library/rust:1.89-slim-bookworm AS builder
...
FROM docker.io/library/debian:bookworm-slimca mange pas de pain et ca precise que tu tires tes images du Docker Hub, c'est un peu plus explicite. |
2 participants
No description provided.