Skip to content

feat(observability): add OpenTelemetry metrics bootstrap with Prometheus sidecar - #653

Merged
Ferryx349 merged 5 commits into
cameri:mainfrom
Ferryx349:ADMIN-CONSOLE-2-metrics
Jun 29, 2026
Merged

feat(observability): add OpenTelemetry metrics bootstrap with Prometheus sidecar#653
Ferryx349 merged 5 commits into
cameri:mainfrom
Ferryx349:ADMIN-CONSOLE-2-metrics

Conversation

@Ferryx349

@Ferryx349Ferryx349 commented Jun 27, 2026•
edited
Loading

Copy link
Copy Markdown
Collaborator

Description

This PR adds foundation for Bootstrap OpenTelemetry metrics export (OTLP -> OTEL Collector -> Prometheus). Defines relay metric instruments and wires startup/shutdown in index.ts.

Related Issue

Fixes - #652

Types of changes

  • Non-functional change (docs, style, minor refactor)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my code changes.
  • I added a changeset, or this is docs-only and I added an empty changeset.
  • All new and existing tests passed.

@Ferryx349
feat(admin): add in-memory metrics collector foundation
bafdb19
Signed-off-by: ABHAY PANDEY <pandeyabhay967@gmail.com>
@changeset-bot

changeset-botBot commented Jun 27, 2026•
edited
Loading

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8b32148

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
NameType
nostreamMinor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coveralls

coveralls commented Jun 27, 2026•
edited
Loading

Copy link
Copy Markdown
Collaborator

Coverage Status

Coverage is 66.747% — Ferryx349:ADMIN-CONSOLE-2-metrics into cameri:main. No base build found for cameri:main.

@Ferryx349@cursoragent
feat(observability): add OpenTelemetry metrics bootstrap with Prometh…
75f6b92
…eus sidecar
Replace in-memory admin metrics collector with OTLP export to OTEL Collector
scraped by Prometheus. Foundation only — no handler wiring or admin API yet.
Co-authored-by: Cursor <cursoragent@cursor.com>
@socket-security

socket-securityBot commented Jun 29, 2026•
edited
Loading

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

DiffPackageSupply Chain
Security		VulnerabilityQualityMaintenanceLicense
Added@​opentelemetry/​api@​1.9.110010010086100
Added@​opentelemetry/​resources@​2.8.0991008794100
Added@​opentelemetry/​sdk-metrics@​2.8.010010010094100
Added@​opentelemetry/​exporter-metrics-otlp-http@​0.219.09910010096100

View full report

@Ferryx349Ferryx349 changed the title feat(admin): add in-memory metrics collector foundationfeat(observability): add OpenTelemetry metrics bootstrap with Prometheus sidecarJun 29, 2026
@Ferryx349
Ferryx349 requested a review from CopilotJune 29, 2026 10:17
CopilotAI reviewed Jun 29, 2026

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Ferryx349
Ferryx349 requested a review from cameriJune 29, 2026 10:17
@cameri
cameri requested a review from CopilotJune 29, 2026 18:24
CopilotAI reviewed Jun 29, 2026
View reviewed changes

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 5 comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

Comment threadsrc/telemetry/metrics.ts
Comment threadsrc/index.ts Outdated
Comment threadotel-collector-config.yaml Outdated
Comment threadsrc/telemetry/metrics.ts
Comment threadsrc/telemetry/metrics.ts
@Ferryx349
fix(observability): harden OTEL config and await metrics shutdown on …
beda2f8
…exit
Signed-off-by: ABHAY PANDEY <pandeyabhay967@gmail.com>
@Ferryx349
Ferryx349force-pushed the ADMIN-CONSOLE-2-metrics branch from 33fee1e to beda2f8CompareJune 29, 2026 18:55
@Ferryx349
Ferryx349 merged commit 8ab4825 into cameri:mainJun 29, 2026
18 checks passed
@camericameri mentioned this pull request Jun 29, 2026
Open
This was referenced Jun 30, 2026
Merged
Closed
@Justxd22Justxd22 mentioned this pull request Jul 7, 2026
Open
@Ferryx349Ferryx349 mentioned this pull request Jul 19, 2026
Closed
@Ferryx349Ferryx349 mentioned this pull request Aug 18, 2026
Closed
9 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@camericamericameri approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Ferryx349@coveralls@cameri