test: AWS and Azure should tolerate no API disruption
4.7 brought us to a no disruption on upgrade state for the control
plane. In 4.8 lock that requirement in so that future changes do
not regress us further.
@@ -106,7 +106,16 @@ func (t *availableTest) Test(f *framework.Framework, done <-chan struct{}, upgra
cancel()
end := time.Now()
- disruption.ExpectNoDisruption(f, 0.08, end.Sub(start), m.Events(time.Time{}, time.Time{}), fmt.Sprintf("API %q was unreachable during disruption", t.name))
+ // AWS and Azure in 4.7 reached the point of having no disruption and thus
+ // now lock in that requirement. GCP has a known issue with health checking
+ // https://bugzilla.redhat.com/show_bug.cgi?id=1925698 that when fixed will
+ // relax. Having two providers lock in the requirement ensures we block
+ // regression and ratchet
+ toleratedDisruption := 0.08
+ if framework.ProviderIs("aws", "azure") {
+ toleratedDisruption = 0
+ }
+ disruption.ExpectNoDisruption(f, toleratedDisruption, end.Sub(start), m.Events(time.Time{}, time.Time{}), fmt.Sprintf("API %q was unreachable during disruption", t.name))
}
// Teardown cleans up any remaining resources.
