Skip to content

Commit da406e6

Browse files
fix: remove api-store from container (#617)
1 parent 28f9697 commit da406e6

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

container/Dockerfile.vllm

+1-4
Original file line numberDiff line numberDiff line change
@@ -342,8 +342,6 @@ RUN source /opt/dynamo/venv/bin/activate && \
342342
uv build --wheel --out-dir /workspace/dist --python 3.10; \
343343
fi && \
344344
cd /workspace && \
345-
uv build --wheel --out-dir /workspace/dist && \
346-
cd /workspace/deploy/dynamo/api-store && \
347345
uv build --wheel --out-dir /workspace/dist
348346

349347
#######################################
@@ -362,8 +360,7 @@ RUN mkdir -p /opt/dynamo/bindings/wheels && \
362360
cp -r lib/bindings/c/include /opt/dynamo/bindings/.
363361

364362
RUN uv pip install /workspace/dist/ai_dynamo_runtime*cp312*.whl && \
365-
uv pip install /workspace/dist/ai_dynamo*any.whl && \
366-
uv pip install /workspace/dist/ai_dynamo_store*any.whl
363+
uv pip install /workspace/dist/ai_dynamo*any.whl
367364

368365
##########################################
369366
########## Perf Analyzer Image ###########

pyproject.toml

+2
Original file line numberDiff line numberDiff line change
@@ -122,6 +122,7 @@ tmp_path_retention_policy = "failed"
122122
# NOTE
123123
# We ignore model.py explcitly here to avoid mypy errors with duplicate modules
124124
# pytest overrides the default mypy exclude configuration and so we exclude here as well
125+
# Ignore mypy check for api-store component from Dynamo Deploy. Mypy analysis will fail since this package (and its dependencies) are not installed.
125126
addopts = [
126127
"-ra",
127128
"--showlocals",
@@ -130,6 +131,7 @@ addopts = [
130131
"--mypy",
131132
"--ignore-glob=*model.py",
132133
"--ignore-glob=*_inc.py",
134+
"--ignore-glob=deploy/dynamo/api-store/*",
133135
# FIXME: Get relative/generic blob paths to work here
134136
]
135137
xfail_strict = true

0 commit comments

Comments
 (0)