From 0db442610ba0cb9477b55b61721a78282f4c3c39 Mon Sep 17 00:00:00 2001 From: Tigran Najaryan Date: Thu, 3 Feb 2022 16:01:38 -0500 Subject: [PATCH] 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