Skip to content

Add an installed Codex tracing journey - #739

Open
tt-le wants to merge 1 commit into
tien/codex-otel-gateway-proxyfrom
tien/codex-otel-tracing-integration
Open

tt-le wants to merge 1 commit into
tien/codex-otel-gateway-proxyfrom
tien/codex-otel-tracing-integration

Conversation

@tt-le

@tt-le tt-le commented Sep 18, 2026

Copy link
Copy Markdown
Collaborator

What & why

Stacked on #718.

Add one installed-product customer journey for the Codex tracing path introduced in #718:

  1. Install the checkout wheel and exact Codex version through the integration runner.
  2. Configure Codex against eng-ml-inference.staging, where tracing ingestion is enabled.
  3. Complete a real headless file task with a unique UUID in the prompt.
  4. Wait 30 seconds, query the configured OTel spans table, and require a matching span.

Codex intentionally excludes prompt text from trace-safe spans, so the same UUID is also supplied through Codex's supported otel.span_attributes configuration and queried as ug_integration_marker. This proves the installed Codex → loopback OTLP proxy → gateway → tracing-table path without weakening prompt redaction.

The CI lane is separate from the ordinary live jobs and targets staging until ingestion is available in their production workspace. It requires the staging-scoped UCODE_TRACING_DATABRICKS_BEARER repository secret. The bearer is supplied to ug; Codex sends credential-free OTLP requests to the loopback proxy, which injects the current Databricks token only on the upstream trace request. Inference traffic does not use this proxy.

Testing

  • Integration contract and evidence tests pass.
  • Ruff check and formatting pass.
  • The new test files pass ty.
  • Integration collection selects exactly one tracing and codex case; documented suite counts were reconciled with collection.
  • A live staging execution was not run locally because no staging credential was selected.

This pull request and its description were written by Isaac.

@tt-le
tt-le force-pushed the tien/codex-otel-tracing-integration branch from e47cf08 to 7f8a34b Compare September 18, 2026 14:20
An error occurred while trying to automatically change base from tien/codex-otel-gateway-proxy to tien/gateway-proxy-parameterize September 18, 2026 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant