Setting appProtocol for service objects

percona · Feb 23, 2024 · 6415417 · 6415417
1 parent 1d90e1e
commit 6415417
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 10 deletions.
diff --git a/e2e-tests/init-deploy/compare/service_another-name-rs0.yml b/e2e-tests/init-deploy/compare/service_another-name-rs0.yml
@@ -15,7 +15,8 @@ metadata:
       name: another-name
 spec:
   ports:
-    - name: mongodb
+    - appProtocol: mongo
+      name: mongodb
       port: 27017
       protocol: TCP
       targetPort: 27017

diff --git a/e2e-tests/upgrade-consistency-sharded/compare/service_some-name-cfg-1140.yml b/e2e-tests/upgrade-consistency-sharded/compare/service_some-name-cfg-1140.yml
@@ -15,7 +15,8 @@ metadata:
       name: some-name
 spec:
   ports:
-    - name: mongodb
+    - appProtocol: mongo
+      name: mongodb
       port: 27017
       protocol: TCP
       targetPort: 27017

diff --git a/e2e-tests/upgrade-consistency-sharded/compare/service_some-name-rs0-1150.yml b/e2e-tests/upgrade-consistency-sharded/compare/service_some-name-rs0-1150.yml
@@ -15,7 +15,8 @@ metadata:
       name: some-name
 spec:
   ports:
-    - name: mongodb
+    - appProtocol: mongo
+      name: mongodb
       port: 27017
       protocol: TCP
       targetPort: 27017

diff --git a/e2e-tests/upgrade-consistency/compare/service_some-name-rs0-1150.yml b/e2e-tests/upgrade-consistency/compare/service_some-name-rs0-1150.yml
@@ -15,7 +15,8 @@ metadata:
       name: some-name
 spec:
   ports:
-    - name: mongodb
+    - appProtocol: mongo
+      name: mongodb
       port: 27017
       protocol: TCP
       targetPort: 27017

diff --git a/pkg/psmdb/service.go b/pkg/psmdb/service.go
@@ -28,7 +28,7 @@ func Service(cr *api.PerconaServerMongoDB, replset *api.ReplsetSpec) *corev1.Ser
 		"app.kubernetes.io/managed-by": "percona-server-mongodb-operator",
 		"app.kubernetes.io/part-of":    "percona-server-mongodb",
 	}
-	appProtocol := "mongo"
+
 	svc := &corev1.Service{
 		TypeMeta: metav1.TypeMeta{
 			APIVersion: "v1",
@@ -42,10 +42,9 @@ func Service(cr *api.PerconaServerMongoDB, replset *api.ReplsetSpec) *corev1.Ser
 		Spec: corev1.ServiceSpec{
 			Ports: []corev1.ServicePort{
 				{
-					Name:        mongodPortName,
-					Port:        api.DefaultMongodPort,
-					TargetPort:  intstr.FromInt(int(api.DefaultMongodPort)),
-					AppProtocol: &appProtocol,
+					Name:       mongodPortName,
+					Port:       api.DefaultMongodPort,
+					TargetPort: intstr.FromInt(int(api.DefaultMongodPort)),
 				},
 			},
 			ClusterIP: "None",
@@ -64,7 +63,12 @@ func Service(cr *api.PerconaServerMongoDB, replset *api.ReplsetSpec) *corev1.Ser
 			}
 		}
 	}
-
+	if cr.CompareVersion("1.16.0") < 0 {
+		appProtocol := "mongo"
+		for _, v := range svc.Spec.Ports {
+			v.AppProtocol = &appProtocol
+		}
+	}
 	return svc
 }