adopt otel exporter names

Signed-off-by: Benedikt Bongartz <bongartz@klimlive.de>

Author: frzifus

.chloggen/otlp-exporter-migration.yaml

@@ -0,0 +1,22 @@
+# Use this changelog template to create an entry for release notes.
+
+# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
+change_type: enhancement
+
+# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver)
+component: operator
+
+# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`).
+note: Add automatic migration and deprecation warnings for OTLP exporter name changes (otlp→otlp_grpc, otlphttp→otlp_http)
+
+# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists.
+issues: [4689]
+
+# (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: |
+  The operator now automatically renames deprecated otlp and otlphttp exporters to otlp_grpc and otlp_http when upgrading to collector v0.145.0+.
+  Deprecation warnings are emitted for existing configs using old names to help users migrate their configurations.
+  This change aligns with the OpenTelemetry Collector's deprecation of generic OTLP exporter names in favor of explicit protocol names.
+

apis/v1alpha1/convert_test.go

@@ -37,7 +37,7 @@ service:
     traces:
       receivers: [otlp]
       processors: [resourcedetection]
-      exporters: [otlp]
+      exporters: [otlp_grpc]
 `
 
 func Test_tov1beta1_config(t *testing.T) {

apis/v1beta1/config.go

@@ -9,6 +9,7 @@ import (
 	"fmt"
 	"math"
 	"sort"
+	"strings"
 
 	"dario.cat/mergo"
 	"github.com/go-logr/logr"
@@ -323,6 +324,26 @@ func (c *Config) applyDefaultForComponentKinds(logger logr.Logger, componentKind
 		}
 	}
 
+	// Check for deprecated OTLP exporter names and emit warnings
+	if c.Exporters.Object != nil {
+		for exporterName := range c.Exporters.Object {
+			if exporterName == "otlp" || strings.HasPrefix(exporterName, "otlp/") {
+				events = append(events, EventInfo{
+					Type:    corev1.EventTypeWarning,
+					Reason:  "DeprecatedExporter",
+					Message: fmt.Sprintf("Exporter '%s' uses deprecated name 'otlp'. Use 'otlp_grpc' instead. Config will be auto-migrated on next operator upgrade.", exporterName),
+				})
+			}
+			if exporterName == "otlphttp" || strings.HasPrefix(exporterName, "otlphttp/") {
+				events = append(events, EventInfo{
+					Type:    corev1.EventTypeWarning,
+					Reason:  "DeprecatedExporter",
+					Message: fmt.Sprintf("Exporter '%s' uses deprecated name 'otlphttp'. Use 'otlp_http' instead. Config will be auto-migrated on next operator upgrade.", exporterName),
+				})
+			}
+		}
+	}
+
 	return events, nil
 }
 

config/samples/core_v1alpha1_opentelemetrycollector.yaml

@@ -5,7 +5,7 @@ metadata:
 spec:
   config: |
     receivers:
-      otlp:
+      otlp_grpc:
         protocols: 
           grpc: {}
           http: {}

config/samples/core_v1beta1_opentelemetrycollector.yaml

@@ -5,7 +5,7 @@ metadata:
 spec:
   config:
     receivers:
-      otlp:
+      otlp_grpc:
         protocols:
           grpc: {}
           http: {}

config/samples/exporter-with-tls.yaml

@@ -34,7 +34,7 @@ spec:
       batch: {}
 
     exporters:
-      otlp:
+      otlp_grpc:
         endpoint: "simplest-collector-headless.default.svc:14250"
         tls:
           ca_file: "/etc/pki/ca-trust/source/service-ca/service-ca.crt"
@@ -45,4 +45,4 @@ spec:
         traces:
           receivers: [jaeger]
           processors: []
-          exporters: [otlp]
+          exporters: [otlp_grpc]

config/samples/sidecar.yaml

@@ -6,7 +6,7 @@ spec:
   mode: sidecar
   config: |
     receivers:
-      otlp:
+      otlp_grpc:
         protocols:
           grpc: {}
           http: {}

internal/controllers/testdata/ingress_testdata.yaml

@@ -1,10 +1,10 @@
 ---
 receivers:
-  otlp:
+  otlp_grpc:
     protocols:
       grpc:
         endpoint: 0.0.0.0:12345
-  otlp/test:
+  otlp_grpc/test:
     protocols:
       grpc:
         endpoint: 0.0.0.0:12346
@@ -13,5 +13,5 @@ exporters:
 service:
   pipelines:
     traces:
-      receivers: [otlp, otlp/test]
+      receivers: [otlp_grpc, otlp/test]
       exporters: [debug]

internal/controllers/testdata/otlp_test.yaml

@@ -1,11 +1,11 @@
 receivers:
-  otlp:
+  otlp_grpc:
     protocols:
       grpc:
       http:
 processors:
 exporters:
-  otlp:
+  otlp_grpc:
     endpoint: jaeger-allinone-collector-headless.chainsaw-otlp-metrics.svc:4317
     tls:
       insecure: true
@@ -17,7 +17,7 @@ service:
   pipelines:
     traces:
       receivers: [otlp]
-      exporters: [otlp]
+      exporters: [otlp_grpc]
     metrics:
       receivers: [otlp]
       exporters: [prometheus]

internal/manifests/collector/testdata/ingress_testdata.yaml

@@ -1,10 +1,10 @@
 ---
 receivers:
-  otlp:
+  otlp_grpc:
     protocols:
       grpc:
         endpoint: 0.0.0.0:12345
-  otlp/test:
+  otlp_grpc/test:
     protocols:
       grpc:
         endpoint: 0.0.0.0:98765
@@ -15,5 +15,5 @@ exporters:
 service:
   pipelines:
     traces:
-      receivers: [otlp, otlp/test]
+      receivers: [otlp_grpc, otlp/test]
       exporters: [nop]