Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename server.socket.address to network.peer.address #93255

Merged
merged 2 commits into from
Oct 10, 2023

Conversation

antonfirsov
Copy link
Member

Address #92956:

Keep .NET metrics in sync with OTel by implementing a naming change from open-telemetry/semantic-conventions#342. An 8.0 backport is planned to avoid future breaking changes.

@ghost
Copy link

ghost commented Oct 9, 2023

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

Issue Details

Address #92956:

Keep .NET metrics in sync with OTel by implementing a naming change from open-telemetry/semantic-conventions#342. An 8.0 backport is planned to avoid future breaking changes.

Author: antonfirsov
Assignees: -
Labels:

area-System.Net.Http

Milestone: -

@karelz karelz added this to the 9.0.0 milestone Oct 10, 2023
@antonfirsov antonfirsov merged commit 1a14851 into dotnet:main Oct 10, 2023
109 checks passed
antonfirsov added a commit to antonfirsov/runtime that referenced this pull request Oct 12, 2023
carlossanlop pushed a commit that referenced this pull request Oct 13, 2023
…istency (#93414)

* Rename `server.socket.address` to `network.peer.address` (#93255)

* Report Exception.FullName for http.client.request.duration/error.type (#93322)

* Adjust the DNS lookup duration metric (#93254)
@ghost ghost locked as resolved and limited conversation to collaborators Nov 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants