K8SPSMDB-1074 fixed MongoDB Cluster cannot failover when down time all pods using mode External (NodePort and LB) by chungtd (#1535)

chungtd203338 · hors · egegunes · web-flow · commit 6cd1efd5e539 · 2024-05-17T11:17:57.000+03:00
* K8SPSMDB-1074 fixed by chungtd

* edit compare files to fix tests

---------

Co-authored-by: Viacheslav Sarzhan &lt;slava.sarzhan@percona.com&gt;
Co-authored-by: Ege Güneş &lt;ege.gunes@percona.com&gt;
diff --git a/e2e-tests/ignore-labels-annotations/compare/service_some-name-cfg-0-crmeta.yml b/e2e-tests/ignore-labels-annotations/compare/service_some-name-cfg-0-crmeta.yml
@@ -24,6 +24,7 @@ spec:
       port: 27017
       protocol: TCP
       targetPort: 27017
+  publishNotReadyAddresses: true
   selector:
     statefulset.kubernetes.io/pod-name: some-name-cfg-0
   sessionAffinity: None
diff --git a/e2e-tests/ignore-labels-annotations/compare/service_some-name-cfg-0-manual.yml b/e2e-tests/ignore-labels-annotations/compare/service_some-name-cfg-0-manual.yml
@@ -24,6 +24,7 @@ spec:
       port: 27017
       protocol: TCP
       targetPort: 27017
+  publishNotReadyAddresses: true
   selector:
     statefulset.kubernetes.io/pod-name: some-name-cfg-0
   sessionAffinity: None
diff --git a/e2e-tests/ignore-labels-annotations/compare/service_some-name-cfg-0-no-ignored-meta.yml b/e2e-tests/ignore-labels-annotations/compare/service_some-name-cfg-0-no-ignored-meta.yml
@@ -22,6 +22,7 @@ spec:
       port: 27017
       protocol: TCP
       targetPort: 27017
+  publishNotReadyAddresses: true
   selector:
     statefulset.kubernetes.io/pod-name: some-name-cfg-0
   sessionAffinity: None
diff --git a/e2e-tests/ignore-labels-annotations/compare/service_some-name-cfg-0-one-service-label-annotation.yml b/e2e-tests/ignore-labels-annotations/compare/service_some-name-cfg-0-one-service-label-annotation.yml
@@ -24,6 +24,7 @@ spec:
       port: 27017
       protocol: TCP
       targetPort: 27017
+  publishNotReadyAddresses: true
   selector:
     statefulset.kubernetes.io/pod-name: some-name-cfg-0
   sessionAffinity: None
diff --git a/e2e-tests/ignore-labels-annotations/compare/service_some-name-cfg-0-two-service-labels-annotations.yml b/e2e-tests/ignore-labels-annotations/compare/service_some-name-cfg-0-two-service-labels-annotations.yml
@@ -26,6 +26,7 @@ spec:
       port: 27017
       protocol: TCP
       targetPort: 27017
+  publishNotReadyAddresses: true
   selector:
     statefulset.kubernetes.io/pod-name: some-name-cfg-0
   sessionAffinity: None
diff --git a/e2e-tests/ignore-labels-annotations/compare/service_some-name-rs0-0-crmeta.yml b/e2e-tests/ignore-labels-annotations/compare/service_some-name-rs0-0-crmeta.yml
@@ -24,6 +24,7 @@ spec:
       port: 27017
       protocol: TCP
       targetPort: 27017
+  publishNotReadyAddresses: true
   selector:
     statefulset.kubernetes.io/pod-name: some-name-rs0-0
   sessionAffinity: None
diff --git a/e2e-tests/ignore-labels-annotations/compare/service_some-name-rs0-0-manual.yml b/e2e-tests/ignore-labels-annotations/compare/service_some-name-rs0-0-manual.yml
@@ -24,6 +24,7 @@ spec:
       port: 27017
       protocol: TCP
       targetPort: 27017
+  publishNotReadyAddresses: true
   selector:
     statefulset.kubernetes.io/pod-name: some-name-rs0-0
   sessionAffinity: None
diff --git a/e2e-tests/ignore-labels-annotations/compare/service_some-name-rs0-0-no-ignored-meta.yml b/e2e-tests/ignore-labels-annotations/compare/service_some-name-rs0-0-no-ignored-meta.yml
@@ -22,6 +22,7 @@ spec:
       port: 27017
       protocol: TCP
       targetPort: 27017
+  publishNotReadyAddresses: true
   selector:
     statefulset.kubernetes.io/pod-name: some-name-rs0-0
   sessionAffinity: None
diff --git a/e2e-tests/ignore-labels-annotations/compare/service_some-name-rs0-0-one-service-label-annotation.yml b/e2e-tests/ignore-labels-annotations/compare/service_some-name-rs0-0-one-service-label-annotation.yml
@@ -24,6 +24,7 @@ spec:
       port: 27017
       protocol: TCP
       targetPort: 27017
+  publishNotReadyAddresses: true
   selector:
     statefulset.kubernetes.io/pod-name: some-name-rs0-0
   sessionAffinity: None
diff --git a/e2e-tests/ignore-labels-annotations/compare/service_some-name-rs0-0-two-service-labels-annotations.yml b/e2e-tests/ignore-labels-annotations/compare/service_some-name-rs0-0-two-service-labels-annotations.yml
@@ -26,6 +26,7 @@ spec:
       port: 27017
       protocol: TCP
       targetPort: 27017
+  publishNotReadyAddresses: true
   selector:
     statefulset.kubernetes.io/pod-name: some-name-rs0-0
   sessionAffinity: None
diff --git a/e2e-tests/service-per-pod/compare/service_cluster-ip-rs0-0.yml b/e2e-tests/service-per-pod/compare/service_cluster-ip-rs0-0.yml
@@ -20,6 +20,7 @@ spec:
       port: 27017
       protocol: TCP
       targetPort: 27017
+  publishNotReadyAddresses: true
   selector:
     statefulset.kubernetes.io/pod-name: cluster-ip-rs0-0
   sessionAffinity: None
diff --git a/e2e-tests/service-per-pod/compare/service_local-balancer-rs0-0.yml b/e2e-tests/service-per-pod/compare/service_local-balancer-rs0-0.yml
@@ -21,6 +21,7 @@ spec:
       port: 27017
       protocol: TCP
       targetPort: 27017
+  publishNotReadyAddresses: true
   selector:
     statefulset.kubernetes.io/pod-name: local-balancer-rs0-0
   sessionAffinity: None
diff --git a/e2e-tests/service-per-pod/compare/service_node-port-rs0-0-updated.yml b/e2e-tests/service-per-pod/compare/service_node-port-rs0-0-updated.yml
@@ -23,6 +23,7 @@ spec:
       port: 27017
       protocol: TCP
       targetPort: 27017
+  publishNotReadyAddresses: true
   selector:
     statefulset.kubernetes.io/pod-name: node-port-rs0-0
   sessionAffinity: None
diff --git a/e2e-tests/service-per-pod/compare/service_node-port-rs0-0.yml b/e2e-tests/service-per-pod/compare/service_node-port-rs0-0.yml
@@ -21,6 +21,7 @@ spec:
       port: 27017
       protocol: TCP
       targetPort: 27017
+  publishNotReadyAddresses: true
   selector:
     statefulset.kubernetes.io/pod-name: node-port-rs0-0
   sessionAffinity: None
diff --git a/pkg/psmdb/service.go b/pkg/psmdb/service.go
@@ -104,7 +104,8 @@ func ExternalService(cr *api.PerconaServerMongoDB, replset *api.ReplsetSpec, pod
 				TargetPort: intstr.FromInt(int(api.DefaultMongodPort)),
 			},
 		},
-		Selector: map[string]string{"statefulset.kubernetes.io/pod-name": podName},
+		Selector:                 map[string]string{"statefulset.kubernetes.io/pod-name": podName},
+		PublishNotReadyAddresses: true,
 	}
 
 	switch replset.Expose.ExposeType {
