Browse Source

fix: add port name to deployment (#697)

Co-authored-by: Alexander Huck <alexander.huck@inovex.de>
Co-authored-by: Alberto Llamas <albertollamaso@gmail.com>
Alexander Huck 4 years ago
parent
commit
2bbc033699

+ 1 - 0
deploy/charts/external-secrets/templates/deployment.yaml

@@ -68,6 +68,7 @@ spec:
           ports:
           ports:
             - containerPort: {{ .Values.prometheus.service.port }}
             - containerPort: {{ .Values.prometheus.service.port }}
               protocol: TCP
               protocol: TCP
+              name: metrics
           {{- with .Values.extraEnv }}
           {{- with .Values.extraEnv }}
           env:
           env:
             {{- toYaml . | nindent 12 }}
             {{- toYaml . | nindent 12 }}

+ 1 - 1
deploy/charts/external-secrets/templates/service.yaml

@@ -13,8 +13,8 @@ spec:
   type: ClusterIP
   type: ClusterIP
   ports:
   ports:
     - port: {{ .Values.prometheus.service.port }}
     - port: {{ .Values.prometheus.service.port }}
-      targetPort: {{ .Values.prometheus.service.port }}
       protocol: TCP
       protocol: TCP
+      name: metrics
   selector:
   selector:
     {{- include "external-secrets.selectorLabels" . | nindent 4 }}
     {{- include "external-secrets.selectorLabels" . | nindent 4 }}
 {{- end }}
 {{- end }}