Skip to content

Conversation

@mxiamxia
Copy link
Member

@mxiamxia mxiamxia commented Jul 1, 2025

Description of changes:

  • Rename aws.ai.agent.type to/and set aws.service.type with default value gen_ai_agent if no custom set
  • rename

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@mxiamxia mxiamxia marked this pull request as ready for review July 1, 2025 00:18
@mxiamxia mxiamxia requested a review from a team as a code owner July 1, 2025 00:18
Copy link
Contributor

@srprash srprash left a comment

Choose a reason for hiding this comment

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

LGTM. However, I have some concerns with introducing aws.service.type without considering cases other than gen_ai_agent. Since it is such a broad attribute, we probably need to have an idea on possible values for it and some spec around those values.
Anyway, I know we want to get this in quickly and I don't have strong suggestions for this right now. Hopefully, this will work out well in the future.

@mxiamxia mxiamxia merged commit 21ba7fc into aws-observability:main Jul 1, 2025
13 checks passed
jj22ee added a commit to aws-observability/aws-otel-js-instrumentation that referenced this pull request Jul 2, 2025
*Issue #, if available:*
JS equivalent of:
-
aws-observability/aws-otel-python-instrumentation#414
-
aws-observability/aws-otel-python-instrumentation#415

*Description of changes:*


By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice.
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.

2 participants