Skip to content

Commit

Permalink
test/extended/prometheus: allow a pending state for AlertmanagerRecei…
Browse files Browse the repository at this point in the history
…versNotConfigured
  • Loading branch information
paulfantom committed May 4, 2020
1 parent 6c88aa2 commit 5683b9f
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions test/extended/prometheus/prometheus.go
Original file line number Diff line number Diff line change
Expand Up @@ -291,7 +291,7 @@ var _ = g.Describe("[sig-instrumentation] Prometheus", func() {
}
o.Expect(insecureTargets).To(o.BeEmpty(), "some services expose metrics over insecure channel")
})
g.It("should have a AlertmanagerReceiversNotConfigured alert in firing state", func() {
g.It("should have a AlertmanagerReceiversNotConfigured alert", func() {
oc.SetupProject()
ns := oc.Namespace()
execPod := exutil.CreateCentosExecPodOrFail(oc.AdminKubeClient(), ns, "execpod", nil)
Expand All @@ -300,8 +300,7 @@ var _ = g.Describe("[sig-instrumentation] Prometheus", func() {
}()

tests := map[string]bool{
// should have constantly firing a watchdog alert
`ALERTS{alertstate="firing",alertname="AlertmanagerReceiversNotConfigured"} == 1`: true,
`ALERTS{alertstate=~"firing|pending",alertname="AlertmanagerReceiversNotConfigured"} == 1`: true,
}
helper.RunQueries(tests, oc, ns, execPod.Name, url, bearerToken)

Expand Down

0 comments on commit 5683b9f

Please sign in to comment.