Skip to content

Commit

Permalink
Warn instead of Info log instrument conflict (#4202)
Browse files Browse the repository at this point in the history
* Warn instead of Info log instrument conflict

* Add change to changelog
  • Loading branch information
MrAlias committed Jun 6, 2023
1 parent 135c64f commit ce46eb5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
### Changed

- Starting from `v1.21.0` of semantic conventions, `go.opentelemetry.io/otel/semconv/{version}/httpconv` and `go.opentelemetry.io/otel/semconv/{version}/netconv` packages will no longer be published. (#4145)
- Log duplicate instrument conflict at a warning level instead of info in `go.opentelemetry.io/otel/sdk/metric`. (#4202)

## [1.16.0/0.39.0] 2023-05-18

Expand Down
2 changes: 1 addition & 1 deletion sdk/metric/pipeline.go
Original file line number Diff line number Diff line change
Expand Up @@ -354,7 +354,7 @@ func (i *inserter[N]) logConflict(id streamID) {
return
}

global.Info(
global.Warn(
"duplicate metric stream definitions",
"names", fmt.Sprintf("%q, %q", existing.Name, id.Name),
"descriptions", fmt.Sprintf("%q, %q", existing.Description, id.Description),
Expand Down

0 comments on commit ce46eb5

Please sign in to comment.