Skip to content

Commit 9d8b616

Browse files
dmathieuXSAM
andauthored
Remove prometheus legacy validation warning (#6590)
I have been chatting with @bwplotka, who wouldn't recommend removing the support for the legacy scheme now. Indeed, some system don't support the new scheme yet (such as cortex). So removing this will entirely prevent those users form upgrading their SDK. They are going to remove the deprecation from the legacy method on common, at least until the new scheme can be used in enough systems. --------- Co-authored-by: Sam Xie <[email protected]>
1 parent d146da5 commit 9d8b616

File tree

2 files changed

+1
-9
lines changed

2 files changed

+1
-9
lines changed

CHANGELOG.md

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -8,14 +8,6 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
88

99
## [Unreleased]
1010

11-
> [!WARNING]
12-
> This is the last version to support `model.LegacyValidation` for
13-
> `go.opentelemetry.io/otel/exporters/prometheus`.
14-
> The next version (v0.59.0) will only support the default `model.UTF8Validation`.
15-
>
16-
> See also [Change default validation scheme to UTF8Validation](https://github.com/prometheus/common/pull/724)
17-
> in the prometheus repository.
18-
1911
### Added
2012

2113
- Add exponential histogram support in `go.opentelemetry.io/otel/exporters/prometheus`. (#6421)

exporters/prometheus/config.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ type config struct {
2929

3030
var logDeprecatedLegacyScheme = sync.OnceFunc(func() {
3131
global.Warn(
32-
"prometheus exporter legacy scheme deprecated: support for the legacy NameValidationScheme will be removed in the next release",
32+
"prometheus exporter legacy scheme deprecated: support for the legacy NameValidationScheme will be removed in a future release",
3333
)
3434
})
3535

0 commit comments

Comments
 (0)