Unverified Commit cf44d02b authored by Keshav Prasad's avatar Keshav Prasad Committed by GitHub
Browse files

fix: escape alert varaible in helm (#3171)

* fix: escape alert varaible in helm
* fix: rename rule
Showing with 2 additions and 2 deletions
+2 -2
......@@ -5,7 +5,7 @@ metadata:
role: alert-rules
app: {{ .Values.prometheus_rule_selector_app }}
release: {{ .Values.prometheus_rule_selector_release }}
name: {{ .Values.fullnameOverride }}-api-response-rules
name: {{ .Values.fullnameOverride }}-opa-403-errors
namespace: {{ .Values.namespace }}
spec:
groups:
......@@ -13,7 +13,7 @@ spec:
rules:
- alert: 403_errors_spiking_due_to_opa
annotations:
message: There is an upward trend in 403 API errors due to OPA for {{$labels.job}}
message: {{`'There is an upward trend in 403 API errors due to OPA for {{$labels.job}}'`}}
expr: (sum(rate(envoy_cluster_upstream_rq{envoy_response_code="403",envoy_cluster_name="service"}[5m])) by(job, envoy_response_code) / ignoring(envoy_response_code) group_left sum(rate(envoy_cluster_upstream_rq_total{envoy_cluster_name="service"}[5m])) by(job)) > 0.05
for: 5m
labels:
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment