azure-metrics-exporter: The fix for namespaceSelector breaks the deployment

[ThomasMinor]

The addition of a namespaceSelector breaks the deployment in our environment throwing a nil pointer error:

Error: template: azure-metrics-exporter/templates/prometheus/servicemonitor.metricprobe.yaml:23:16: executing "azure-metrics-exporter/templates/prometheus/servicemonitor.metricprobe.yaml" at <.Values.prometheus.monitor.namespaceSelector>: nil pointer evaluating interface {}.prometheus

[ThomasMinor]

Is addressed by #62

[Moglum]

#62 fixed only part of the problem. We're still getting:

Error: template: azure-metrics-exporter/templates/_helpers.tpl:36:16: executing "azure-metrics-exporter.namespace" at <.Values.namespaceOverride>: nil pointer evaluating interface {}.namespaceOverride

It's caused by passing the local scope instead of $root here https://github.com/webdevops/helm-charts/blob/main/charts/azure-metrics-exporter/templates/prometheus/servicemonitor.metricprobe.yaml#L28

I'll open a PR to fix it shortly.

[x418]

#62 fixed only part of the problem. We're still getting:

Error: template: azure-metrics-exporter/templates/_helpers.tpl:36:16: executing "azure-metrics-exporter.namespace" at <.Values.namespaceOverride>: nil pointer evaluating interface {}.namespaceOverride

It's caused by passing the local scope instead of $root here https://github.com/webdevops/helm-charts/blob/main/charts/azure-metrics-exporter/templates/prometheus/servicemonitor.metricprobe.yaml#L28

I'll open a PR to fix it shortly.

My bad. I saw the comment too late (after the merge). Thanks for taking care of it