Skip to content

Commit 66c136b

Browse files
committed
Build image in CI checks
Signed-off-by: Pierangelo Di Pilato <[email protected]>
1 parent 6300e50 commit 66c136b

File tree

2 files changed

+21
-11
lines changed

2 files changed

+21
-11
lines changed

.github/workflows/ci-pr-checks.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,10 @@ jobs:
5757
CPATH: ${{ env.CPATH }}
5858
PKG_CONFIG_PATH: ${{ env.PKG_CONFIG_PATH }}
5959

60+
- name: Build Container image
61+
run: |
62+
make image-build SIM_TAG=pr-check
63+
6064
- name: Run go test
6165
shell: bash
6266
run: |

Dockerfile

Lines changed: 17 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -28,11 +28,13 @@ ARG TOKENIZER_VERSION=v1.22.1
2828
RUN curl -L https://github.com/daulet/tokenizers/releases/download/${TOKENIZER_VERSION}/libtokenizers.${TARGETOS}-${TARGETARCH}.tar.gz | tar -xz -C lib
2929
RUN ranlib lib/*.a
3030

31-
# Copy Python wrapper from kv-cache-manager dependency
31+
# Copy Python wrapper and requirements from kv-cache-manager dependency
3232
# Extract version dynamically and copy to a known location
3333
RUN KV_CACHE_MGR_VERSION=$(go list -m -f '{{.Version}}' github.com/llm-d/llm-d-kv-cache-manager) && \
3434
mkdir -p /workspace/kv-cache-manager-wrapper && \
3535
cp /go/pkg/mod/github.com/llm-d/llm-d-kv-cache-manager@${KV_CACHE_MGR_VERSION}/pkg/preprocessing/chat_completions/render_jinja_template_wrapper.py \
36+
/workspace/kv-cache-manager-wrapper/ && \
37+
cp /go/pkg/mod/github.com/llm-d/llm-d-kv-cache-manager@${KV_CACHE_MGR_VERSION}/pkg/preprocessing/chat_completions/requirements.txt \
3638
/workspace/kv-cache-manager-wrapper/
3739

3840
# Build
@@ -59,24 +61,28 @@ USER root
5961
RUN curl -L -o /tmp/epel-release.rpm https://dl.fedoraproject.org/pub/epel/epel-release-latest-9.noarch.rpm && \
6062
rpm -i /tmp/epel-release.rpm && \
6163
rm /tmp/epel-release.rpm && \
62-
microdnf install -y --setopt=install_weak_deps=0 zeromq python3 python3-libs python3-pip && \
64+
microdnf install -y --setopt=install_weak_deps=0 zeromq python3.12 python3.12-libs python3.12-pip && \
6365
microdnf clean all && \
64-
rm -rf /var/cache/yum /var/lib/yum
66+
rm -rf /var/cache/yum /var/lib/yum && \
67+
ln -sf /usr/bin/python3.12 /usr/bin/python3 && \
68+
ln -sf /usr/bin/python3.12 /usr/bin/python
6569

6670
# Install wrapper as a module in site-packages
67-
# Extract the kv-cache-manager version dynamically from go.mod in the builder stage
68-
RUN mkdir -p /usr/local/lib/python3.9/site-packages/
69-
COPY --from=builder /workspace/kv-cache-manager-wrapper/render_jinja_template_wrapper.py /usr/local/lib/python3.9/site-packages/
71+
RUN mkdir -p /usr/local/lib/python3.12/site-packages/
72+
COPY --from=builder /workspace/kv-cache-manager-wrapper/render_jinja_template_wrapper.py /usr/local/lib/python3.12/site-packages/
7073

71-
# Python deps (no cache, single target) – install transformers
74+
# Python deps (no cache, single target) – filter out torch
7275
ENV PIP_NO_CACHE_DIR=1 PIP_DISABLE_PIP_VERSION_CHECK=1
73-
RUN python3 -m pip install --no-cache-dir --upgrade pip setuptools wheel && \
74-
python3 -m pip install --no-cache-dir --target /usr/local/lib/python3.9/site-packages transformers && \
76+
COPY --from=builder /workspace/kv-cache-manager-wrapper/requirements.txt /tmp/requirements.txt
77+
RUN sed '/^torch\b/d' /tmp/requirements.txt > /tmp/requirements.notorch.txt && \
78+
python3.12 -m pip install --no-cache-dir --upgrade pip setuptools wheel && \
79+
python3.12 -m pip install --no-cache-dir --target /usr/local/lib/python3.12/site-packages -r /tmp/requirements.notorch.txt && \
80+
rm /tmp/requirements.txt /tmp/requirements.notorch.txt && \
7581
rm -rf /root/.cache/pip
7682

7783
# Python env
78-
ENV PYTHONPATH="/usr/local/lib/python3.9/site-packages:/usr/lib/python3.9/site-packages"
79-
ENV PYTHON=python3
84+
ENV PYTHONPATH="/usr/local/lib/python3.12/site-packages:/usr/lib/python3.12/site-packages"
85+
ENV PYTHON=python3.12
8086

8187
COPY --from=builder /workspace/bin/llm-d-inference-sim /app/llm-d-inference-sim
8288

0 commit comments

Comments
 (0)