Skip to content

Conversation

@lmolkova
Copy link
Member

Mark the following attributes sampling-relevant on all GenAI spans they appear

  • gen_ai.operation.name
  • gen_ai.provider.name
  • gen_ai.request.model
  • server.address
  • server.port

(also add gen_ai.provider.name to embeddings spans - it seems it has been forgotten)

@lmolkova lmolkova force-pushed the gen-ai-sampling-relevant branch from 6ac16cc to 05eb438 Compare October 28, 2025 19:28
@lmolkova lmolkova moved this from Untriaged to Awaiting codeowners approval in Semantic Conventions Triage Oct 28, 2025
@github-actions
Copy link

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Nov 13, 2025
@github-actions github-actions bot removed the Stale label Nov 16, 2025
@github-actions
Copy link

This PR has been labeled as stale due to lack of activity. It will be automatically closed if there is no further activity over the next 14 days.

@github-actions github-actions bot added the Stale label Nov 30, 2025
@github-actions github-actions bot closed this Dec 7, 2025
@lmolkova lmolkova reopened this Dec 30, 2025
@lmolkova lmolkova removed the Stale label Dec 30, 2025
Copy link

@JWinermaSplunk JWinermaSplunk left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link

@wrisa wrisa left a comment

Choose a reason for hiding this comment

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

Added a couple of comments. Rest LGTM.

it by default. Instrumentations MAY provide a way to enable
populating this attribute.

The following attributes can be important for making sampling decisions
Copy link

Choose a reason for hiding this comment

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

Why gen_ai.request.model is missing ?

recommended: when span kind is `CLIENT`.
sampling_relevant: true
- ref: server.port
sampling_relevant: true
Copy link

Choose a reason for hiding this comment

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

gen_ai.request.model missing ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:gen-ai enhancement New feature or request

Projects

Status: Awaiting codeowners approval

Development

Successfully merging this pull request may close these issues.

5 participants