canonical / seldon-core-operator

Seldon Core Operator
Apache License 2.0
5 stars 9 forks source link

tests: add a retry when asserting the up metric (#243) #245

Closed DnPlas closed 5 months ago

DnPlas commented 5 months ago

Adding a retry for checking the state of an alert will allow time to prometheus-k8s to scrape the necessary metrics for a unit, without it we may run into a race condition where the assertion of the metric is run before prometheus is even able to scrape. This commit adds a retry logic to avoid this.

Fixes #244