VictoriaMetrics / helm-charts

Helm charts for VictoriaMetrics, VictoriaLogs and ecosystem
https://victoriametrics.github.io/helm-charts/
Apache License 2.0
348 stars 332 forks source link

bug: Alertmanager deployment does not respect alertmanager.configMap value #1734

Closed cwstrommer closed 1 week ago

cwstrommer commented 1 week ago

Chart name and version chart: victoria-metrics-alert version: v0.12.4

Describe the bug In our environment we manage the alertmanager routes and receiver configuration externally, and reference it via alertmanager.configMap when we render the Helm chart. Version 0.12.4 retains alertmanager.configMap in the values, but does not respect it because the Deployment spec changed from

        - name: config
          configMap:
            name: {{ include "vmalert.alertmanager.configname" . }}

to

        - name: config
          configMap:
            name: {{ $fullname }}-config

This causes the pod to fail with

  Warning  FailedMount      98s (x53 over 93m)  kubelet       MountVolume.SetUp failed for volume "config" : configmap "victoria-metrics-alert-alertmanager-config" not found

Custom values Please provide only custom values (excluding default ones):

alertmanager:
  podMetadata:
    annotations:
      checksum/external-config: a40ca23bddd5c80ab3cf63c7dcb80844
  configMap: victoria-metrics-alert-external-config
AndrewChubatiuk commented 1 week ago

hey @cwstrommer please try chart 0.12.5, which should contain fixes

cwstrommer commented 1 week ago

hey @cwstrommer please try chart 0.12.5, which should contain fixes

Nice, thank you for the quick response! I'll test it over the next couple of days, you can assume that it's solved if you don't hear back from me.