From e18299f7b785beab822708c6f62f5c1a39955381 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Robert=20Paj=C4=85k?= Date: Fri, 17 Jan 2025 07:00:28 +0100 Subject: [PATCH] log: Make whole Logs API user-facing (#6167) Reason: https://github.com/open-telemetry/opentelemetry-specification/pull/4352 Prior-art: https://github.com/open-telemetry/opentelemetry-go/pull/6018 Support for `EventName` field will be added in separate PRs. --- CHANGELOG.md | 4 ++++ log/logger.go | 6 ------ 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 8123dcec87c..b61b8bd3a09 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -8,6 +8,10 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm ## [Unreleased] +### Changed + +- Remove the notices from `Logger` to make the whole Logs API user-facing in `go.opentelemetry.io/otel/log`. (#6167) + ### Fixed - Relax minimum Go version to 1.22.0 in various modules. (#6073) diff --git a/log/logger.go b/log/logger.go index 0773a49b608..d7d8831be5b 100644 --- a/log/logger.go +++ b/log/logger.go @@ -28,9 +28,6 @@ type Logger interface { // // Implementations of this method need to be safe for a user to call // concurrently. - // - // Notice: Emit is intended to be used by log bridges. - // Is should not be used for writing instrumentation. Emit(ctx context.Context, record Record) // Enabled returns whether the Logger emits for the given context and @@ -53,9 +50,6 @@ type Logger interface { // // Implementations of this method need to be safe for a user to call // concurrently. - // - // Notice: Enabled is intended to be used by log bridges. - // Is should not be used for writing instrumentation. Enabled(ctx context.Context, param EnabledParameters) bool }