From e31b2080f5aab80105b05b855d8c45cfe9a98c1d Mon Sep 17 00:00:00 2001 From: Antonio Ojea Date: Thu, 25 May 2023 11:20:08 +0000 Subject: [PATCH] e2e framework retry on Service unavailable errors the e2e framwork use active loops to wait for certain async operations, these loops need to retry on some operations and fail in others. For the functions that depend on some operations to happen, the apiserver may return 503 errors until that specific service is available, so we should retry on those too. Change-Id: Ib3d194184f6385b9d3d151c7055f27c97c21c3ff --- test/e2e/framework/get.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/test/e2e/framework/get.go b/test/e2e/framework/get.go index ffa26d3078d..1e1a83856d6 100644 --- a/test/e2e/framework/get.go +++ b/test/e2e/framework/get.go @@ -100,7 +100,10 @@ func ShouldRetry(err error) (retry bool, retryAfter time.Duration) { } // these errors indicate a transient error that should be retried. - if apierrors.IsTimeout(err) || apierrors.IsTooManyRequests(err) || errors.As(err, &transientError{}) { + if apierrors.IsTimeout(err) || + apierrors.IsTooManyRequests(err) || + apierrors.IsServiceUnavailable(err) || + errors.As(err, &transientError{}) { return true, 0 }