Skip to content

[codex] add connector runtime latency metrics#31319

Merged
mzeng-openai merged 1 commit into
mainfrom
dev/mzeng/apps-runtime-metrics
Jul 7, 2026
Merged

[codex] add connector runtime latency metrics#31319
mzeng-openai merged 1 commit into
mainfrom
dev/mzeng/apps-runtime-metrics

Conversation

@mzeng-openai

Copy link
Copy Markdown
Collaborator

Summary

  • measure successful legacy app/list latency with codex.apps.installed.duration_ms, segmented by path=legacy and reload
  • measure successful host-owned codex_apps startup and explicit refresh latency with codex.apps.refresh.duration_ms
  • add the refresh trigger to successful codex.mcp.tools.fetch_uncached.duration_ms samples for codex_apps without changing other MCP-server samples

Why

This establishes a small latency baseline for the current connector path before ConnectorRuntimeManager, app/installed, and app/read land. Error-rate and broader runtime-state instrumentation are intentionally deferred.

This is telemetry-only and does not change connector behavior.

Validation

  • just test -p codex-mcp (94 passed)
  • just test -p codex-app-server list_apps (13 passed)
  • just fix -p codex-mcp
  • just fix -p codex-app-server
  • just fmt
  • git diff --check

@mzeng-openai mzeng-openai force-pushed the dev/mzeng/apps-runtime-metrics branch from 796153d to 2378474 Compare July 7, 2026 00:02
@mzeng-openai mzeng-openai marked this pull request as ready for review July 7, 2026 00:47
@mzeng-openai mzeng-openai merged commit 1345c16 into main Jul 7, 2026
63 of 66 checks passed
@mzeng-openai mzeng-openai deleted the dev/mzeng/apps-runtime-metrics branch July 7, 2026 17:49
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants