Compare commits

...

4 Commits

Author SHA1 Message Date
Michel Hollands
a14b07239b Add check for empty list of namespaces to monitor
Signed-off-by: Michel Hollands <michel.hollands@grafana.com>
2023-07-06 14:11:08 +01:00
Michel Hollands
116586e383 Add check to if anything is enabled
Signed-off-by: Michel Hollands <michel.hollands@grafana.com>
2023-07-06 14:08:15 +01:00
Michel Hollands
67b13388a7 Add validation.yaml
Signed-off-by: Michel Hollands <michel.hollands@grafana.com>
2023-07-06 11:58:00 +01:00
Michel Hollands
39a187457c Merge pull request #2 from grafana/enable_dbs_individually
Enable DBs separately
2023-06-30 12:45:08 +01:00
3 changed files with 44 additions and 12 deletions

View File

@@ -42,7 +42,4 @@ dependencies:
- name: minio
repository: https://charts.min.io
version: "5.0.11"
tags:
- local-logs
- local-metrics
- local-traces
condition: local.minio.enabled

View File

@@ -0,0 +1,39 @@
{{- if not (eq .Values.local.minio.enabled (or .Values.local.traces.enabled (or .Values.local.logs.enabled .Values.local.metrics.enabled))) -}}
{{- fail "minio.enabled should be set to true if any of the the local databases is enabled " -}}
{{- end -}}
{{- if eq .Values.cloud.logs.enabled true -}}
{{- if or (empty .Values.cloud.logs.endpoint) (or (empty .Values.cloud.logs.username) (empty .Values.cloud.logs.password)) -}}
{{- fail "if cloud.logs is enabled then the endpoint, username and password have to be filled in" -}}
{{- end -}}
{{- end -}}
{{- if eq .Values.cloud.metrics.enabled true -}}
{{- if or (empty .Values.cloud.metrics.endpoint) (or (empty .Values.cloud.metrics.username) (empty .Values.cloud.metrics.password)) -}}
{{- fail "if cloud.metrics is enabled then the endpoint, username and password have to be filled in" -}}
{{- end -}}
{{- end -}}
{{- if eq .Values.cloud.traces.enabled true -}}
{{- if or (empty .Values.cloud.traces.endpoint) (or (empty .Values.cloud.traces.username) (empty .Values.cloud.traces.password)) -}}
{{- fail "if cloud.traces is enabled then the endpoint, username and password have to be filled in" -}}
{{- end -}}
{{- end -}}
{{- if eq .Values.cloud.traces.enabled false -}}
{{- if eq .Values.cloud.metrics.enabled false -}}
{{- if eq .Values.cloud.logs.enabled false -}}
{{- if eq .Values.local.traces.enabled false -}}
{{- if eq .Values.local.metrics.enabled false -}}
{{- if eq .Values.local.logs.enabled false -}}
{{- fail "nothing has been enabled" -}}
{{- end -}}
{{- end -}}
{{- end -}}
{{- end -}}
{{- end -}}
{{- end -}}
{{- if empty .Values.namespacesToMonitor -}}
{{- fail "No namespaces have been specified in namespacesToMonitor" -}}
{{- end -}}

View File

@@ -2,15 +2,8 @@ namespacesToMonitor:
- loki
- mimir
- tempo
clusterName: "observability" # TODO check if this can be derived
clusterName: "meta-monitoring" # TODO check if this can be derived
# Turn on or off for local storing of metrics, logs or traces
tags:
local-logs: true
local-metrics: true
local-traces: true
# Unfortunately this is duplicate as Helm's tags and conditions fro dependencies are limited
local:
logs:
enabled: false
@@ -18,6 +11,9 @@ local:
enabled: false
traces:
enabled: false
minio:
enabled: false # This should be set to true if any of the previous is enabled
cloud:
logs: