Skip to content

Commit 9669983

Browse files
authored
Adapt container for new optional requirements (#420)
## Summary <!-- Include a short paragraph of the changes introduced in this PR. If this PR requires additional context or rationale, explain why the changes are necessary. --> Install all extras in the container and add `ffmpeg`. --- - [x] "I certify that all code in this PR is my own, except as noted below." ## Use of AI - [ ] Includes AI-assisted code completion - [ ] Includes code generated by an AI application - [ ] Includes AI-generated tests (NOTE: AI written tests should have a docstring that includes `## WRITTEN BY AI ##`)
2 parents c245fbd + 58665b6 commit 9669983

File tree

1 file changed

+13
-4
lines changed

1 file changed

+13
-4
lines changed

Containerfile

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -31,19 +31,28 @@ COPY / /src
3131

3232
# Install guidellm and locked dependencies
3333
RUN pdm use -p /src -f /opt/app-root \
34-
&& pdm install -p /src --check --prod --no-editable
34+
&& pdm install -p /src -G all --check --prod --no-editable
3535

3636
# Prod image
3737
FROM $BASE_IMAGE
3838

39+
# Switch to root for installing packages
40+
USER root
41+
42+
# Install some helpful utilities and deps
43+
RUN dnf install -y --setopt=install_weak_deps=False \
44+
vi tar rsync ffmpeg-free \
45+
&& dnf clean all
46+
47+
# Switch back to unpriv user
48+
# Root group for k8s
49+
USER 1001:0
50+
3951
# Add guidellm bin to PATH
4052
# Argument defaults can be set with GUIDELLM_<ARG>
4153
ENV HOME="/home/guidellm" \
4254
GUIDELLM_OUTPUT_PATH="/results/benchmarks.json"
4355

44-
# Make sure root is the primary group
45-
USER 1001:0
46-
4756
# Create the user home dir
4857
WORKDIR $HOME
4958

0 commit comments

Comments
 (0)