Skip to content

channel should flow to all scopes#70

Merged
nikhilNava merged 2 commits intomainfrom
nikhilc/channel
Nov 25, 2025
Merged

channel should flow to all scopes#70
nikhilNava merged 2 commits intomainfrom
nikhilc/channel

Conversation

@nikhilNava
Copy link
Contributor

Task
channel info should be propagated to all tables

Copilot AI review requested due to automatic review settings November 24, 2025 20:44
@nikhilNava nikhilNava requested a review from a team as a code owner November 24, 2025 20:44
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 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 review requested due to automatic review settings November 25, 2025 12:42
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

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 nikhilNava enabled auto-merge (squash) November 25, 2025 17:16
@nikhilNava nikhilNava merged commit bf32f95 into main Nov 25, 2025
14 checks passed
@nikhilNava 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