[prometheus-kafka-exporter] Use helm test hooks (prometheus-community…

…#498)

* Use helm test hooks

Signed-off-by: Minutis <minutis@gmail.com>

* Change order of pods

Signed-off-by: Minutis <minutis@exacaster.com>

* Get zookeeper pod to start

Signed-off-by: Minutis <minutis@exacaster.com>

* Remove kafka/zookeeper pods
Use dependencies

Signed-off-by: Minutis <minutis@exacaster.com>

* Add kafka repo

Signed-off-by: Minutis <minutis@exacaster.com>

* Add kafka repo

Signed-off-by: Minutis <minutis@exacaster.com>

* Update apiVersion

Signed-off-by: Minutis <minutis@exacaster.com>

* Comment kafka.enabled value

Signed-off-by: Minutis <minutis@exacaster.com>

* Disable kafka by default
Change version to 1.0.0
Rename kafka repo to bitnami
Enable kafka for CI

Signed-off-by: Minutis <minutis@exacaster.com>

* Disable serviceAccount creation for CI

Signed-off-by: Minutis <minutis@exacaster.com>

* update ci

Signed-off-by: André Bauer <monotek23@gmail.com>

* use ubuntu 20.04

Signed-off-by: André Bauer <monotek23@gmail.com>

* set ct.yaml path

Signed-off-by: André Bauer <monotek23@gmail.com>

* moved kafka dependency to the bottom / resort some values / add linebreaks

Signed-off-by: André Bauer <monotek23@gmail.com>

* removed linebreak

Signed-off-by: André Bauer <monotek23@gmail.com>

* move kube state metrics back so we have no change in ct.yaml

Signed-off-by: André Bauer <monotek23@gmail.com>

Co-authored-by: Minutis <minutis@exacaster.com>
Co-authored-by: André Bauer <monotek23@gmail.com>

charts/prometheus-kafka-exporter/Chart.yaml

@@ -1,9 +1,9 @@
-apiVersion: v1
+apiVersion: v2
 appVersion: "v1.2.0"
 description: A Helm chart to export the metrics from Kafka in Prometheus format using the kafka-exporter from https://github.com/danielqsj/kafka_exporter
 name: prometheus-kafka-exporter
 home: https://github.com/danielqsj/kafka_exporter
-version: 0.2.1
+version: 1.0.0
 sources:
   - https://gkarthiks.github.io/helm-charts/charts/prometheus-kafka-exporter
 keywords:
@@ -19,3 +19,8 @@ annotations:
   "artifacthub.io/links": |
     - name: Chart Source
       url: https://github.com/prometheus-community/helm-charts/tree/main/charts/prometheus-kafka-exporter
+dependencies:
+- name: kafka
+  version: "12.6.3"
+  repository: https://charts.bitnami.com/bitnami
+  condition: kafka.enabled

charts/prometheus-kafka-exporter/ci/ci-values.yaml

@@ -0,0 +1,10 @@
+kafkaServer:
+  - ci-kafka:9092
+
+kafka:
+  enabled: true
+  fullnameOverride: ci-kafka
+
+  zookeeper:
+    enabled: true
+    fullnameOverride: ci-zookeeper

charts/prometheus-kafka-exporter/values.yaml

@@ -1,24 +1,26 @@
+image:
+  repository: danielqsj/kafka-exporter
+  tag: latest
+  pullPolicy: IfNotPresent
+
+replicaCount: 1
+
+kafkaServer:
+  - kafka-server:9092
+
 rbac:
   # Specifies whether RBAC resources should be created
   create: true
   # Specifies whether a PodSecurityPolicy should be created
   pspEnabled: true
+
 serviceAccount:
   # Specifies whether a ServiceAccount should be created
   create: true
   # The name of the ServiceAccount to use.
   # If not set and create is true, a name is generated using the fullname template
   name:
 
-replicaCount: 1
-image:
-  repository: danielqsj/kafka-exporter
-  tag: latest
-  pullPolicy: IfNotPresent
-
-kafkaServer:
-  - kafka-server:9092
-
 env: []
 
 service:
@@ -32,6 +34,7 @@ liveness:
     httpGet:
       path: /
       port: exporter-port
+
 readiness:
   enabled: false
   probe:
@@ -79,3 +82,8 @@ tls:
   enabled: false
   # mountPath: /secret/certs
   # secretName: <name of an existing secret>
+
+# If enabled Kafka dependency chart will be used.
+# This is only needed for the CI job so it should always be disabled.
+kafka:
+  enabled: false