Metrics port update (#66)

* metrics port updation

* update metrics condition

* metrics port updation 2

---------

Co-authored-by: arunesh-kops <arunesh.j@kops.dev>

Author: jaya-kops

charts/cron-job/templates/cronJob.yaml

@@ -64,8 +64,10 @@ spec:
               {{- end }}
               imagePullPolicy: IfNotPresent
               ports:
-                - name: metrics-port
-                  containerPort: {{ .Values.metricsPort }}
+                {{- if and .Values.metricsPort (ne (int .Values.metricsPort) 0) }}
+                  - name: metrics-port
+                    containerPort: {{ .Values.metricsPort }}
+                {{- end }}
               envFrom:
                 {{- range $i, $name := .Values.envFrom.configmaps }}
                 - configMapRef:

charts/cron-job/templates/service.yaml

@@ -9,9 +9,11 @@ spec:
     - name: http-port
       port: 80
       targetPort: {{ .Values.httpPort}}
+    {{- if and .Values.metricsPort (ne (int .Values.metricsPort) 0) }}
     - name: metrics-port
       port: 2121
-      targetPort: {{ .Values.metricsPort}}
+      targetPort: {{ .Values.metricsPort }}
+    {{- end }}
   selector:
     app: {{ .Values.name }}
   type: NodePort

charts/cron-job/templates/serviceMonitor.yaml

@@ -10,7 +10,9 @@ spec:
     matchLabels:
       app: {{.Values.name}}
   endpoints:
+    {{- if .Values.metricsPort}}
     - port: metrics-port
-      interval: {{.Values.metricsScrapeInterval }}
+      interval: {{ .Values.metricsScrapeInterval }}
       path: /metrics
+    {{- end }}
 ---

charts/service/templates/deployment.yaml

@@ -77,8 +77,10 @@ spec:
           ports:
             - name: http-port
               containerPort: {{ .Values.httpPort }}
+            {{- if and .Values.metricsPort (ne ( int .Values.metricsPort ) 0) }}
             - name: metrics-port
               containerPort: {{ .Values.metricsPort }}
+            {{- end }}
             {{- range $k, $v := .Values.ports}}
             - name: {{ $k }}
               containerPort: {{ $v }}

charts/service/templates/service.yaml

@@ -9,9 +9,11 @@ spec:
     - name: http-port
       port: 80
       targetPort: {{ .Values.httpPort}}
+    {{- if and .Values.metricsPort (ne (int .Values.metricsPort) 0) }}
     - name: metrics-port
       port: 2121
-      targetPort: {{ .Values.metricsPort}}
+      targetPort: {{ .Values.metricsPort }}
+    {{- end}}
     {{- range $k, $v := .Values.ports}}
     - name: {{ $k}}
       port: {{ $v }}

charts/service/templates/serviceMonitor.yaml

@@ -10,7 +10,9 @@ spec:
     matchLabels:
       app: {{.Values.name}}
   endpoints:
-    - port: metrics-port
-      interval: {{.Values.metricsScrapeInterval }}
-      path: /metrics
+   {{- if .Values.metricsPort}}
+  - port: metrics-port
+    interval: {{ .Values.metricsScrapeInterval }}
+    path: /metrics
+  {{- end }}
 ---