Use http constants and use switch instead of if

mpalmer · Feb 10, 2021 · 50a6732 · 50a6732
1 parent bcd60c3
commit 50a6732
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/builder/oracle/oci/driver_oci.go b/builder/oracle/oci/driver_oci.go
@@ -6,6 +6,7 @@ import (
 	"fmt"
 	"math"
 	"math/rand"
+	"net/http"
 	"regexp"
 	"sync/atomic"
 	"time"
@@ -28,7 +29,8 @@ var retryPolicy = &common.RetryPolicy{
 	ShouldRetryOperation: func(res common.OCIOperationResponse) bool {
 		var e common.ServiceError
 		if errors.As(res.Error, &e) {
-			if e.GetHTTPStatusCode() == 429 || e.GetHTTPStatusCode() == 500 || e.GetHTTPStatusCode() == 503 {
+			switch e.GetHTTPStatusCode() {
+			case http.StatusTooManyRequests, http.StatusInternalServerError, http.StatusServiceUnavailable:
 				return true
 			}
 		}