remove mtls config option from v1alpha1 as it's going to be removed shortly

Signed-off-by: Praful Khanduri <holiodin@gmail.com>

Author: Horiodino

.github/workflows/e2e-reusable.yaml

@@ -96,7 +96,7 @@ jobs:
         - group: e2e-metadata-filters
           setup: "add-operator-arg OPERATOR_ARG='--annotations-filter=.*filter.out --annotations-filter=config.*.gke.io.* --labels-filter=.*filter.out' prepare-e2e"
         - group: e2e-ta-collector-mtls
-          setup: "add-operator-arg OPERATOR_ARG='--feature-gates=operator.targetallocator.mtls' add-certmanager-permissions prepare-e2e"
+          setup: "add-certmanager-permissions prepare-e2e"
         - group: e2e-automatic-rbac
           setup: "add-rbac-permissions-to-operator prepare-e2e"
 

apis/v1alpha1/convert.go

@@ -174,13 +174,6 @@ func tov1beta1Ports(in []PortsSpec) []v1beta1.PortsSpec {
 }
 
 func tov1beta1TA(in OpenTelemetryTargetAllocator) v1beta1.TargetAllocatorEmbedded {
-	var mtls *v1beta1.TargetAllocatorMTLS
-	if in.Mtls != nil {
-		mtls = &v1beta1.TargetAllocatorMTLS{
-			Enabled: in.Mtls.Enabled,
-		}
-	}
-
 	return v1beta1.TargetAllocatorEmbedded{
 		Replicas:           in.Replicas,
 		NodeSelector:       in.NodeSelector,
@@ -215,7 +208,6 @@ func tov1beta1TA(in OpenTelemetryTargetAllocator) v1beta1.TargetAllocatorEmbedde
 			},
 		},
 		PodDisruptionBudget: tov1beta1PodDisruptionBudget(in.PodDisruptionBudget),
-		Mtls:                mtls,
 	}
 }
 
@@ -453,12 +445,6 @@ func tov1alpha1TA(in v1beta1.TargetAllocatorEmbedded) OpenTelemetryTargetAllocat
 	if in.PrometheusCR.ServiceMonitorSelector != nil {
 		serviceMonitorSelector = in.PrometheusCR.ServiceMonitorSelector.MatchLabels
 	}
-	var mtls *TargetAllocatorMTLS
-	if in.Mtls != nil {
-		mtls = &TargetAllocatorMTLS{
-			Enabled: in.Mtls.Enabled,
-		}
-	}
 
 	return OpenTelemetryTargetAllocator{
 		Replicas:           in.Replicas,
@@ -489,7 +475,6 @@ func tov1alpha1TA(in v1beta1.TargetAllocatorEmbedded) OpenTelemetryTargetAllocat
 			},
 		},
 		PodDisruptionBudget: tov1alpha1PodDisruptionBudget(in.PodDisruptionBudget),
-		Mtls:                mtls,
 	}
 }
 

apis/v1alpha1/opentelemetrycollector_types.go

@@ -381,9 +381,6 @@ type OpenTelemetryTargetAllocator struct {
 	//
 	// +optional
 	PodDisruptionBudget *PodDisruptionBudgetSpec `json:"podDisruptionBudget,omitempty"`
-
-	// Mtls defines the mTLS configuration for the target allocator. If enabled, the target allocator will communicate with the collector over mTLS.
-	Mtls *TargetAllocatorMTLS `json:"mtls,omitempty"`
 }
 
 type OpenTelemetryTargetAllocatorPrometheusCR struct {
@@ -616,9 +613,3 @@ type ConfigMapsSpec struct {
 	Name      string `json:"name"`
 	MountPath string `json:"mountpath"`
 }
-
-type TargetAllocatorMTLS struct {
-	// Enabled indicates whether to enable mTLS between the target allocator and the collector.
-	// +optional
-	Enabled bool `json:"enabled,omitempty"`
-}

apis/v1alpha1/targetallocator_types.go

@@ -93,9 +93,6 @@ type TargetAllocatorSpec struct {
 	// +kubebuilder:default:="30s"
 	// +kubebuilder:validation:Format:=duration
 	CollectorNotReadyGracePeriod *metav1.Duration `json:"collectorNotReadyGracePeriod,omitempty"`
-	// Mtls defines the mTLS configuration for the target allocator. If enabled, the target allocator will communicate with the collector over mTLS.
-	// +optional
-	Mtls *TargetAllocatorMTLS `json:"mtls,omitempty"`
 	// LivenessProbe defines the liveness probe configuration for the Target Allocator container.
 	// +optional
 	LivenessProbe *corev1.Probe `json:"livenessProbe,omitempty"`

apis/v1alpha1/zz_generated.deepcopy.go

@@ -3255,11 +3255,6 @@ spec:
                     type: string
                   image:
                     type: string
-                  mtls:
-                    properties:
-                      enabled:
-                        type: boolean
-                    type: object
                   nodeSelector:
                     additionalProperties:
                       type: string

bundle/community/manifests/opentelemetry.io_opentelemetrycollectors.yaml

@@ -2299,11 +2299,6 @@ spec:
                 - managed
                 - unmanaged
                 type: string
-              mtls:
-                properties:
-                  enabled:
-                    type: boolean
-                type: object
               networkPolicy:
                 properties:
                   enabled:

bundle/community/manifests/opentelemetry.io_targetallocators.yaml

@@ -3254,11 +3254,6 @@ spec:
                     type: string
                   image:
                     type: string
-                  mtls:
-                    properties:
-                      enabled:
-                        type: boolean
-                    type: object
                   nodeSelector:
                     additionalProperties:
                       type: string

bundle/openshift/manifests/opentelemetry.io_opentelemetrycollectors.yaml

@@ -2299,11 +2299,6 @@ spec:
                 - managed
                 - unmanaged
                 type: string
-              mtls:
-                properties:
-                  enabled:
-                    type: boolean
-                type: object
               networkPolicy:
                 properties:
                   enabled:

bundle/openshift/manifests/opentelemetry.io_targetallocators.yaml

@@ -3241,11 +3241,6 @@ spec:
                     type: string
                   image:
                     type: string
-                  mtls:
-                    properties:
-                      enabled:
-                        type: boolean
-                    type: object
                   nodeSelector:
                     additionalProperties:
                       type: string