Merge pull request #1002 from kubeshop/update-metrics-for-operator

feat: add an option for insecure metrics port for the operator

Author: ypoplavs

charts/testkube-operator/templates/deployment.yaml

@@ -44,6 +44,10 @@ spec:
         - --webhook-cert-path=/tmp/k8s-webhook-server/serving-certs
         - --metrics-cert-path=/tmp/k8s-webhook-server/serving-certs
         - --leader-elect
+        {{- if .Values.metricsServerPortInsecure }}
+        - --metrics-bind-address=:{{ .Values.metricsServerPortInsecure }}
+        - --metrics-secure=false
+        {{- end }}
         command:
         - /manager
         image: {{ include "testkube-operator.image" . }}

charts/testkube-operator/templates/service.yaml

@@ -20,6 +20,11 @@ spec:
   - name: https
     port: {{ .Values.metricsServerPort }}
     targetPort: https
+  {{- if .Values.metricsServerPortInsecure }}
+  - name: http
+    port: {{ .Values.metricsServerPortInsecure }}
+    targetPort: http
+  {{- end }}
   selector:
     control-plane: controller-manager
 

charts/testkube-operator/values.yaml

@@ -96,6 +96,8 @@ apiPort: 8088
 webhookServerPort: 9443
 # -- Testkube Operator metrics server port
 metricsServerPort: 8443
+# -- Testkube Operator insecure metrics server port
+#metricsServerPortInsecure: 8080
 # -- Testkube Operator healthcheck port
 healthcheckPort: 8081
 # use ArgoCD sync owner references

charts/testkube/values.yaml

@@ -1024,6 +1024,8 @@ testkube-operator:
   webhookServerPort: 9443
   # -- Testkube Operator metrics server port
   metricsServerPort: 8443
+  # -- Testkube Operator insecure metrics server port
+  #metricsServerPortInsecure: 8080
   # -- Testkube Operator healthcheck port
   healthcheckPort: 8081
   # -- Use ArgoCD sync owner references