Add OpenTelemetry tracing with OpenInference support - #3773
Conversation
TL;DR
Remediation
Investigation detailsRoot CauseThe failed
Both functions are part of the PR’s new Phoenix tracing implementation, so the failure is introduced by this change. Evidence
VerificationNo commands were run against the PR checkout; the diagnosis is based on the failed Buildkite log and the PR source/configuration. What is this? | From workflow: PR Buildkite Detective Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not. |
mrodm
left a comment
mrodm
left a comment
There was a problem hiding this comment.
LGTM changes related to go.mod/go.sum and README.
Just added a minor question about the README.
Approved but @elastic/integration-experience must approve for the changes under internal/llmagent before merging this PR.
| - To configure LLM providers for AI-powered documentation generation (`elastic-package update documentation`): | ||
| - `ELASTIC_PACKAGE_LLM_PROVIDER` / **llm.provider**: Provider name. Gemini is the default provider. | ||
| - Gemini: `GOOGLE_API_KEY` / **llm.gemini.api_key**, `GEMINI_MODEL` / **llm.gemini.model**, `GEMINI_THINKING_BUDGET` / **llm.gemini.thinking_budget**. | ||
| - LLM tracing is configured via profile config only: **llm.tracing.enabled**, **llm.tracing.endpoint**, **llm.tracing.api_key**, **llm.tracing.project_name** (see `config.yml.example`). |
There was a problem hiding this comment.
Should it be mentioned that by default is enabled ?
There was a problem hiding this comment.
This is disabled by default. I've updated this line to say so.
mjwolf
changed the title
Adds OpenTelemetry tracing for LLM conversations and tool calls using OpenInference semantic conventions. Tracing is generic and can connect to any OTLP-compatible collector endpoint. Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
Make tracing backend-neutral and preserve accurate span, tool-call, and shutdown behavior across OTLP exporters. Co-authored-by: Cursor <[email protected]>
💚 Build Succeeded
History
|
|
Thanks Mike for adding the commit, I'm approving the overall all change. Requesting @elastic/integration-experience for another set of eyes. |
4 participants
Adds OpenTelemetry tracing for LLM conversations and tool calls using OpenInference semantic conventions. Tracing is generic and can connect to any OTLP-compatible collector endpoint.