Skip to content

Add process-scoped SQLite telemetry#22152

Closed
jif-oai wants to merge 1 commit into
mainfrom
jif/sqlite-telemetry-small
Closed

Add process-scoped SQLite telemetry#22152
jif-oai wants to merge 1 commit into
mainfrom
jif/sqlite-telemetry-small

Conversation

@jif-oai
Copy link
Copy Markdown
Collaborator

@jif-oai jif-oai commented May 11, 2026

Summary

  • add SQLite init, backfill-gate, and fallback telemetry without introducing a cross-cutting state-db access wrapper
  • install one process-scoped telemetry sink after OTEL startup and let low-level state/rollout paths emit through it directly
  • add process-start metrics for the process owners that initialize SQLite

Notes

Test plan

  • RUST_MIN_STACK=33554432 cargo test -p codex-state -p codex-rollout -p codex-thread-store -p codex-otel
  • RUST_MIN_STACK=33554432 cargo test -p codex-app-server --lib
  • RUST_MIN_STACK=33554432 cargo test -p codex-mcp-server --lib
  • RUST_MIN_STACK=33554432 cargo test -p codex-tui --lib
  • cargo check -p codex-state -p codex-rollout -p codex-thread-store -p codex-core -p codex-app-server -p codex-app-server-client -p codex-exec -p codex-tui -p codex-mcp-server
  • just fix -p codex-state -p codex-rollout -p codex-otel -p codex-core -p codex-app-server -p codex-exec -p codex-mcp-server -p codex-tui
  • just fmt

Known unrelated local failures:

  • codex-exec::tests::thread_start_params_include_review_policy_when_review_policy_is_manual_only fails the same way on untouched main
  • broader app-server integration tests fail in this sandbox because sandbox-exec cannot apply the profile here

Co-authored-by: Codex <noreply@openai.com>
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