Revert explicit check on type to recreate when type changes

elastic · Oct 29, 2024 · d38519e · d38519e
1 parent 0d3a897
commit d38519e
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 17 deletions.
diff --git a/pkg/controller/common/service_control.go b/pkg/controller/common/service_control.go
@@ -88,11 +88,6 @@ func needsUpdate(expected *corev1.Service, reconciled *corev1.Service) bool {
 
 // applyServerSideValues applies any default that may have been set from the reconciled version.
 func applyServerSideValues(expected, reconciled *corev1.Service) {
-	// shortcut if the service type has changed
-	if expected.Spec.Type != reconciled.Spec.Type {
-		return
-	}
-
 	// Type may be defaulted by the api server
 	if expected.Spec.Type == "" {
 		expected.Spec.Type = reconciled.Spec.Type

diff --git a/pkg/controller/common/service_control_test.go b/pkg/controller/common/service_control_test.go
@@ -275,18 +275,6 @@ func Test_needsRecreate(t *testing.T) {
 			},
 			want: true,
 		},
-		{
-			name: "Needs recreate to go from LoadBalancer type to ClusterIP",
-			args: args{
-				expected: corev1.Service{Spec: corev1.ServiceSpec{
-					Type: corev1.ServiceTypeClusterIP,
-				}},
-				reconciled: corev1.Service{Spec: corev1.ServiceSpec{
-					Type: corev1.ServiceTypeLoadBalancer,
-				}},
-			},
-			want: true,
-		},
 		{
 			name: "Removing the load balancer class is OK if target type is no longer LoadBalancer",
 			args: args{