add retry for source client in serverside apply e2e test. (#208)

Signed-off-by: morvencao <[email protected]>
openshift-online · Oct 28, 2024 · 1de63c6 · 1de63c6
1 parent 3c09e09
commit 1de63c6
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/test/e2e/pkg/serverside_test.go b/test/e2e/pkg/serverside_test.go
@@ -121,8 +121,10 @@ var _ = Describe("Server Side Apply", Ordered, Label("e2e-tests-serverside-apply
 		nestedWorkNamespace := "default"
 
 		work := NewNestedManifestWork(nestedWorkNamespace, workName, nestedWorkName)
-		_, err := sourceWorkClient.ManifestWorks(agentTestOpts.consumerName).Create(ctx, work, metav1.CreateOptions{})
-		Expect(err).ShouldNot(HaveOccurred())
+		Eventually(func() error {
+			_, err := sourceWorkClient.ManifestWorks(agentTestOpts.consumerName).Create(ctx, work, metav1.CreateOptions{})
+			return err
+		}, 5*time.Minute, 5*time.Second).ShouldNot(HaveOccurred())
 
 		// make sure the nested work is created
 		Eventually(func() error {
@@ -148,7 +150,7 @@ var _ = Describe("Server Side Apply", Ordered, Label("e2e-tests-serverside-apply
 			return nil
 		}, 1*time.Minute, 1*time.Second).Should(BeNil())
 
-		err = sourceWorkClient.ManifestWorks(agentTestOpts.consumerName).Delete(ctx, workName, metav1.DeleteOptions{})
+		err := sourceWorkClient.ManifestWorks(agentTestOpts.consumerName).Delete(ctx, workName, metav1.DeleteOptions{})
 		Expect(err).ShouldNot(HaveOccurred())
 	})
 })