Skip to content

Conversation

lcian
Copy link
Member

@lcian lcian commented Aug 28, 2025

  • Adds a test to ensure that, for each group of attributes that alias each other, there should be (at most) a single canonical way to name that attribute, all of its aliases should be marked as deprecated.

  • Address test failures by editing the conventions in the following ways:

Group [ai.input_messages, gen_ai.prompt, gen_ai.request.messages]
has 2 non-deprecated attributes: [gen_ai.prompt, gen_ai.request.messages]

Action: deprecated gen_ai.prompt in favor of gen_ai.request.messages.
Reason: gen_ai.prompt is deprecated in OTEL.

Group [http.route, url.template]
has 2 non-deprecated attributes: [http.route, url.template]
Group [params.<key>, url.path.parameter.<key>]
has 2 non-deprecated attributes: [params.<key>, url.path.parameter.<key>]
Group [sentry.release, service.version]
has 2 non-deprecated attributes: [sentry.release, service.version]
Group [client.address, http.host, server.address]
has 2 non-deprecated attributes: [client.address, server.address]
Group [http.response.body.size, http.response.header.content-length, http.response.header['content-length'], http.response_content_length]
has 2 non-deprecated attributes: [http.response.body.size, http.response.header.content-length]

@lcian lcian marked this pull request as draft August 28, 2025 11:31
@lcian lcian changed the title test: ensure there's a single non-deprecated attribute per alias group test: Ensure there's a single non-deprecated attribute per alias group Sep 3, 2025
Base automatically changed from lcian/alias-symmetric to main September 4, 2025 07:48
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