Repository navigation
examples: the Docker paths of 07 to 12 run with uv run --with, no venv and no uv pip - #138
Merged
Merged
Conversation
…nv and no uv pip (#136) Examples 07 to 12 built a venv inside each container (python -m venv, activate, uv pip install) before running the example. Each path now bootstraps uv as before and runs `uv run --no-project --with <pkg> ... --with "<wheel>" python -u <example>.py ...`. Same packages, extras, wheel and arguments. The python3-venv apt packages are no longer installed. The Milvus runner image (examples 11 and 12) no longer installs numpy, pymilvus and psutil into the system Python with uv pip; the container runs them through uv run instead. Its tag moves from py312-v1 to py312-v2 so a stale local image is rebuilt. Checked: all 31 container command lines parse with sh -n and contain no venv or pip; 07 arcadedb_sql ran end to end in its container on stackoverflow-tiny. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…nd install ca-certificates (#136) The pgvector path (Postgres image) and the Milvus custom-image path get python3 from apt; without the old venv there is no `python` on PATH ("python: not found"), and dropping python3-venv also dropped ca-certificates (pulled in through python3-pip-whl), so the uv download failed TLS verification. Found by the smoke run of every container path; 11 and 12 pgvector now pass. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #136
The
--dockerpaths of examples 07 to 12 no longer build a venv and runuv pip installinside their containers. After the uv bootstrap, each path runs oneuv run --no-project --with <packages> --with "<wheel>" python -u <example>.py ...with the same packages, extras and wheel as before. The Milvus runner image no longer runsuv pip install --system(its tag moves topy312-v2so a stale local image is rebuilt), andpython3-venvis no longer installed.Two fixes came out of running every container path:
python3from apt. Without the old venv there is nopython, so the uv bootstrap now callspython3there.python3-venvalso droppedca-certificates(Debian pulls it in throughpython3-pip-whl), so the uv download failed TLS verification. Those two paths now installca-certificatesexplicitly.Smoke runs (laptop,
stackoverflow-tiny, the CI arguments,arcadedb_jvm26.11.1.dev0 wheel, containers pinned to two cores): 21 container paths pass, 6 skipped.arcadedb_sqlduckdbpostgresqlarcadedb_sqlduckdbpostgresqlarcadedb_cypherladybugduckdbneo4jarcadedb_cypherladybugduckdbneo4jarcadedb_sqlfaisslancedbpgvectorqdrantmilvusarcadedb_sqlfaisslancedbpgvectorbruteforceqdrantmilvusThe skipped paths (neo4j for 09 and 10, qdrant and Milvus for 11 and 12) start sidecar server containers through the Docker socket. Their runner commands use the same
uv run --withpattern as the passing paths, and every one of the 31 generated container commands passessh -nwith no venv or pip in it, but they were not run here. The examples CI does not take the--dockerpaths.Static gate:
compile()and pyflakes (the same 2 warnings as on main, in 09), black and isort. Follow-up: #137 pins the uv version that the bootstrap downloads.🤖 Generated with Claude Code