From 4bdf6d51eb523754d92c8a6be0df96bd3912562d Mon Sep 17 00:00:00 2001
From: Lari Hotari <lhotari@users.noreply.github.com>
Date: Mon, 3 Mar 2025 21:09:25 +0200
Subject: [PATCH] Improve kube-prometheus-stack config in values.yaml by adding
 missing key and some basic comments (#579)

* Enable prometheusOperator in CI test

* Add comments and add offloader dashboard
---
 .ci/clusters/values-prometheus-grafana.yaml |  2 ++
 charts/pulsar/values.yaml                   | 11 +++++++++++
 2 files changed, 13 insertions(+)

diff --git a/.ci/clusters/values-prometheus-grafana.yaml b/.ci/clusters/values-prometheus-grafana.yaml
index c7fd6db..f31cf6f 100644
--- a/.ci/clusters/values-prometheus-grafana.yaml
+++ b/.ci/clusters/values-prometheus-grafana.yaml
@@ -19,6 +19,8 @@
 
 kube-prometheus-stack:
   enabled: true
+  prometheusOperator:
+    enabled: true
   prometheus:
     enabled: true
   grafana:
diff --git a/charts/pulsar/values.yaml b/charts/pulsar/values.yaml
index 823ff66..9960095 100755
--- a/charts/pulsar/values.yaml
+++ b/charts/pulsar/values.yaml
@@ -1378,9 +1378,15 @@ toolset:
 ## https://github.com/prometheus-community/helm-charts/tree/main/charts/kube-prometheus-stack
 ## For sample values, please see their documentation.
 kube-prometheus-stack:
+  ## Enable the kube-prometheus-stack chart
   enabled: true
+  ## Manages Prometheus and Alertmanager components
+  prometheusOperator:
+    enabled: true
+  ## Prometheus component
   prometheus:
     enabled: true
+  ## Grafana component
   grafana:
     enabled: true
     # Use random password at installation time for Grafana by default by setting empty value to `adminPassword`.
@@ -1451,10 +1457,15 @@ kube-prometheus-stack:
         zookeeper:
           url: https://raw.githubusercontent.com/streamnative/apache-pulsar-grafana-dashboard/master/dashboards.kubernetes/zookeeper-3.6.json
           datasource: Prometheus
+        offloader:
+          url: https://raw.githubusercontent.com/apache/pulsar/refs/heads/master/grafana/dashboards/offloader.json
+          datasource: Prometheus
+  ## Prometheus node exporter component
   prometheus-node-exporter:
     enabled: true
     hostRootFsMount:
       enabled: false
+  ## Alertmanager component
   alertmanager:
     enabled: false