Add OpenTelemetry runtime deps and rebuild

coilysiren commented

2026-07-02 10:47:19 +00:00

Owner

The proxy code already imports OpenTelemetry SDK/exporter and FastAPI/HTTPX instrumentation, but the runtime dependency set does not include those packages. That means the image can boot without real OTel wiring or fail once tracing/export is enabled.

Required work:

Add runtime deps for opentelemetry-api, opentelemetry-sdk, opentelemetry-exporter-otlp-proto-http, opentelemetry-instrumentation-fastapi, and opentelemetry-instrumentation-httpx.
Refresh uv.lock.
Rebuild or run the repo boot validation so the frozen runtime path proves the image serves /healthz, /v1/models, and /metrics.
Run the test suite and the repo pre-commit gate if available.

Scratch validation from the read-only surface session:

uv lock resolved the OTel packages.
uv sync --extra dev succeeded.
uv run pytest passed: 32 passed, 1 Starlette/httpx deprecation warning.
./boot_probe.sh passed.
uv run pre-commit run --all-files could not run because .pre-commit-config.yaml is absent.

Close this issue from the engineer commit with closes #<issue number>.

The proxy code already imports OpenTelemetry SDK/exporter and FastAPI/HTTPX instrumentation, but the runtime dependency set does not include those packages. That means the image can boot without real OTel wiring or fail once tracing/export is enabled. Required work: - Add runtime deps for `opentelemetry-api`, `opentelemetry-sdk`, `opentelemetry-exporter-otlp-proto-http`, `opentelemetry-instrumentation-fastapi`, and `opentelemetry-instrumentation-httpx`. - Refresh `uv.lock`. - Rebuild or run the repo boot validation so the frozen runtime path proves the image serves `/healthz`, `/v1/models`, and `/metrics`. - Run the test suite and the repo pre-commit gate if available. Scratch validation from the read-only surface session: - `uv lock` resolved the OTel packages. - `uv sync --extra dev` succeeded. - `uv run pytest` passed: 32 passed, 1 Starlette/httpx deprecation warning. - `./boot_probe.sh` passed. - `uv run pre-commit run --all-files` could not run because `.pre-commit-config.yaml` is absent. Close this issue from the engineer commit with `closes #<issue number>`.

coilyco-ops commented

2026-07-02 10:47:27 +00:00

Member

🔒 Reserved by ward agent --driver codex — container engineer-codex-agent-proxy-30 on host kais-macbook-pro-2.local is carrying this issue (reserved 2026-07-02T10:47:25Z). Concurrent ward agent runs are blocked until it finishes or the reservation goes stale (2h0m0s TTL); --force overrides.

— Codex, via ward agent

🔒 Reserved by `ward agent --driver codex` — container `engineer-codex-agent-proxy-30` on host `kais-macbook-pro-2.local` is carrying this issue (reserved 2026-07-02T10:47:25Z). Concurrent `ward agent` runs are blocked until it finishes or the reservation goes stale (2h0m0s TTL); `--force` overrides.  — Codex, via `ward agent`

coilysiren referenced this issue from a commit

2026-07-02 10:48:36 +00:00

Add OpenTelemetry runtime deps

coilysiren closed this issue

2026-07-02 10:48:36 +00:00

coilysiren commented

2026-07-02 10:48:54 +00:00

Author

Owner

WARD-OUTCOME: done - added OTel runtime deps and refreshed uv.lock

This was straightforward once I confirmed the imports were already there. The only surprise was that ward was not resolving the repo config in this container, so I validated with the direct uv and boot scripts instead. Confidence is high, and the only rough edge is the absent pre-commit config, which matched the issue notes.

WARD-OUTCOME: done - added OTel runtime deps and refreshed uv.lock This was straightforward once I confirmed the imports were already there. The only surprise was that `ward` was not resolving the repo config in this container, so I validated with the direct `uv` and boot scripts instead. Confidence is high, and the only rough edge is the absent pre-commit config, which matched the issue notes.

Rows
Columns

Add OpenTelemetry runtime deps and rebuild #30