Chore/rust dockerfile #80
Conversation
|
👋 Thanks for assigning @tankyleo as a reviewer! |
|
🔔 1st Reminder Hey @tankyleo! This PR has been waiting for your review. |
|
🔔 2nd Reminder Hey @tankyleo! This PR has been waiting for your review. |
|
🔔 3rd Reminder Hey @tankyleo! This PR has been waiting for your review. |
tankyleo
left a comment
tankyleo
left a comment
There was a problem hiding this comment.
Thanks ! can you make this one commit / remove the merge commit in the current PR ?
| # Copy workspace files | ||
| COPY Cargo.toml Cargo.lock ./ | ||
| COPY rustfmt.toml ./ | ||
|
|
||
| # Copy all workspace members | ||
| COPY server ./server | ||
| COPY api ./api | ||
| COPY impls ./impls | ||
| COPY auth-impls ./auth-impls |
There was a problem hiding this comment.
Not familiar with Docker, looked elsewhere sounds like we could replace all these lines with just COPY . . ?
| ca-certificates \ | ||
| libssl3 \ |
There was a problem hiding this comment.
Wasn't aware of these dependencies are you sure these are needed ?
There was a problem hiding this comment.
Likely yes (cf. #77), though IIRC I had to also install libssl-dev and pkg-config, too (cf. https://docs.rs/openssl/latest/openssl/#automatic).
| COPY --from=builder /build/target/release/vss-server /app/vss-server | ||
|
|
||
| # Copy default configuration file | ||
| #COPY server/vss-server-config.toml /app/vss-server-config.toml |
There was a problem hiding this comment.
For now this step is mandatory, so let's not comment it out.
| #ENV VSS_POSTGRESQL_HOST=postgres | ||
| #ENV VSS_POSTGRESQL_PORT=5432 | ||
| #ENV VSS_POSTGRESQL_DATABASE=postgres |
There was a problem hiding this comment.
These env vars don't exist yet, will be adding all of them in #73, would you mind holding on until that PR gets merged ? Should be soon
4 participants
fixes #74