Setting appProtocol for service objects

Author: sorenmat

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

e2e-tests/init-deploy/compare/service_some-name-rs0.yml

@@ -15,7 +15,8 @@ metadata:
       name: some-name
 spec:
   ports:
-    - name: mongodb
+    - appProtocol: mongo
+      name: mongodb
       port: 27017
       protocol: TCP
       targetPort: 27017

e2e-tests/monitoring-2-0/compare/service_monitoring-rs0.yml

@@ -15,7 +15,8 @@ metadata:
       name: monitoring
 spec:
   ports:
-    - name: mongodb
+    - appProtocol: mongo
+      name: mongodb
       port: 27017
       protocol: TCP
       targetPort: 27017

e2e-tests/one-pod/compare/service_one-pod-rs0.yml

@@ -17,7 +17,8 @@ metadata:
       name: one-pod
 spec:
   ports:
-    - name: mongodb
+    - appProtocol: mongo
+      name: mongodb
       port: 27017
       protocol: TCP
       targetPort: 27017

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

e2e-tests/upgrade-consistency-sharded/compare/service_some-name-rs0-1140.yml

@@ -15,7 +15,8 @@ metadata:
       name: some-name
 spec:
   ports:
-    - name: mongodb
+    - appProtocol: mongo
+      name: mongodb
       port: 27017
       protocol: TCP
       targetPort: 27017

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

e2e-tests/upgrade-consistency/compare/service_some-name-rs0-1140.yml

@@ -15,7 +15,8 @@ metadata:
       name: some-name
 spec:
   ports:
-    - name: mongodb
+    - appProtocol: mongo
+      name: mongodb
       port: 27017
       protocol: TCP
       targetPort: 27017

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

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
 }