Skip to content

Conversation

@EItanya
Copy link
Contributor

@EItanya EItanya commented Feb 2, 2026

No description provided.

Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya EItanya requested a review from peterj as a code owner February 2, 2026 14:21
Copilot AI review requested due to automatic review settings February 2, 2026 14:21
@EItanya EItanya requested a review from yuval-k as a code owner February 2, 2026 14:21
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates the OpenTelemetry tracing configuration to use agent-specific service names instead of the hardcoded "kagent" service name. The changes enable better tracing granularity by identifying each agent individually in telemetry data.

Changes:

  • Modified the configure function in kagent-core to accept name and namespace parameters and use them for the OpenTelemetry Resource
  • Updated all call sites in kagent-openai, kagent-crewai, and kagent-adk packages to pass agent name and namespace to the tracing configuration function

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
python/packages/kagent-core/src/kagent/core/tracing/_utils.py Updated configure function signature to accept name and namespace parameters and use them in the OpenTelemetry Resource instead of hardcoded "kagent"
python/packages/kagent-openai/src/kagent/openai/_a2a.py Updated configure_tracing call to pass self.config.name and self.config.namespace
python/packages/kagent-crewai/src/kagent/crewai/_a2a.py Updated configure_tracing call to pass self.config.name and self.config.namespace
python/packages/kagent-adk/src/kagent/adk/cli.py Updated two configure_tracing calls to pass app_cfg.name and app_cfg.namespace

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya EItanya merged commit 34ef659 into main Feb 2, 2026
16 checks passed
@EItanya EItanya deleted the eitanya/fix-otel-service-name branch February 2, 2026 14:46
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.

3 participants