Skip to content

Commit f5a4255

Browse files
authored
Merge pull request #2438 from k8s-infra-cherrypick-robot/cherry-pick-2436-to-release-1.31
[release-1.31] test: fix GetAccountNumByResourceGroup timeout
2 parents 6bac81e + b54a490 commit f5a4255

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

test/e2e/suite_test.go

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -270,14 +270,17 @@ func execTestCmd(cmds []testCmd) {
270270
}
271271
}
272272

273-
func checkAccountCreationLeak(ctx context.Context) {
273+
func checkAccountCreationLeak(_ context.Context) {
274274
creds, err := credentials.CreateAzureCredentialFile(false)
275275
gomega.Expect(err).NotTo(gomega.HaveOccurred())
276276
azureClient, err := azure.GetAzureClient(creds.Cloud, creds.SubscriptionID, creds.AADClientID, creds.TenantID, creds.AADClientSecret, creds.AADFederatedTokenFile)
277277
gomega.Expect(err).NotTo(gomega.HaveOccurred())
278278

279-
accountNum, err := azureClient.GetAccountNumByResourceGroup(ctx, creds.ResourceGroup)
280-
framework.ExpectNoError(err, fmt.Sprintf("failed to GetAccountNumByResourceGroup(%s): %v", creds.ResourceGroup, err))
279+
accountNum, err := azureClient.GetAccountNumByResourceGroup(context.Background(), creds.ResourceGroup)
280+
if err != nil {
281+
framework.Logf("failed to GetAccountNumByResourceGroup(%s): %v", creds.ResourceGroup, err)
282+
return
283+
}
281284
ginkgo.By(fmt.Sprintf("GetAccountNumByResourceGroup(%s) returns %d accounts", creds.ResourceGroup, accountNum))
282285

283286
accountLimitInTest := 15

0 commit comments

Comments
 (0)