Align insrumentation TLS config with collector (#3358)

Signed-off-by: Pavol Loffay <[email protected]>

Author: pavolloffay

.chloggen/inst-tls.yaml

@@ -23,11 +23,11 @@ subtext: |
         secretName: otel-tls-certs
         configMapName: otel-ca-bundle
         # otel-ca-bundle
-        ca: ca.crt
+        ca_file: ca.crt
         # present in otel-tls-certs
-        cert: tls.crt 
+        cert_file: tls.crt 
         # present in otel-tls-certs
-        key: tls.key
+        key_file: tls.key
   ```
   
   * Propagating secrets across namespaces can be done with https://github.com/EmberStack/kubernetes-reflector or https://github.com/zakkg3/ClusterSecret

apis/v1alpha1/instrumentation_types.go

@@ -121,13 +121,13 @@ type TLS struct {
 	// CA defines the key of certificate (e.g. ca.crt) in the configmap map, secret or absolute path to a certificate.
 	// The absolute path can be used when certificate is already present on the workload filesystem e.g.
 	// /var/run/secrets/kubernetes.io/serviceaccount/service-ca.crt
-	CA string `json:"ca,omitempty"`
+	CA string `json:"ca_file,omitempty"`
 	// Cert defines the key (e.g. tls.crt) of the client certificate in the secret or absolute path to a certificate.
 	// The absolute path can be used when certificate is already present on the workload filesystem.
-	Cert string `json:"cert,omitempty"`
+	Cert string `json:"cert_file,omitempty"`
 	// Key defines a key (e.g. tls.key) of the private key in the secret or absolute path to a certificate.
 	// The absolute path can be used when certificate is already present on the workload filesystem.
-	Key string `json:"key,omitempty"`
+	Key string `json:"key_file,omitempty"`
 }
 
 // Sampler defines sampling configuration.

bundle/community/manifests/opentelemetry-operator.clusterserviceversion.yaml

@@ -99,7 +99,7 @@ metadata:
     categories: Logging & Tracing,Monitoring
     certified: "false"
     containerImage: ghcr.io/open-telemetry/opentelemetry-operator/opentelemetry-operator
-    createdAt: "2024-10-10T15:31:51Z"
+    createdAt: "2024-10-16T10:10:50Z"
     description: Provides the OpenTelemetry components, including the Collector
     operators.operatorframework.io/builder: operator-sdk-v1.29.0
     operators.operatorframework.io/project_layout: go.kubebuilder.io/v3

bundle/community/manifests/opentelemetry.io_instrumentations.yaml

@@ -411,13 +411,13 @@ spec:
                     type: string
                   tls:
                     properties:
-                      ca:
+                      ca_file:
                         type: string
-                      cert:
+                      cert_file:
                         type: string
                       configMapName:
                         type: string
-                      key:
+                      key_file:
                         type: string
                       secretName:
                         type: string

bundle/openshift/manifests/opentelemetry-operator.clusterserviceversion.yaml

@@ -99,7 +99,7 @@ metadata:
     categories: Logging & Tracing,Monitoring
     certified: "false"
     containerImage: ghcr.io/open-telemetry/opentelemetry-operator/opentelemetry-operator
-    createdAt: "2024-10-10T15:31:51Z"
+    createdAt: "2024-10-16T10:10:50Z"
     description: Provides the OpenTelemetry components, including the Collector
     operators.operatorframework.io/builder: operator-sdk-v1.29.0
     operators.operatorframework.io/project_layout: go.kubebuilder.io/v3

bundle/openshift/manifests/opentelemetry.io_instrumentations.yaml

@@ -411,13 +411,13 @@ spec:
                     type: string
                   tls:
                     properties:
-                      ca:
+                      ca_file:
                         type: string
-                      cert:
+                      cert_file:
                         type: string
                       configMapName:
                         type: string
-                      key:
+                      key_file:
                         type: string
                       secretName:
                         type: string

config/crd/bases/opentelemetry.io_instrumentations.yaml

@@ -409,13 +409,13 @@ spec:
                     type: string
                   tls:
                     properties:
-                      ca:
+                      ca_file:
                         type: string
-                      cert:
+                      cert_file:
                         type: string
                       configMapName:
                         type: string
-                      key:
+                      key_file:
                         type: string
                       secretName:
                         type: string

docs/api.md

@@ -1659,7 +1659,7 @@ TLS needs to be enabled by specifying https:// scheme in the Endpoint.
         </tr>
     </thead>
     <tbody><tr>
-        <td><b>ca</b></td>
+        <td><b>ca_file</b></td>
         <td>string</td>
         <td>
           CA defines the key of certificate (e.g. ca.crt) in the configmap map, secret or absolute path to a certificate.
@@ -1668,7 +1668,7 @@ The absolute path can be used when certificate is already present on the workloa
         </td>
         <td>false</td>
       </tr><tr>
-        <td><b>cert</b></td>
+        <td><b>cert_file</b></td>
         <td>string</td>
         <td>
           Cert defines the key (e.g. tls.crt) of the client certificate in the secret or absolute path to a certificate.
@@ -1684,7 +1684,7 @@ used from the secret defined in SecretName.<br/>
         </td>
         <td>false</td>
       </tr><tr>
-        <td><b>key</b></td>
+        <td><b>key_file</b></td>
         <td>string</td>
         <td>
           Key defines a key (e.g. tls.key) of the private key in the secret or absolute path to a certificate.

tests/e2e-instrumentation/instrumentation-java-tls/00-install-instrumentation.yaml

@@ -8,9 +8,9 @@ spec:
     tls:
       secretName: client-certs
       configMapName: ca
-      ca: ca.crt
-      cert: tls.crt
-      key: tls.key
+      ca_file: ca.crt
+      cert_file: tls.crt
+      key_file: tls.key
   propagators:
     - tracecontext
     - baggage