Skip to content

Add OTEL metrics for hook runs#18026

Merged
abhinav-oai merged 16 commits into
mainfrom
dev/abhinav/hook-run-otel-metrics
Apr 16, 2026
Merged

Add OTEL metrics for hook runs#18026
abhinav-oai merged 16 commits into
mainfrom
dev/abhinav/hook-run-otel-metrics

Conversation

@abhinav-oai
Copy link
Copy Markdown
Collaborator

@abhinav-oai abhinav-oai commented Apr 16, 2026

Why

We already emit analytics for completed hook runs, but we don't have matching OTEL metrics to track hook volume and latency.

What

  • add codex.hooks.run and codex.hooks.run.duration_ms
  • tag both metrics with hook_name, source, and status
  • emit the metrics from the completed hook path

Verified locally against a dummy OTLP collector

@abhinav-oai abhinav-oai marked this pull request as ready for review April 16, 2026 16:43
…tics' into dev/abhinav/hook-run-otel-metrics

# Conflicts:
#	codex-rs/analytics/src/facts.rs
#	codex-rs/core/src/hook_runtime.rs
Base automatically changed from dev/abhinav/codex-hook-run-analytics to main April 16, 2026 19:43
@abhinav-oai abhinav-oai enabled auto-merge (squash) April 16, 2026 21:25
@abhinav-oai abhinav-oai merged commit d9c71d4 into main Apr 16, 2026
34 of 36 checks passed
@abhinav-oai abhinav-oai deleted the dev/abhinav/hook-run-otel-metrics branch April 16, 2026 21:30
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants