Skip to content

Commit

Permalink
Make condition_prometheus_query_language.evaluation_interval optional (
Browse files Browse the repository at this point in the history
…#8564)

* Add support for retriggering notifications in AlertPolicy

* Add support for forecast options in AlertPolicy

* Move tests for forecast alerts into the handwritten file

* Add support for PromQL condition type in AlertPolicy

* Added test for promql alerts into the handwritten file

* Add test for condition_prometheus_query_language.labels

* Remove trailing whitespace

* Update mmv1/products/monitoring/AlertPolicy.yaml

* Make condition_prometheus_query_language.evaluation_interval optional

* Remove test for condition_prometheus_query_language.evaluation_interval

---------

Co-authored-by: James Edouard <[email protected]>
Co-authored-by: Cameron Thornton <[email protected]>
  • Loading branch information
3 people authored Aug 8, 2023
1 parent 7eef4d2 commit 1ea0eda
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
1 change: 0 additions & 1 deletion mmv1/products/monitoring/AlertPolicy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -852,7 +852,6 @@ properties:
default value is zero. Must be zero or positive.
- !ruby/object:Api::Type::String
name: evaluationInterval
required: true
description: |
How often this rule should be evaluated. Must be a positive multiple
of 30 seconds or missing. The default value is 30 seconds. If this
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -430,7 +430,6 @@ resource "google_monitoring_alert_policy" "promql" {
condition_prometheus_query_language {
query = "vector(1)"
duration = "60s"
evaluation_interval = "60s"
labels = {
"severity" = "page"
}
Expand Down

0 comments on commit 1ea0eda

Please sign in to comment.