[v10.0.x] Unified Alerting: Set max_attempts to 1 by default (#79101)

Unified Alerting: Set `max_attempts` to 1 by default (#79095) * Unified Alerting: Set `max_attempts` to 1 by default The retry logic for unified alerting has been broken as far as v9.4.x, rather than fixing it in one go and causing a headache to our users with rules putting extra load on their datasources - I think a better approach is to simply set 1 as a default and then let our users change it. I see two cons with this approach: - Configuration for legacy to unified alerting cannot be ported over automatically, users will have to manually set `max_attempts` to 3 when migrating. - Users expecting to get any sort of retrying (as with legacy alerting) will not have it out of the box and will have to manually edit the configuration. Signed-off-by: gotjosh <josue.abreu@gmail.com> --------- Signed-off-by: gotjosh <josue.abreu@gmail.com> (cherry picked from commit 0c9356a3c7) Co-authored-by: gotjosh <josue.abreu@gmail.com>
2023-12-05 18:57:37 +00:00
parent b6b8554c5e
commit cb9f9a0bd7
5 changed files with 11 additions and 20 deletions
@@ -1516,7 +1516,7 @@ The timeout string is a possibly signed sequence of decimal numbers, followed by

 ### max_attempts

-Sets a maximum number of times we'll attempt to evaluate an alert rule before giving up on that evaluation. The default value is `3`. This option has a [legacy version in the alerting section]({{< relref "#max_attempts-1" >}}) that takes precedence.
+Sets a maximum number of times we'll attempt to evaluate an alert rule before giving up on that evaluation. The default value is `1`.

 ### min_interval