What is the purpose of this namespace value in the monitoring sub chart?

[markussiebert]

Just wonding what I can configure with this option, or what was the intention to configure a namespace in 2 places:

policy-reporter/charts/policy-reporter/charts/monitoring/values.yaml

Line 2 in bcd9e32

namespace: cattle-dashboards

and then the online place it is referenced is here:

policy-reporter/charts/policy-reporter/charts/monitoring/templates/_helpers.tpl

Lines 44 to 51 in bcd9e32

	{{- define "monitoring.namespace" -}}
	{{- if .Values.grafana.namespace -}}
	{{- .Values.grafana.namespace -}}
	{{- else if .Values.namespace -}}
	{{- .Values.namespace -}}
	{{- else -}}
	{{- .Release.Namespace -}}
	{{- end }}

So is this redundant to grafana.namespace setting? At the moment I dont understand it's purpose, so maybe I am using it wrong.

[fjogeleit]

Hey, thanks for your question. It's redundant. The namespace option was the initial one, with increasing configuration possibilities it was also added under Grafana. I'm working currently on a cleanup which will remove the namespace setting. It's still in for backward compatibility.

[fjogeleit]

If there is any question left feel free to reopen the issue