From 0db442610ba0cb9477b55b61721a78282f4c3c39 Mon Sep 17 00:00:00 2001 From: Tigran Najaryan Date: Thu, 3 Feb 2022 16:01:38 -0500 Subject: [PATCH 1/2] Deprecate LogRecord.Name field The field is removed from the spec: https://github.com/open-telemetry/opentelemetry-specification/pull/2271 I am deprecating it for now, to give some time to existing users to get rid of it (most notably in the Collector). This will be removed March 1, 2022 (after the Collector cleanup). --- CHANGELOG.md | 2 +- opentelemetry/proto/logs/v1/logs.proto | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 0d1a8521d..433aaa1a2 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -19,7 +19,7 @@ Full list of differences found in [this compare](https://github.com/open-telemet ### Removed -* Remove if no changes for this section before release. +* Deprecate LogRecord.Name field. (#357) ## 0.12.0 - 2022-01-19 diff --git a/opentelemetry/proto/logs/v1/logs.proto b/opentelemetry/proto/logs/v1/logs.proto index 73b9c929c..92e9cde30 100644 --- a/opentelemetry/proto/logs/v1/logs.proto +++ b/opentelemetry/proto/logs/v1/logs.proto @@ -127,7 +127,8 @@ message LogRecord { // Short event identifier that does not contain varying parts. Name describes // what happened (e.g. "ProcessStarted"). Recommended to be no longer than 50 // characters. Not guaranteed to be unique in any way. [Optional]. - string name = 4; + // This deprecated field is planned to be removed March 1, 2022. + string name = 4 [deprecated = true]; // A value containing the body of the log record. Can be for example a human-readable // string message (including multi-line) describing the event in a free form or it can From 2dd90a01956f6c669a1da94268e7af0eb23b0468 Mon Sep 17 00:00:00 2001 From: Tigran Najaryan Date: Mon, 14 Feb 2022 08:57:54 -0500 Subject: [PATCH 2/2] Change deprecation notice --- opentelemetry/proto/logs/v1/logs.proto | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/opentelemetry/proto/logs/v1/logs.proto b/opentelemetry/proto/logs/v1/logs.proto index 92e9cde30..dd9cdfbf9 100644 --- a/opentelemetry/proto/logs/v1/logs.proto +++ b/opentelemetry/proto/logs/v1/logs.proto @@ -127,7 +127,8 @@ message LogRecord { // Short event identifier that does not contain varying parts. Name describes // what happened (e.g. "ProcessStarted"). Recommended to be no longer than 50 // characters. Not guaranteed to be unique in any way. [Optional]. - // This deprecated field is planned to be removed March 1, 2022. + // This deprecated field is planned to be removed March 15, 2022. Receivers can + // ignore this field. string name = 4 [deprecated = true]; // A value containing the body of the log record. Can be for example a human-readable