Skip to content

refactor(pprof): extract infrastructure provider#10673

Merged
therealpandey merged 3 commits intomainfrom
feat/pprof-infra-layer
Mar 23, 2026
Merged

refactor(pprof): extract infrastructure provider#10673
therealpandey merged 3 commits intomainfrom
feat/pprof-infra-layer

Conversation

@therealpandey
Copy link
Member

@therealpandey therealpandey commented Mar 22, 2026

Summary

  • extract pprof into a dedicated infrastructure provider with http and noop implementations
  • wire pprof through top-level config and the shared service registry
  • remove ad hoc pprof startup from community and ee query-service servers

Testing

  • go test ./pkg/pprof/... ./pkg/signoz ./pkg/query-service/app ./ee/query-service/app ./cmd ./cmd/community ./cmd/enterprise

@therealpandey therealpandey added the safe-to-integrate Run integration tests label Mar 22, 2026
@therealpandey therealpandey added this pull request to the merge queue Mar 23, 2026
Merged via the queue into main with commit 807211b Mar 23, 2026
17 checks passed
@therealpandey therealpandey deleted the feat/pprof-infra-layer branch March 23, 2026 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe-to-integrate Run integration tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants