[COMMON] Disable metrics by default 77/142577/8
authorFiete Ostkamp <fiete.ostkamp@telekom.de>
Sun, 30 Nov 2025 09:55:01 +0000 (10:55 +0100)
committerFiete Ostkamp <fiete.ostkamp@telekom.de>
Sun, 30 Nov 2025 11:27:15 +0000 (12:27 +0100)
Similar to how we do not enable tracing by default, metrics should not
be enabled by default either [0]
This requires some adjustments, since it practically was not possible to deploy
ONAP without it:

- add if condition to "common.serviceMonitor" for .Values.global.metrics.enabled
- remove chart-specific if wrappers that do the same
- this allows to toggle metrics monitoring via Prometheus for the whole platform
- without this change, it would not be possible to deploy ONAP without the Prometheus
  Operator

[0] The prometheus backend that is required for this is not part of the ONAP deployment
    and it should thus be a conscious decision to both enable metrics and deploy it

Issue-ID: OOM-3372
Change-Id: I7c585f6f4b77e9e33ea334cc5ee64dcd7f2d164f
Signed-off-by: Fiete Ostkamp <fiete.ostkamp@telekom.de>
47 files changed:
kubernetes/aai/components/aai-babel/templates/servicemonitor.yaml
kubernetes/aai/components/aai-babel/values.yaml
kubernetes/aai/components/aai-graphadmin/templates/servicemonitor.yaml
kubernetes/aai/components/aai-graphadmin/values.yaml
kubernetes/aai/components/aai-modelloader/templates/podmonitor.yaml
kubernetes/aai/components/aai-modelloader/values.yaml
kubernetes/aai/components/aai-resources/templates/servicemonitor.yaml
kubernetes/aai/components/aai-resources/values.yaml
kubernetes/aai/components/aai-traversal/templates/servicemonitor.yaml
kubernetes/aai/components/aai-traversal/values.yaml
kubernetes/cds/components/cds-command-executor/templates/servicemonitor.yaml
kubernetes/cds/components/cds-command-executor/values.yaml
kubernetes/common/cassandra/templates/servicemonitor.yaml
kubernetes/common/cassandra/values.yaml
kubernetes/common/common/templates/_serviceMonitor.tpl
kubernetes/common/mariadb-galera/templates/servicemonitor.yaml
kubernetes/common/mariadb-galera/values.yaml
kubernetes/common/mongodb/templates/servicemonitor.yaml
kubernetes/common/nginx/templates/servicemonitor.yaml
kubernetes/common/postgres/templates/servicemonitor.yaml
kubernetes/common/postgres/values.yaml
kubernetes/cps/components/cps-core/templates/serviceMonitor.yaml
kubernetes/cps/components/cps-core/values.yaml
kubernetes/cps/components/cps-temporal/templates/serviceMonitor.yaml
kubernetes/cps/components/cps-temporal/values.yaml
kubernetes/cps/components/ncmp-dmi-plugin/templates/serviceMonitor.yaml
kubernetes/cps/components/ncmp-dmi-plugin/values.yaml
kubernetes/onap/values.yaml
kubernetes/policy/components/policy-apex-pdp/templates/serviceMonitor.yaml
kubernetes/policy/components/policy-apex-pdp/values.yaml
kubernetes/policy/components/policy-api/templates/serviceMonitor.yaml
kubernetes/policy/components/policy-api/values.yaml
kubernetes/policy/components/policy-distribution/templates/serviceMonitor.yaml
kubernetes/policy/components/policy-distribution/values.yaml
kubernetes/policy/components/policy-drools-pdp/templates/serviceMonitor.yaml
kubernetes/policy/components/policy-drools-pdp/values.yaml
kubernetes/policy/components/policy-opa-pdp/templates/serviceMonitor.yaml
kubernetes/policy/components/policy-opa-pdp/values.yaml
kubernetes/policy/components/policy-pap/templates/serviceMonitor.yaml
kubernetes/policy/components/policy-pap/values.yaml
kubernetes/policy/components/policy-xacml-pdp/templates/serviceMonitor.yaml
kubernetes/policy/components/policy-xacml-pdp/values.yaml
kubernetes/policy/values.yaml
kubernetes/sdc/components/sdc-be/templates/servicemonitor.yaml
kubernetes/sdc/components/sdc-be/values.yaml
kubernetes/sdc/components/sdc-onboarding-be/templates/servicemonitor.yaml
kubernetes/sdc/components/sdc-onboarding-be/values.yaml

index 3dd7369..482c2d9 100644 (file)
@@ -130,11 +130,8 @@ profiling:
 
 metrics:
   serviceMonitor:
-    enabled: true
     targetPort: 9516
     path: /services/babel-service/actuator/prometheus
-    basicAuth:
-      enabled: false
 
 #Pods Service Account
 serviceAccount:
index c36a576..48ba8a4 100644 (file)
@@ -321,11 +321,8 @@ resources:
 
 metrics:
   serviceMonitor:
-    enabled: true
     targetPort: 8448
     path: /actuator/prometheus
-    basicAuth:
-      enabled: false
 
     selector:
       app: '{{ include "common.name" . }}'
index 961a850..272aabc 100644 (file)
@@ -14,7 +14,8 @@
 # limitations under the License.
 */}}
 
-{{- if .Values.metrics.podMonitor.enabled }}
+{{- $metrics := .Values.global.metrics | default dict -}}
+{{- if $metrics.enabled }}
 apiVersion: monitoring.coreos.com/v1
 kind: PodMonitor
 metadata:
index 57c3d83..48a3430 100644 (file)
@@ -134,7 +134,6 @@ tracing:
 
 metrics:
   podMonitor:
-    enabled: true
     port: http
     path: /actuator/prometheus
 
index 50bb0ee..74a0e69 100644 (file)
@@ -346,7 +346,6 @@ autoscaling:
 
 metrics:
   serviceMonitor:
-    enabled: true
     targetPort: 8448
     path: /actuator/prometheus
     basicAuth:
index dc70602..47b84fb 100644 (file)
@@ -1,3 +1 @@
-{{- if .Values.metrics.serviceMonitor.enabled }}
-{{  include "common.serviceMonitor" . }}
-{{- end }}
+{{  include "common.serviceMonitor" . }}
\ No newline at end of file
index 98442a9..3f2154f 100644 (file)
@@ -363,14 +363,8 @@ podAnnotations:
 
 metrics:
   serviceMonitor:
-    enabled: true
     targetPort: 8448
     path: /actuator/prometheus
-    basicAuth:
-      enabled: false
-      externalSecretName: mysecretname
-      externalSecretUserKey: login
-      externalSecretPasswordKey: password
 
     ## Namespace in which Prometheus is running
     ##
index 101b24a..aee16cb 100644 (file)
@@ -14,6 +14,4 @@
 # limitations under the License.
 */}}
 
-{{- if .Values.metrics.serviceMonitor.enabled }}
-{{  include "common.serviceMonitor" . }}
-{{- end }}
\ No newline at end of file
+{{  include "common.serviceMonitor" . }}
\ No newline at end of file
index 36718cf..3c52557 100755 (executable)
@@ -103,7 +103,6 @@ serviceAccount:
 
 metrics:
   serviceMonitor:
-    enabled: false
     port: *metricsPortname
     internalPort: *metricsPort
     path: /actuator/prometheus
index 57f4d3f..5cb7e0a 100644 (file)
@@ -15,7 +15,5 @@
 */}}
 
 {{- if not .Values.global.cassandra.useOperator }}
-{{- if .Values.metrics.serviceMonitor.enabled }}
-{{  include "common.serviceMonitor" . }}
-{{- end }}
+{{ include "common.serviceMonitor" . }}
 {{- end }}
index 0cd8545..e88ee1e 100644 (file)
@@ -278,7 +278,6 @@ metrics:
     successThreshold: 1
     failureThreshold: 3
   serviceMonitor:
-    enabled: false
     targetPort: 8080
     path: /metrics
     basicAuth:
index ccb1fbb..752e9fb 100644 (file)
@@ -118,7 +118,10 @@ labels: {{- include "common.labels" (dict "labels" $labels "dot" $dot) | nindent
 */}}
 {{- define "common.serviceMonitor" -}}
 {{-   $dot := default . .dot -}}
+{{-   $global := $dot.Values.global | default dict -}}
 {{-   $labels := default (dict) .labels -}}
+{{-   $metrics := $global.metrics | default dict -}}
+{{- if $metrics.enabled | default false }}
 apiVersion: monitoring.coreos.com/v1
 kind: ServiceMonitor
 metadata:
@@ -180,4 +183,5 @@ spec:
     {{- else }}
     matchLabels: {{- include "common.matchLabels" (dict "labels" $labels "dot" $dot) | nindent 6 }}
     {{- end }}
+{{- end }}
 {{- end -}}
index 1bffb24..62f3a1c 100644 (file)
@@ -15,7 +15,5 @@
 */}}
 
 {{- if not .Values.global.mariadbGalera.useOperator }}
-{{-   if .Values.metrics.serviceMonitor.enabled }}
-{{  include "common.serviceMonitor" . }}
-{{-   end }}
+{{ include "common.serviceMonitor" . }}
 {{- end }}
index 36f47e7..54ff4f1 100644 (file)
@@ -740,7 +740,6 @@ metrics:
   ## Prometheus Operator ServiceMonitor configuration
   ##
   serviceMonitor:
-    enabled: false
     ## Namespace in which Prometheus is running
     ##
     # namespace: monitoring
index 7849d46..d2fe9b6 100644 (file)
@@ -3,7 +3,8 @@ Copyright Broadcom, Inc. All Rights Reserved.
 SPDX-License-Identifier: APACHE-2.0
 */}}
 
-{{- if and .Values.metrics.enabled .Values.metrics.serviceMonitor.enabled }}
+{{-   $metrics := .Values.global.metrics | default dict -}}
+{{- if $metrics.enabled }}
 apiVersion: monitoring.coreos.com/v1
 kind: ServiceMonitor
 metadata:
index d597e06..2c239bc 100644 (file)
@@ -3,7 +3,8 @@ Copyright Broadcom, Inc. All Rights Reserved.
 SPDX-License-Identifier: APACHE-2.0
 */}}
 
-{{- if and .Values.metrics.enabled .Values.metrics.serviceMonitor.enabled }}
+{{- $metrics := .Values.global.metrics | default dict -}}
+{{- if $metrics.enabled }}
 apiVersion: monitoring.coreos.com/v1
 kind: ServiceMonitor
 metadata:
index 3fb716b..729a371 100644 (file)
@@ -14,7 +14,5 @@
 # limitations under the License.
 */}}
 {{- if not .Values.global.postgres.useOperator }}
-{{- if .Values.metrics.serviceMonitor.enabled }}
-{{  include "common.serviceMonitor" . }}
-{{- end }}
+{{ include "common.serviceMonitor" . }}
 {{- end }}
index c0cdfef..47ea86a 100644 (file)
@@ -234,9 +234,6 @@ metrics:
       prometheus.io/scrape: "true"
       prometheus.io/port: "9187"
   serviceMonitor:
-    enabled: false
-    basicAuth:
-      enabled: false
     ## Namespace in which Prometheus is running
     ##
     ## namespace: monitoring
index a49a662..c45e726 100644 (file)
@@ -18,6 +18,4 @@
 # ============LICENSE_END=========================================================
 */}}
 
-{{- if .Values.prometheus.enabled }}
-{{ include "common.serviceMonitor" . }}
-{{- end }}
\ No newline at end of file
+{{ include "common.serviceMonitor" . }}
\ No newline at end of file
index 79fd3eb..94e9569 100644 (file)
@@ -92,9 +92,6 @@ service:
     - name: tcp-hazelcast
       port: 5701
 
-prometheus:
-  enabled: false
-
 metrics:
   serviceMonitor:
     port: http
@@ -103,8 +100,6 @@ metrics:
     # targetPort: 8080
     path: /actuator/prometheus
     interval: 60s
-    basicAuth:
-      enabled: false
 
 pullPolicy: Always
 # flag to enable debugging - application support required
index a49a662..c45e726 100644 (file)
@@ -18,6 +18,4 @@
 # ============LICENSE_END=========================================================
 */}}
 
-{{- if .Values.prometheus.enabled }}
-{{ include "common.serviceMonitor" . }}
-{{- end }}
\ No newline at end of file
+{{ include "common.serviceMonitor" . }}
\ No newline at end of file
index 696d9bf..d08b0b6 100644 (file)
@@ -45,9 +45,6 @@ image: onap/cps-temporal:1.2.1
 containerPort: &svc_port 8080
 managementPort: &mgt_port 8081
 
-prometheus:
-  enabled: false
-
 service:
   type: ClusterIP
   name: cps-temporal
@@ -66,9 +63,7 @@ metrics:
       ##
       # targetPort: 8080
     path: /manage/prometheus
-    interval: 60s
-    basicAuth:
-      enabled: false
+    interval: 30s
 
 pullPolicy: IfNotPresent
 # flag to enable debugging - application support required
index a49a662..c45e726 100644 (file)
@@ -18,6 +18,4 @@
 # ============LICENSE_END=========================================================
 */}}
 
-{{- if .Values.prometheus.enabled }}
-{{ include "common.serviceMonitor" . }}
-{{- end }}
\ No newline at end of file
+{{ include "common.serviceMonitor" . }}
\ No newline at end of file
index dd08b07..d711c45 100644 (file)
@@ -67,9 +67,7 @@ metrics:
       ##
     # targetPort: 8080
     path: /actuator/prometheus
-    interval: 60s
-    basicAuth:
-      enabled: false
+    interval: 30s
 
 pullPolicy: IfNotPresent
 # flag to enable debugging - application support required
index 92f4e73..d3ebfdd 100755 (executable)
@@ -229,12 +229,12 @@ global:
 
   # metrics part
   # If enabled, exporters (for prometheus) will be deployed
-  # if custom resources set to yes, CRD from prometheus operartor will be
+  # if custom resources set to yes, CRD from prometheus operator will be
   # created
   # Not all components have it enabled.
   #
   metrics:
-    enabled: true
+    enabled: false
     custom_resources: false
 
   # Disabling AAF
index 8284c73..d261392 100644 (file)
@@ -18,6 +18,4 @@
 # ============LICENSE_END=========================================================
 */}}
 
-{{- if .Values.global.prometheusEnabled }}
 {{ include "common.serviceMonitor" . }}
-{{- end }}
index 5a9d211..42f2a54 100644 (file)
@@ -139,7 +139,6 @@ metrics:
       app.kubernetes.io/managed-by: '{{ .Release.Service }}'
       version: '{{ .Chart.Version | replace "+" "_" }}'
       release: prometheus
-    enabled: true
     port: policy-apex-pdp
     interval: 60s
     isHttps: false
index 8284c73..d261392 100644 (file)
@@ -18,6 +18,4 @@
 # ============LICENSE_END=========================================================
 */}}
 
-{{- if .Values.global.prometheusEnabled }}
 {{ include "common.serviceMonitor" . }}
-{{- end }}
index 562a087..18517fd 100644 (file)
@@ -173,7 +173,6 @@ metrics:
       app.kubernetes.io/managed-by: '{{ .Release.Service }}'
       version: '{{ .Chart.Version | replace "+" "_" }}'
       release: prometheus
-    enabled: true
     port: http
     path: /policy/api/v1/metrics
     interval: 60s
index 8284c73..d261392 100644 (file)
@@ -18,6 +18,4 @@
 # ============LICENSE_END=========================================================
 */}}
 
-{{- if .Values.global.prometheusEnabled }}
 {{ include "common.serviceMonitor" . }}
-{{- end }}
index d2848bb..acf3115 100644 (file)
@@ -169,7 +169,6 @@ metrics:
       app.kubernetes.io/managed-by: '{{ .Release.Service }}'
       version: '{{ .Chart.Version | replace "+" "_" }}'
       release: prometheus
-    enabled: true
     port: policy-distribution
     interval: 60s
     isHttps: false
index f8c450c..c7177f0 100644 (file)
@@ -18,6 +18,4 @@
 # ============LICENSE_END=========================================================
 */}}
 
-{{- if .Values.global.prometheusEnabled }}
 {{ include "common.serviceMonitor" . }}
-{{- end }}
index 1dc1127..4e42525 100644 (file)
@@ -202,7 +202,6 @@ metrics:
       app.kubernetes.io/managed-by: '{{ .Release.Service }}'
       version: '{{ .Chart.Version | replace "+" "_" }}'
       release: prometheus
-    enabled: true
     port: policy-drools-pdp-9696
     interval: 60s
     isHttps: false
index 9677420..e58e71a 100755 (executable)
@@ -18,6 +18,4 @@
 # ============LICENSE_END=========================================================
 */}}
 
-{{- if .Values.global.prometheusEnabled }}
 {{ include "common.serviceMonitor" . }}
-{{- end }}
index d4d77fe..f4a7784 100644 (file)
@@ -212,7 +212,6 @@ metrics:
       app.kubernetes.io/managed-by: '{{ .Release.Service }}'
       version: '{{ .Chart.Version | replace "+" "_" }}'
       release: prometheus
-    enabled: true
     port: http
     interval: 60s
     path: /metrics
index 8284c73..d261392 100644 (file)
@@ -18,6 +18,4 @@
 # ============LICENSE_END=========================================================
 */}}
 
-{{- if .Values.global.prometheusEnabled }}
 {{ include "common.serviceMonitor" . }}
-{{- end }}
index 8370f63..da7a482 100755 (executable)
@@ -186,7 +186,6 @@ metrics:
       app.kubernetes.io/managed-by: '{{ .Release.Service }}'
       version: '{{ .Chart.Version | replace "+" "_" }}'
       release: prometheus
-    enabled: true
     port: http-api
     path: /policy/pap/v1/metrics
     interval: 60s
index 8284c73..d261392 100644 (file)
@@ -18,6 +18,4 @@
 # ============LICENSE_END=========================================================
 */}}
 
-{{- if .Values.global.prometheusEnabled }}
 {{ include "common.serviceMonitor" . }}
-{{- end }}
index 061ceed..ae6aacb 100644 (file)
@@ -195,7 +195,6 @@ metrics:
       app.kubernetes.io/managed-by: '{{ .Release.Service }}'
       version: '{{ .Chart.Version | replace "+" "_" }}'
       release: prometheus
-    enabled: true
     port: http
     path: /metrics
     interval: 60s
index 2f7f141..284c2c3 100644 (file)
@@ -19,7 +19,6 @@
 # Global configuration defaults.
 #################################################################
 global:
-  prometheusEnabled: true
   postgres:
     localCluster: true
     # flag to enable the DB creation via pgo-operator
index c0d9f21..47b84fb 100644 (file)
@@ -1,3 +1 @@
-{{- if .Values.metrics.serviceMonitor.enabled }}
-{{  include "common.serviceMonitor" . }}
-{{- end }}
\ No newline at end of file
+{{  include "common.serviceMonitor" . }}
\ No newline at end of file
index 8e14cc2..09a73fe 100644 (file)
@@ -158,11 +158,8 @@ resources:
 
 metrics:
   serviceMonitor:
-    enabled: true
     targetPort: 8080
     path: /sdc2/rest/actuator/prometheus
-    basicAuth:
-      enabled: false
 
 #Pods Service Account
 serviceAccount:
index 937f5ef..9f1916a 100644 (file)
@@ -14,6 +14,4 @@
 # limitations under the License.
 */}}
 
-{{- if .Values.metrics.serviceMonitor.enabled }}
 {{  include "common.serviceMonitor" . }}
-{{- end }}
index 32342c5..bec446f 100644 (file)
@@ -262,11 +262,8 @@ updateStrategy:
 
 metrics:
   serviceMonitor:
-    enabled: true
     targetPort: 8081
     path: /onboarding-api/v1.0/actuator/prometheus
-    basicAuth:
-      enabled: false
 
 logback:
   rootLogLevel: INFO