fix: change semconv of container name (#1575)

Co-authored-by: Tamir David <tamirdavid@Tamirs-MacBook-Pro.local>
odigos-io · Oct 8, 2024 · 8b16bd0 · 8b16bd0
1 parent 880ef68
commit 8b16bd0
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/opampserver/pkg/server/handlers.go b/opampserver/pkg/server/handlers.go
@@ -215,7 +215,7 @@ func extractOpampAgentAttributes(agentDescription *protobufs.AgentDescription) o
 		switch attr.Key {
 		case string(semconv.TelemetrySDKLanguageKey):
 			result.ProgrammingLanguage = attr.Value.GetStringValue()
-		case string(semconv.ContainerNameKey):
+		case string(semconv.K8SContainerNameKey):
 			result.ContainerName = attr.Value.GetStringValue()
 		case string(semconv.K8SPodNameKey):
 			result.PodName = attr.Value.GetStringValue()
@@ -250,8 +250,8 @@ func resolveFromDirectAttributes(ctx context.Context, attrs opampAgentAttributes
 		}
 	}
 
-	serviceName := podInfoResolver.ResolveServiceName(ctx, attrs.PodName, workloadName, &di.ContainerDetails{
-		PodName: attrs.PodName,
+	serviceName := podInfoResolver.ResolveServiceName(ctx, workloadName, string(workloadKind), &di.ContainerDetails{
+		PodNamespace: attrs.Namespace,
 	})
 
 	k8sAttributes := &di.K8sResourceAttributes{