[Instrumentation.GrpcCore] Remove usage of "otel.status_code" and "otel.status_description"

src/OpenTelemetry.Instrumentation.GrpcCore/CHANGELOG.md

@@ -10,6 +10,9 @@
   ([#1624](https://github.com/open-telemetry/opentelemetry-dotnet-contrib/pull/1624))
 * Update `OpenTelemetry.Api` to `1.8.1`.
   ([#1668](https://github.com/open-telemetry/opentelemetry-dotnet-contrib/pull/1668))
+* Use `Activity.Status` and `Activity.StatusDescription`
+  instead of `otel.status_code` and `otel.status_description` tags.
+  ([#1799](https://github.com/open-telemetry/opentelemetry-dotnet-contrib/pull/1799))
 
 ## 1.0.0-beta.5
 

src/OpenTelemetry.Instrumentation.GrpcCore/RpcScope.cs

@@ -7,7 +7,6 @@
 using System.Threading;
 using Google.Protobuf;
 using Grpc.Core;
-using OpenTelemetry.Trace;
 
 namespace OpenTelemetry.Instrumentation.GrpcCore;
 
@@ -104,7 +103,7 @@
         }
 
         // The overall Span status should remain unset however the grpc status code attribute is required
         this.StopActivity((int)Grpc.Core.StatusCode.OK);
     }
 
     /// <summary>
@@ -118,7 +117,7 @@
             return;
         }
 
         var grpcStatusCode = Grpc.Core.StatusCode.Unknown;
         var description = exception.Message;
 
         if (exception is RpcException rpcException)
@@ -139,7 +138,7 @@
         }
 
         // If not already completed this will mark the Activity as cancelled.
         this.StopActivity((int)Grpc.Core.StatusCode.Cancelled);
     }
 
     /// <summary>
@@ -184,7 +183,7 @@
             this.activity.SetTag(SemanticConventions.AttributeRpcGrpcStatusCode, statusCode);
             if (statusDescription != null)
             {
-                this.activity.SetStatus(Trace.Status.Error.WithDescription(statusDescription));
+                this.activity.SetStatus(ActivityStatusCode.Error, statusDescription);
             }
 
             this.activity.Stop();

src/OpenTelemetry.Instrumentation.GrpcCore/SemanticConventions.cs

@@ -17,8 +17,6 @@ internal static class SemanticConventions
     public const string AttributeMessageID = "message.id";
     public const string AttributeMessageCompressedSize = "message.compressed_size";
     public const string AttributeMessageUncompressedSize = "message.uncompressed_size";
-    public const string AttributeOtelStatusCode = "otel.status_code";
-    public const string AttributeOtelStatusDescription = "otel.status_description";
 
     // Used for unit testing only.
     internal const string AttributeActivityIdentifier = "activityidentifier";

test/OpenTelemetry.Instrumentation.GrpcCore.Tests/GrpcCoreClientInterceptorTests.cs

@@ -327,7 +327,7 @@ static void ValidateNewTagOnActivity(InterceptorActivityListener listener)
         // Cancelled is not an error.
         if (expectedStatusCode != StatusCode.OK && expectedStatusCode != StatusCode.Cancelled)
         {
-            Assert.Contains(activity.TagObjects, t => t.Key == SemanticConventions.AttributeOtelStatusCode && (string)t.Value == "ERROR");
+            Assert.Equal(ActivityStatusCode.Error, activity.Status);
         }
 
         if (recordedMessages)
@@ -490,7 +490,7 @@ private static async Task TestHandlerSuccess(Func<Foobar.FoobarClient, Metadata,
 
         if (validateErrorDescription)
         {
-            Assert.Contains(activity.TagObjects, t => t.Key == SemanticConventions.AttributeOtelStatusDescription && ((string)t.Value).Contains("fubar"));
+            Assert.Contains("fubar", activity.StatusDescription);
         }
     }