Commit 8e8cfaf
authored
[codex] install backend Docker deps from uv locks (#8328)
## Summary
- install backend and pusher Docker Python dependencies from uv pylock
files
- add runtime-only backend and pusher-specific lock files for Linux
container builds
- update lock regeneration docs/script behavior so existing pins are
preserved by default and `PYLOCK_UPGRADE=1` performs intentional
upgrades
- add the missing pusher `google-cloud-tasks` dependency surfaced by
container smoke testing
## Validation
- `./scripts/update-python-lock.sh`
- `sh -n backend/scripts/update-python-lock.sh`
- workflow YAML parse checks for backend workflows
- `git diff --check`
- `./scripts/pre-commit`
- Oracle second-model review, with must-fix feedback addressed
- `docker pull --platform linux/amd64
gcr.io/based-hardware-dev/python:3.11-slim-forky`
- `docker build --platform linux/amd64 -f backend/Dockerfile -t
omi-backend-uv-runtime:gcr-test .`
- `docker run --rm --platform linux/amd64
omi-backend-uv-runtime:gcr-test python -c "from google.cloud import
tasks_v2; import fastapi, uvicorn, onnxruntime, lc3, av; print('backend
runtime imports ok')"`
- `docker build --platform linux/amd64 -f backend/pusher/Dockerfile -t
omi-pusher-uv-runtime:gcr-test .`
- `docker run --rm --platform linux/amd64 omi-pusher-uv-runtime:gcr-test
python -c "from google.cloud import tasks_v2; import fastapi, uvicorn,
lc3, av, scipy; print('pusher runtime imports ok')"`
## Notes
- `backend/test-preflight.sh` was not a useful signal on this machine
because the local shell is on Python 3.9 without backend packages
installed; Redis and test file checks were fine.
- Docker still reports the existing `$LD_LIBRARY_PATH` undefined-var
warning in these Dockerfiles; it does not block either image build.
<!-- This is an auto-generated description by cubic. -->
<a
href="https://cubic.dev/pr/BasedHardware/omi/pull/8328?utm_source=github"
target="_blank" rel="noopener noreferrer"
data-no-image-dialog="true"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://www.cubic.dev/buttons/review-in-cubic-dark.svg"><source
media="(prefers-color-scheme: light)"
srcset="https://www.cubic.dev/buttons/review-in-cubic-light.svg"><img
alt="Review in cubic"
src="https://www.cubic.dev/buttons/review-in-cubic-dark.svg"></picture></a>
<!-- End of auto-generated description by cubic. -->12 files changed
Lines changed: 2524 additions & 24 deletions
File tree
- backend
- pusher
- scripts
- docs/doc/developer/backend
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
| 22 | + | |
| 23 | + | |
22 | 24 | | |
23 | 25 | | |
24 | 26 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
2 | 7 | | |
3 | | - | |
4 | 8 | | |
| 9 | + | |
| 10 | + | |
5 | 11 | | |
6 | 12 | | |
7 | 13 | | |
| |||
28 | 34 | | |
29 | 35 | | |
30 | 36 | | |
31 | | - | |
| 37 | + | |
32 | 38 | | |
33 | | - | |
34 | | - | |
| 39 | + | |
| 40 | + | |
35 | 41 | | |
36 | | - | |
| 42 | + | |
37 | 43 | | |
38 | 44 | | |
39 | 45 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
2 | 7 | | |
3 | | - | |
4 | 8 | | |
| 9 | + | |
| 10 | + | |
5 | 11 | | |
6 | 12 | | |
7 | 13 | | |
| |||
28 | 34 | | |
29 | 35 | | |
30 | 36 | | |
31 | | - | |
| 37 | + | |
32 | 38 | | |
33 | | - | |
34 | | - | |
| 39 | + | |
| 40 | + | |
35 | 41 | | |
36 | | - | |
| 42 | + | |
37 | 43 | | |
38 | 44 | | |
39 | 45 | | |
| |||
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
| 22 | + | |
22 | 23 | | |
23 | 24 | | |
24 | 25 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| |||
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| |||
0 commit comments