feat: Add support for service.selector override

charts/common/templates/service.yaml

@@ -28,6 +28,10 @@ spec:
     targetPort: {{ .Values.service.internalPort }}
   {{- end }}
   selector:
+  {{- if .Values.service.selector }}
+  {{- toYaml .Values.service.selector | nindent 4 }}
+  {{- else }}
     app: {{ $releaseName }}
+  {{- end }}
   type: ClusterIP
 {{- end }}

charts/common/tests/service_test.yaml

@@ -70,3 +70,14 @@ tests:
       - equal:
           path: metadata.name
           value: override
+  - it: can override selector
+    set:
+      service:
+        selector:
+          traffic-pool: override
+    asserts:
+      - equal:
+          path: spec.selector["traffic-pool"]
+          value: override
+      - isNull:
+          path: spec.selector.app

charts/common/values.schema.json

@@ -250,6 +250,10 @@
           "description": "Custom ports spec, overrides default port config",
           "items": { "type": "object" },
           "type": "array"
+        },
+        "selector": {
+          "description": "Optionally override the Service selector. Defaults to app: <release-name>",
+          "type": "object"
         }
       },
       "type": "object"

charts/common/values.yaml

@@ -177,6 +177,8 @@ service:
   internalPort: 8080
   # -- Optionally set annotations for the service
   annotations: {}
+  # -- Optionally override the Service selector. Defaults to app: <release-name>
+  selector: {}
 
 # -- Takes a list of `container` entries, you must add a `name` field for each entry
 containers: []