Skip to content

Rename http.user_agent to user_agent.original#3190

Merged
carlosalberto merged 15 commits intoopen-telemetry:mainfrom
trask:user-agent
Mar 2, 2023
Merged

Rename http.user_agent to user_agent.original#3190
carlosalberto merged 15 commits intoopen-telemetry:mainfrom
trask:user-agent

Conversation

@trask
Copy link
Copy Markdown
Member

@trask trask commented Feb 9, 2023

Fixes #3180

Changes

Renames http.user_agent to user_agent.original.

@trask trask force-pushed the user-agent branch 2 times, most recently from 87e986c to 00cbb2f Compare February 9, 2023 00:38
@trask trask changed the title Rename http.user_agent to user_agent.full Rename http.user_agent to user_agent.original Feb 9, 2023
@trask trask added area:semantic-conventions Related to semantic conventions semconv:HTTP spec:trace Related to the specification/trace directory labels Feb 9, 2023
@trask trask marked this pull request as ready for review February 13, 2023 23:14
@trask trask requested review from a team February 13, 2023 23:14
@arminru arminru added the area:client RUM and other client instrumentation issues label Feb 14, 2023
@MSNev
Copy link
Copy Markdown
Contributor

MSNev commented Feb 14, 2023

Do we want to include removal / renaming of browser.user_agent as part of this or as a separate PR?

@trask
Copy link
Copy Markdown
Member Author

trask commented Feb 14, 2023

Do we want to include removal / renaming of browser.user_agent as part of this or as a separate PR?

@MSNev I added to this PR, ptal 👍

@tigrannajaryan
Copy link
Copy Markdown
Member

Fix submitted to build tools: open-telemetry/build-tools#138

tigrannajaryan added a commit to tigrannajaryan/opentelemetry-specification that referenced this pull request Feb 27, 2023
build-tools 1.16.0 was just released and is necessary to unblock
open-telemetry#3190
@tigrannajaryan
Copy link
Copy Markdown
Member

Here is the PR to update to latest build-tools which includes the fix to schema check tool: #3259

Once that is merged this PR needs to be rebased.

tigrannajaryan added a commit that referenced this pull request Feb 27, 2023
build-tools 1.16.0 was just released and is necessary to unblock
#3190
@tigrannajaryan
Copy link
Copy Markdown
Member

This can be rebased now and the schema check should pass.

@trask
Copy link
Copy Markdown
Member Author

trask commented Feb 27, 2023

@tigrannajaryan thx for fixing this, schema check is passing now. can you remove your "request changes"? thx!

@tigrannajaryan tigrannajaryan dismissed their stale review February 27, 2023 23:54

schema check fixed

jsuereth pushed a commit to jsuereth/opentelemetry-specification that referenced this pull request Feb 28, 2023
build-tools 1.16.0 was just released and is necessary to unblock
open-telemetry#3190
@carlosalberto
Copy link
Copy Markdown
Contributor

@trask Conflicts need to be solved but otherwise (it sounds like) we are ready to merge this.

@carlosalberto
Copy link
Copy Markdown
Contributor

Merging after letting this PR settle and get some more attention.

@carlosalberto carlosalberto merged commit 0e19398 into open-telemetry:main Mar 2, 2023
jsuereth pushed a commit to jsuereth/otel-semconv-test that referenced this pull request Apr 19, 2023
build-tools 1.16.0 was just released and is necessary to unblock
open-telemetry/opentelemetry-specification#3190
@trask trask deleted the user-agent branch April 26, 2023 23:06
jsuereth pushed a commit to open-telemetry/semantic-conventions that referenced this pull request May 11, 2023
build-tools 1.16.0 was just released and is necessary to unblock
open-telemetry/opentelemetry-specification#3190
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
build-tools 1.16.0 was just released and is necessary to unblock
open-telemetry#3190
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
Fixes open-telemetry#3180

## Changes

Renames `http.user_agent` to `user_agent.original`.
apollo0202 pushed a commit to apollo0202/conventions-semantic that referenced this pull request Mar 1, 2025
build-tools 1.16.0 was just released and is necessary to unblock
open-telemetry/opentelemetry-specification#3190
schmikei pushed a commit to schmikei/opentelemetry-specification that referenced this pull request Apr 17, 2025
Fixes open-telemetry#3180

## Changes

Renames `http.user_agent` to `user_agent.original`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:client RUM and other client instrumentation issues area:semantic-conventions Related to semantic conventions semconv:HTTP spec:trace Related to the specification/trace directory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Generalize the http.user_agent attribute to user_agent.*

10 participants