From f31c478cbc2488d1d5f38510914d395e807bd595 Mon Sep 17 00:00:00 2001 From: Chi Zhang Date: Mon, 23 Sep 2019 09:59:14 -0700 Subject: [PATCH] minor changes (#713) --- testutils/clustermanager/gke.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/testutils/clustermanager/gke.go b/testutils/clustermanager/gke.go index 4c881fe8a..c66557486 100644 --- a/testutils/clustermanager/gke.go +++ b/testutils/clustermanager/gke.go @@ -320,7 +320,7 @@ func (gc *GKECluster) Acquire() error { } // Retry another region if cluster creation failed. // TODO(chaodaiG): catch specific errors as we know what the error look like for stockout etc. - if len(regions) != i+1 { + if i != len(regions)-1 { errMsg = fmt.Sprintf("%sRetry another region %q for cluster creation", errMsg, regions[i+1]) } log.Printf(errMsg) @@ -430,8 +430,6 @@ func (gc *GKECluster) wait(location, opName string, wait time.Duration) error { } } } - - return err } // ensureProtected ensures not operating on protected project/cluster