Skip to content

channel should flow to all scopes - #70

Merged
Nikhil Navakiran (nikhilNava) merged 2 commits into
mainfrom
nikhilc/channel
Nov 25, 2025
Merged

channel should flow to all scopes#70
Nikhil Navakiran (nikhilNava) merged 2 commits into
mainfrom
nikhilc/channel

Conversation

@nikhilNava

Copy link
Copy Markdown
Contributor

Task
channel info should be propagated to all tables

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR restructures the propagation of channel-related execution context attributes to flow to all span scopes instead of being limited to invoke-agent operations. Two constants (GEN_AI_EXECUTION_SOURCE_NAME_KEY and GEN_AI_EXECUTION_SOURCE_DESCRIPTION_KEY) are moved from INVOKE_AGENT_ATTRIBUTES to COMMON_ATTRIBUTES, ensuring channel information is captured across all telemetry scopes.

Key Changes

  • Channel source metadata attributes (gen_ai.channel.name and gen_ai.channel.link) are now propagated to all spans, not just invoke-agent spans
  • The "Execution context" comment grouping is moved with these attributes to maintain logical organization

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

…gents_a365/observability/core/trace_processor/util.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 1 out of 1 changed files in this pull request and generated 1 comment.


You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

@nikhilNava
Nikhil Navakiran (nikhilNava) merged commit bf32f95 into main Nov 25, 2025
14 checks passed
@nikhilNava
Nikhil Navakiran (nikhilNava) deleted the nikhilc/channel branch November 25, 2025 17:16
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.

5 participants