Skip to content

Commit

Permalink
instrumentation/net/http/otelhttp: upgrade semconv to v1.26.0 (open-t…
Browse files Browse the repository at this point in the history
…elemetry#5923)

Bases on the discussion in open-telemetry#5902 , we have came to the conclusion that
there's no new stable attributes introduced in semconv v1.26.0 for
otelhttp. We can directly bump from v1.24.0 to v1.26.0

Changes in this PR:
- bump semconv from v1.24.0 to v1.26.0 for otelhttp
- rename the
`instrumentation/net/http/otelhttp/internal/semconv/v1.24.0.go` to
`instrumentation/net/http/otelhttp/internal/semconv/httpconv.go` to
match with client side naming convention

---------

Co-authored-by: Damien Mathieu <[email protected]>
  • Loading branch information
VinozzZ and dmathieu authored Jul 19, 2024
1 parent 7ce8deb commit 000758e
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
This module provides an OpenTelemetry logging bridge for `go.uber.org/zap`. (#5191)
- The `go.opentelemetry.io/contrib/config` package supports configuring `with_resource_constant_labels` for the prometheus exporter. (#5890)
- Add new runtime metrics to `go.opentelemetry.io/contrib/instrumentation/runtime`, which are still disabled by default. (#5870)
- Support for the `OTEL_HTTP_CLIENT_COMPATIBILITY_MODE=http/dup` environment variable in `go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp` to emit attributes for both the v1.20.0 and v1.24.0 semantic conventions. (#5401)
- Support for the `OTEL_HTTP_CLIENT_COMPATIBILITY_MODE=http/dup` environment variable in `go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp` to emit attributes for both the v1.20.0 and v1.26.0 semantic conventions. (#5401)

### Removed

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import (
"strings"

"go.opentelemetry.io/otel/attribute"
semconvNew "go.opentelemetry.io/otel/semconv/v1.24.0"
semconvNew "go.opentelemetry.io/otel/semconv/v1.26.0"
)

type newHTTPServer struct{}
Expand Down
2 changes: 1 addition & 1 deletion instrumentation/net/http/otelhttp/internal/semconv/util.go
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ import (
"strings"

"go.opentelemetry.io/otel/attribute"
semconvNew "go.opentelemetry.io/otel/semconv/v1.24.0"
semconvNew "go.opentelemetry.io/otel/semconv/v1.26.0"
)

// splitHostPort splits a network address hostport of the form "host",
Expand Down

0 comments on commit 000758e

Please sign in to comment.