fix: Use eventually when updating component (#979)

Use eventually when updating component
konflux-ci · Dec 22, 2023 · 442c70c · 442c70c
1 parent 933fcc3
commit 442c70c
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/tests/build/build.go b/tests/build/build.go
@@ -516,7 +516,14 @@ var _ = framework.BuildSuiteDescribe("Build service E2E tests", Label("build", "
 
 			It("After updating image visibility to private, it should not trigger another PipelineRun", func() {
 				Expect(f.AsKubeAdmin.TektonController.DeleteAllPipelineRunsInASpecificNamespace(testNamespace)).To(Succeed())
-				Expect(f.AsKubeAdmin.HasController.SetComponentAnnotation(componentName, controllers.ImageRepoGenerateAnnotationName, constants.ImageControllerAnnotationRequestPrivateRepo[controllers.ImageRepoGenerateAnnotationName], testNamespace)).To(Succeed())
+				Eventually(func() error {
+					err := f.AsKubeAdmin.HasController.SetComponentAnnotation(componentName, controllers.ImageRepoGenerateAnnotationName, constants.ImageControllerAnnotationRequestPrivateRepo[controllers.ImageRepoGenerateAnnotationName], testNamespace)
+					if err != nil {
+						GinkgoWriter.Printf("failed to update the component %s with error %v\n", componentName, err)
+						return err
+					}
+					return nil
+				}, time.Second*20, time.Second*1).Should(Succeed(), fmt.Sprintf("timed out when trying to update the component %s/%s", testNamespace, componentName))
 
 				Consistently(func() bool {
 					componentPipelineRun, _ := f.AsKubeAdmin.HasController.GetComponentPipelineRun(componentName, applicationName, testNamespace, "")