K8SPSMDB-791: fix loadBalancerSourceRanges (#1264)

https://jira.percona.com/browse/K8SPSMDB-791

Co-authored-by: Inel Pandzic <[email protected]>
Co-authored-by: Viacheslav Sarzhan <[email protected]>

Author: 3 people

e2e-tests/demand-backup-sharded/conf/some-name-rs0.yml

@@ -62,6 +62,8 @@ spec:
       expose:
         enabled: true
         exposeType: ClusterIP
+        loadBalancerSourceRanges:
+        - 10.0.0.0/8
       sidecars:
       - image: busybox
         command: ["/bin/sh"]
@@ -76,6 +78,8 @@ spec:
           localPingThresholdMs: 15
       expose:
         exposeType: ClusterIP
+        loadBalancerSourceRanges:
+        - 10.0.0.0/8
       sidecars:
       - image: busybox
         command: ["/bin/sh"]
@@ -175,6 +179,8 @@ spec:
     expose:
       enabled: true
       exposeType: ClusterIP
+      loadBalancerSourceRanges:
+      - 10.0.0.0/8
     resources:
       limits:
         cpu: 500m

pkg/psmdb/mongos.go

@@ -439,8 +439,7 @@ func MongosServiceSpec(cr *api.PerconaServerMongoDB, podName string) corev1.Serv
 				TargetPort: intstr.FromInt(int(cr.Spec.Sharding.Mongos.Port)),
 			},
 		},
-		Selector:                 ls,
-		LoadBalancerSourceRanges: cr.Spec.Sharding.Mongos.Expose.LoadBalancerSourceRanges,
+		Selector: ls,
 	}
 
 	switch cr.Spec.Sharding.Mongos.Expose.ExposeType {
@@ -450,6 +449,7 @@ func MongosServiceSpec(cr *api.PerconaServerMongoDB, podName string) corev1.Serv
 	case corev1.ServiceTypeLoadBalancer:
 		spec.Type = corev1.ServiceTypeLoadBalancer
 		spec.ExternalTrafficPolicy = "Cluster"
+		spec.LoadBalancerSourceRanges = cr.Spec.Sharding.Mongos.Expose.LoadBalancerSourceRanges
 	default:
 		spec.Type = corev1.ServiceTypeClusterIP
 	}

pkg/psmdb/service.go

@@ -47,9 +47,8 @@ func Service(cr *api.PerconaServerMongoDB, replset *api.ReplsetSpec) *corev1.Ser
 					TargetPort: intstr.FromInt(int(api.MongodPort(cr))),
 				},
 			},
-			ClusterIP:                "None",
-			Selector:                 ls,
-			LoadBalancerSourceRanges: replset.Expose.LoadBalancerSourceRanges,
+			ClusterIP: "None",
+			Selector:  ls,
 		},
 	}