move to a top level. it's more of an all or nothing setting

Signed-off-by: lemonprogis <edward.briggler@gmail.com>

Author: lemonprogis

deployment/chainloop/templates/cas/service-metrics.yaml

@@ -3,22 +3,22 @@ Copyright Chainloop, Inc. All Rights Reserved.
 SPDX-License-Identifier: APACHE-2.0
 */}}
 
-{{- if .Values.cas.metrics.enabled }}
+{{- if .Values.metrics.enabled }}
 apiVersion: v1
 kind: Service
 metadata:
   name: {{ include "chainloop.cas.fullname" . }}-metrics
   namespace: {{ include "common.names.namespace" . | quote }}
   labels: {{ include "chainloop.cas.labels" . | nindent 4 }}
-  {{- if or .Values.cas.metrics.service.annotations .Values.commonAnnotations }}
-  {{- $annotations := include "common.tplvalues.merge" (dict "values" (list .Values.cas.metrics.service.annotations .Values.commonAnnotations) "context" .) }}
+  {{- if or .Values.metrics.service.annotations .Values.commonAnnotations }}
+  {{- $annotations := include "common.tplvalues.merge" (dict "values" (list .Values.metrics.service.annotations .Values.commonAnnotations) "context" .) }}
   annotations: {{- include "common.tplvalues.render" ( dict "value" $annotations "context" $ ) | nindent 4 }}
   {{- end }}
 spec:
   type: ClusterIP
   ports:
     - name: metrics
-      port: {{ .Values.cas.metrics.service.port }}
+      port: {{ .Values.metrics.service.port }}
       targetPort: metrics
       protocol: TCP
   selector: {{ include "chainloop.cas.selectorLabels" . | nindent 4 }}

deployment/chainloop/templates/controlplane/service-metrics.yaml

@@ -3,22 +3,22 @@ Copyright Chainloop, Inc. All Rights Reserved.
 SPDX-License-Identifier: APACHE-2.0
 */}}
 
-{{- if .Values.controlplane.metrics.enabled }}
+{{- if .Values.metrics.enabled }}
 apiVersion: v1
 kind: Service
 metadata:
   name: {{ include "chainloop.controlplane.fullname" . }}-metrics
   namespace: {{ include "common.names.namespace" . | quote }}
   labels: {{- include "chainloop.controlplane.labels" . | nindent 4 }}
-  {{- if or .Values.controlplane.metrics.service.annotations .Values.commonAnnotations }}
-  {{- $annotations := include "common.tplvalues.merge" (dict "values" (list .Values.controlplane.metrics.service.annotations .Values.commonAnnotations) "context" .) }}
+  {{- if or .Values.metrics.service.annotations .Values.commonAnnotations }}
+  {{- $annotations := include "common.tplvalues.merge" (dict "values" (list .Values.metrics.service.annotations .Values.commonAnnotations) "context" .) }}
   annotations: {{- include "common.tplvalues.render" ( dict "value" $annotations "context" $ ) | nindent 4 }}
   {{- end }}
 spec:
   type: ClusterIP
   ports:
     - name: metrics
-      port: {{ .Values.controlplane.metrics.service.port }}
+      port: {{ .Values.metrics.service.port }}
       targetPort: metrics
       protocol: TCP
   {{- $podLabels := include "common.tplvalues.merge" (dict "values" (list .Values.controlplane.podLabels .Values.commonLabels) "context" .) | fromYaml }}

deployment/chainloop/values.yaml

@@ -51,6 +51,18 @@ commonLabels: {}
 ##
 extraDeploy: []
 
+## @section Prometheus Metrics
+## @extra metrics Prometheus metrics configuration for controlplane and CAS
+## @param metrics.enabled Enable metrics services for both controlplane and CAS
+metrics:
+  enabled: false
+  ## @extra metrics.service Metrics service configuration
+  service:
+    ## @param metrics.service.port Metrics service port
+    port: 5000
+    ## @param metrics.service.annotations Additional custom annotations for metrics services
+    annotations: {}
+
 ## @section Secrets Backend
 ##
 
@@ -338,17 +350,6 @@ controlplane:
     ##
     sessionAffinityConfig: {}
 
-  ## @extra controlplane.metrics Prometheus metrics configuration
-  ## @param controlplane.metrics.enabled Enable metrics service
-  metrics:
-    enabled: false
-    ## @extra controlplane.metrics.service Metrics service configuration
-    service:
-      ## @param controlplane.metrics.service.port Metrics service port
-      port: 5000
-      ## @param controlplane.metrics.service.annotations Additional custom annotations for metrics service
-      annotations: {}
-
   serviceAPI:
     ## @param controlplane.serviceAPI.type Service type
     type: ClusterIP
@@ -1103,17 +1104,6 @@ cas:
     ##
     sessionAffinityConfig: {}
 
-  ## @extra cas.metrics Prometheus metrics configuration
-  ## @param cas.metrics.enabled Enable metrics service
-  metrics:
-    enabled: false
-    ## @extra cas.metrics.service Metrics service configuration
-    service:
-      ## @param cas.metrics.service.port Metrics service port
-      port: 5000
-      ## @param cas.metrics.service.annotations Additional custom annotations for metrics service
-      annotations: {}
-
   serviceAPI:
     ## @param cas.serviceAPI.type Service type
     type: ClusterIP