fix(otel): use http/grpc status over span status (#6879)

Prioritize http/grpc status over span status when we generate a status for a sentry span. This is because they are more thorough then OpenTelemetry's span status values.

Author: AbhiPrasad

packages/opentelemetry-node/src/utils/map-otel-status.ts

@@ -46,16 +46,6 @@ const canonicalCodesGrpcMap: Record<string, SentryStatus> = {
 export function mapOtelStatus(otelSpan: OtelSpan): SentryStatus {
   const { status, attributes } = otelSpan;
 
-  const statusCode = status.code;
-
-  if (statusCode < 0 || statusCode > 2) {
-    return 'unknown_error';
-  }
-
-  if (statusCode === 0 || statusCode === 1) {
-    return 'ok';
-  }
-
   const httpCode = attributes[SemanticAttributes.HTTP_STATUS_CODE];
   const grpcCode = attributes[SemanticAttributes.RPC_GRPC_STATUS_CODE];
 
@@ -74,5 +64,10 @@ export function mapOtelStatus(otelSpan: OtelSpan): SentryStatus {
     }
   }
 
+  const statusCode = status.code;
+  if (statusCode === 0 || statusCode === 1) {
+    return 'ok';
+  }
+
   return 'unknown_error';
 }