remove retry of Out of capacity error

jotruon · jotruon · commit f08ef5f2194a · 2021-05-25T18:31:32.000Z
diff --git a/oci/retry.go b/oci/retry.go
@@ -139,7 +139,7 @@ func getDefaultExpectedRetryDuration(response oci_common.OCIOperationResponse, d
 		defaultRetryTime = longRetryTime
 	case 500:
 		if e != nil && (strings.Contains(e.Error(), "Out of host capacity")) {
-			return defaultRetryTime
+			return 0
 		}
 		if configuredRetryDuration != nil {
 			return *configuredRetryDuration
diff --git a/oci/retry_test.go b/oci/retry_test.go
@@ -130,7 +130,7 @@ func TestUnitRetryLoop_outOfCapacity(t *testing.T) {
 		httpResponseStatusCode:   500,
 		header:                   map[string][]string{},
 		responseError:            fmt.Errorf("Out of host capacity. "),
-		expectedRetryTimeSeconds: 15,
+		expectedRetryTimeSeconds: 0,
 		jitterMode:               true,
 	}
 	retryLoop(t, &r)

Original file line number	Diff line number	Diff line change
`@@ -139,7 +139,7 @@ func getDefaultExpectedRetryDuration(response oci_common.OCIOperationResponse, d`
`139`	`139`	`defaultRetryTime = longRetryTime`
`140`	`140`	`case 500:`
`141`	`141`	`if e != nil && (strings.Contains(e.Error(), "Out of host capacity")) {`
`142`		`- return defaultRetryTime`
	`142`	`+ return 0`
`143`	`143`	`}`
`144`	`144`	`if configuredRetryDuration != nil {`
`145`	`145`	`return *configuredRetryDuration`
Original file line number	Diff line number	Diff line change
`@@ -130,7 +130,7 @@ func TestUnitRetryLoop_outOfCapacity(t *testing.T) {`
`130`	`130`	`httpResponseStatusCode: 500,`
`131`	`131`	`header: map[string][]string{},`
`132`	`132`	`responseError: fmt.Errorf("Out of host capacity. "),`
`133`		`- expectedRetryTimeSeconds: 15,`
	`133`	`+ expectedRetryTimeSeconds: 0,`
`134`	`134`	`jitterMode: true,`
`135`	`135`	`}`
`136`	`136`	`retryLoop(t, &r)`