Disable alertmanager URL in ruler if alertmanager is disabled (#618)

* Disable alertmanager URL in ruler if alertmanager is disabled

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>

* Update CHANGELOG.md

Co-authored-by: Tom Hayward <thayward@infoblox.com>
Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>

---------

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
Co-authored-by: Tom Hayward <thayward@infoblox.com>

Author: friedrichg

CHANGELOG.md

@@ -31,6 +31,7 @@
 * [CHANGE] Use default value for num_tokens in ingester configuration #544
 * [BUGFIX] Fix discovering memcached instances #551
 * [BUGFIX] Fix restarting query-schedulers when config changes #559
+* [BUGFIX] Don't configure alertmanager in ruler if alertmanager is disabled #618
 * [ENHANCEMENT] Run overrides-exporter with high availability #557
 * [ENHANCEMENT] Persist subchart #572
 * [ENHANCEMENT] Expose query frontend GRPC port on service for ruler #574

templates/ruler/ruler-dep.yaml

@@ -121,7 +121,7 @@ spec:
           args:
             - "-target=ruler"
             - "-config.file=/etc/cortex/cortex.yaml"
-            {{- if not .Values.config.ruler.alertmanager_url }}
+            {{- if and .Values.alertmanager.enabled (not .Values.config.ruler.alertmanager_url) }}
               {{- if .Values.config.ruler.enable_alertmanager_discovery }}
             - "-ruler.alertmanager-url=http://_http-metrics._tcp.{{ template "cortex.alertmanagerFullname" . }}-headless.{{ .Release.Namespace }}.svc.{{ .Values.clusterDomain }}/api/prom/alertmanager/"
               {{- else }}