update recommended attributes to match semconv

Author: zeitlinger

.chloggen/recommedded-resource-attributes.yaml

@@ -0,0 +1,22 @@
+# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
+change_type: breaking
+
+# The name of the component, or a single word describing the area of concern, (e.g. collector, target allocator, auto-instrumentation, opamp, github action)
+component: auto-instrumentation
+
+# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`).
+note: Update recommended resource attributes to match the [semantic conventions](https://github.com/open-telemetry/semantic-conventions/blob/main/docs/non-normative/k8s-attributes.md)
+
+# One or more tracking issues related to the change
+issues: []
+
+# (Optional) One or more lines of additional information to render under the primary note.
+# These lines will be padded with 2 spaces and then inserted directly into the document.
+# Use pipe (|) for multiline entries.
+subtext: |
+  This change updates the recommended resource attributes to match the [semantic conventions](https://github.com/open-telemetry/semantic-conventions/blob/main/docs/non-normative/k8s-attributes.md).
+  The following attributes have been updated:
+  - `service.version` now uses the docker image digest in addition to the tag
+  - the well-known label `app.kubernetes.io/part-of` has been removed
+  - the well-known label `app.kubernetes.io/instance` has been added (translates to `service.name`)
+  

README.md

@@ -752,12 +752,12 @@ spec:
 
 ### Configure resource attributes with labels
 
-You can also use common labels to set resource attributes.
+You can also use common labels to set resource attributes (first entry wins).
 
 The following labels are supported:
+- `app.kubernetes.io/instance` becomes `service.name`
 - `app.kubernetes.io/name` becomes `service.name`
 - `app.kubernetes.io/version` becomes `service.version`
-- `app.kubernetes.io/part-of` becomes `service.namespace`
 
 ```yaml
 apiVersion: v1

apis/v1alpha1/instrumentation_types.go

@@ -138,9 +138,10 @@ type Sampler struct {
 // Defaults defines default values for the instrumentation.
 type Defaults struct {
 	// UseLabelsForResourceAttributes defines whether to use common labels for resource attributes:
+	// Note: first entry wins:
+	//   - `app.kubernetes.io/instance` becomes `service.name`
 	//   - `app.kubernetes.io/name` becomes `service.name`
 	//   - `app.kubernetes.io/version` becomes `service.version`
-	//   - `app.kubernetes.io/part-of` becomes `service.namespace`
 	UseLabelsForResourceAttributes bool `json:"useLabelsForResourceAttributes,omitempty"`
 }
 

pkg/constants/env.go

@@ -24,10 +24,6 @@ const (
 	AnnotationDefaultAutoInstrumentationApacheHttpd = InstrumentationPrefix + "default-auto-instrumentation-apache-httpd-image"
 	AnnotationDefaultAutoInstrumentationNginx       = InstrumentationPrefix + "default-auto-instrumentation-nginx-image"
 
-	LabelAppName    = "app.kubernetes.io/name"
-	LabelAppVersion = "app.kubernetes.io/version"
-	LabelAppPartOf  = "app.kubernetes.io/part-of"
-
 	LabelTargetAllocator              = "opentelemetry.io/target-allocator"
 	ResourceAttributeAnnotationPrefix = "resource.opentelemetry.io/"
 
@@ -51,3 +47,11 @@ const (
 	TACollectorTLSKeyFileName  = "tls.key"
 	TACollectorTLSCertFileName = "tls.crt"
 )
+
+var (
+	LabelAppName = []string{
+		"app.kubernetes.io/instance",
+		"app.kubernetes.io/name",
+	}
+	LabelAppVersion = []string{"app.kubernetes.io/version"}
+)

pkg/instrumentation/sdk.go

@@ -431,13 +431,15 @@ func chooseServiceName(pod corev1.Pod, useLabelsForResourceAttributes bool, reso
 // The precedence is as follows:
 // 1. annotation with key resource.opentelemetry.io/<resource>.
 // 2. label with key labelKey.
-func chooseLabelOrAnnotation(pod corev1.Pod, useLabelsForResourceAttributes bool, resource attribute.Key, labelKey string) string {
+func chooseLabelOrAnnotation(pod corev1.Pod, useLabelsForResourceAttributes bool, resource attribute.Key, labelKeys []string) string {
 	if v := pod.GetAnnotations()[(constants.ResourceAttributeAnnotationPrefix + string(resource))]; v != "" {
 		return v
 	}
 	if useLabelsForResourceAttributes {
-		if v := pod.GetLabels()[labelKey]; v != "" {
-			return v
+		for _, labelKey := range labelKeys {
+			if v := pod.GetLabels()[labelKey]; v != "" {
+				return v
+			}
 		}
 	}
 	return ""
@@ -472,7 +474,7 @@ func createServiceInstanceId(pod corev1.Pod, namespaceName, podName, containerNa
 	// which violates the uniqueness requirement of service instance id -
 	// see https://opentelemetry.io/docs/specs/semconv/resource/#service-experimental.
 	// We still allow the user to set the service instance id via annotation, because this is explicitly set by the user.
-	serviceInstanceId := chooseLabelOrAnnotation(pod, false, semconv.ServiceInstanceIDKey, "")
+	serviceInstanceId := chooseLabelOrAnnotation(pod, false, semconv.ServiceInstanceIDKey, nil)
 	if serviceInstanceId != "" {
 		return serviceInstanceId
 	}
@@ -539,7 +541,7 @@ func (i *sdkInjector) createResourceMap(ctx context.Context, otelinst v1alpha1.I
 			}
 		}
 	}
-	partOf := chooseLabelOrAnnotation(pod, useLabelsForResourceAttributes, semconv.ServiceNamespaceKey, constants.LabelAppPartOf)
+	partOf := chooseLabelOrAnnotation(pod, useLabelsForResourceAttributes, semconv.ServiceNamespaceKey, nil)
 	if partOf != "" && !existingRes[string(semconv.ServiceNamespaceKey)] {
 		res[string(semconv.ServiceNamespaceKey)] = partOf
 	}