Skip to content

Conversation

@crossoverJie
Copy link
Member

@crossoverJie crossoverJie commented Apr 11, 2025

  • Added attribute server.address, server.port to the http.server.request.duration metric.
  • Added attribute server.address, server.port to the http.server.active_requests metric.
  • Added attribute server.address, server.port to the http.server.request.body.size metric.
  • Added attribute url.scheme to the http.client.request.duration metric.
  • Added attribute url.scheme to the http.client.request.body.size metric.
  • Added attribute url.scheme to the http.client.response.body.size metric.

crossoverJie and others added 19 commits June 14, 2024 15:40
…nstrumentation/api/incubator/semconv/messaging/MessagingMetricsAdvice.java

Co-authored-by: Lauri Tulmin <tulmin@gmail.com>
…nstrumentation/api/incubator/semconv/messaging/MessagingProducerMetrics.java

Co-authored-by: Lauri Tulmin <tulmin@gmail.com>
…nstrumentation/api/incubator/semconv/messaging/MessagingProducerMetricsTest.java

Co-authored-by: Steve Rao <raozihao.rzh@alibaba-inc.com>
…nstrumentation/api/incubator/semconv/messaging/MessagingProducerMetricsTest.java

Co-authored-by: Steve Rao <raozihao.rzh@alibaba-inc.com>
@crossoverJie crossoverJie requested a review from a team as a code owner April 11, 2025 06:42
ServerAttributes.SERVER_ADDRESS,
ServerAttributes.SERVER_PORT));
ServerAttributes.SERVER_PORT,
UrlAttributes.URL_SCHEME));
Copy link
Member

Choose a reason for hiding this comment

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

this is opt-in, so users would need to provide their own metric view to override this

Comment on lines +54 to +55
ServerAttributes.SERVER_ADDRESS,
ServerAttributes.SERVER_PORT));
Copy link
Member

Choose a reason for hiding this comment

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

same here

HttpAttributes.HTTP_REQUEST_METHOD, UrlAttributes.URL_SCHEME));
// https://github.com/open-telemetry/semantic-conventions/blob/v1.32.0/docs/http/http-metrics.md#metric-httpserveractive_requests
HttpAttributes.HTTP_REQUEST_METHOD, UrlAttributes.URL_SCHEME,
ServerAttributes.SERVER_ADDRESS, ServerAttributes.SERVER_PORT));
Copy link
Member

Choose a reason for hiding this comment

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

and here

@crossoverJie crossoverJie requested a review from trask April 23, 2025 10:30
@github-actions
Copy link
Contributor

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 Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants