Skip to content

feat: add OpenTelemetry observability - #179

Merged
kavix merged 2 commits into
kavix:mainfrom
Sanchar127:feat/opentelemetry-observability
Aug 22, 2026
Merged

feat: add OpenTelemetry observability#179
kavix merged 2 commits into
kavix:mainfrom
Sanchar127:feat/opentelemetry-observability

Conversation

@Sanchar127

Copy link
Copy Markdown
Contributor

Implemented OpenTelemetry tracing and Prometheus metrics for Eko, covering LLM calls, CAS operations, and SQLite queries.

Added proper telemetry initialization and shutdown handling for Eko’s short-lived CLI lifecycle, ensuring metrics and traces are flushed before exit.

Related Issues

Resolves #146

@vercel

vercel Bot commented Aug 17, 2026

Copy link
Copy Markdown

@Sanchar127 is attempting to deploy a commit to the Kavindu's projects Team on Vercel.

A member of the Team first needs to authorize it.

@kavix
kavix requested review from kavix and a lite review from Copilot August 18, 2026 06:39

This comment was marked as off-topic.

@kavix

kavix commented Aug 18, 2026

Copy link
Copy Markdown
Owner

@Sanchar127 can u resolve the conflicts

@kavix
kavix requested a lite review from Copilot August 18, 2026 07:27

This comment was marked as off-topic.

@Sanchar127

Copy link
Copy Markdown
Contributor Author

Resolved the merge conflicts with upstream/main. I also verified go test -race ./... against a clean upstream/main worktree; the existing TestProgress_StartStop race reproduces there as well, in the upstream progress-bar implementation.

@kavix
kavix merged commit 742bb75 into kavix:main Aug 22, 2026
2 of 3 checks passed
@kavix

kavix commented Aug 22, 2026

Copy link
Copy Markdown
Owner

Resolved the merge conflicts with upstream/main. I also verified go test -race ./... against a clean upstream/main worktree; the existing TestProgress_StartStop race reproduces there as well, in the upstream progress-bar implementation.

Thanks for the contribution

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.

arch: Add OpenTelemetry (OTel) Distributed Tracing & Prometheus Metrics

3 participants