Skip to content

[observability] Add profiler and benchmark support#5

Open
mfittko wants to merge 1 commit intofeat/initial-openai-proxyfrom
feat/profiler-follow-up
Open

[observability] Add profiler and benchmark support#5
mfittko wants to merge 1 commit intofeat/initial-openai-proxyfrom
feat/profiler-follow-up

Conversation

@mfittko
Copy link
Copy Markdown
Contributor

@mfittko mfittko commented Apr 3, 2026

Summary

Add hot-path profiling hooks and local benchmark support.

Changes

  • add OpenAIProxy::HotPathProfiler
  • add profiler instrumentation through Application, Proxy, TokenValidator, UsageEventBuilder, and UsageQueue
  • add local benchmark scripts under script/benchmark/
  • add the related Makefile target and documentation for local profiling flows
  • add profiler-specific test coverage

Scope

PR #1 keeps the runtime MVP path narrower while still exposing upstream timing headers.

This PR adds the optional instrumentation and local benchmark tooling on top.

Testing

  • make coverage
  • make lint

@mfittko mfittko self-assigned this Apr 3, 2026
@mfittko mfittko force-pushed the feat/profiler-follow-up branch from 1b01b62 to 0043ae7 Compare April 3, 2026 21:34
@mfittko mfittko changed the title [observability] Restore profiler and benchmark support [observability] Add profiler and benchmark support Apr 3, 2026
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