Closed trajan0x closed 1 week ago
The recent updates focus on upgrading the version of the semconv
package from v1.7.0
to v1.25.0
to leverage new features and standards. Key changes include replacing HTTPServerNameKey
with ServiceNameKey
and adjusting HTTP status code handling in middleware, aligning with the updated OpenTelemetry semantic conventions.
Files | Change Summary |
---|---|
core/metrics/instrumentation/otelginmetrics/config.go |
Updated semconv package from v1.7.0 to v1.25.0 , replaced HTTPServerNameKey with ServiceNameKey . |
core/metrics/instrumentation/otelginmetrics/middleware.go |
Updated semconv package from v1.7.0 to v1.25.0 , replaced HTTPAttributesFromHTTPStatusCode with HTTPResponseStatusCodeKey.Int . |
π° A shift in keys and codes anew, Metrics now with clearer view. Updated paths, from old to bright, Service names shine in the light. With versions high and status keen, Our tracing now is twice as lean. π
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Attention: Patch coverage is 0%
with 2 lines
in your changes missing coverage. Please review.
Project coverage is 25.54179%. Comparing base (
4448759
) to head (50b6e68
).
Files | Patch % | Lines |
---|---|---|
...e/metrics/instrumentation/otelginmetrics/config.go | 0.00000% | 1 Missing :warning: |
...trics/instrumentation/otelginmetrics/middleware.go | 0.00000% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
Summary by CodeRabbit