Skip to content

Commit 743cd3a

Browse files
committed
wait for catalogsource pod deletion
Signed-off-by: akihikokuroda <[email protected]>
1 parent 4cc28bb commit 743cd3a

File tree

1 file changed

+19
-5
lines changed

1 file changed

+19
-5
lines changed

test/e2e/util_test.go

Lines changed: 19 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -504,11 +504,25 @@ func TearDown(namespace string) {
504504
logf("test resources deleted")
505505
}
506506

507-
func buildCatalogSourceCleanupFunc(crc versioned.Interface, namespace string, catalogSource *operatorsv1alpha1.CatalogSource) cleanupFunc {
507+
func buildCatalogSourceCleanupFunc(c operatorclient.ClientInterface, crc versioned.Interface, namespace string, catalogSource *operatorsv1alpha1.CatalogSource) cleanupFunc {
508508
return func() {
509509
ctx.Ctx().Logf("Deleting catalog source %s...", catalogSource.GetName())
510510
err := crc.OperatorsV1alpha1().CatalogSources(namespace).Delete(context.Background(), catalogSource.GetName(), metav1.DeleteOptions{})
511511
Expect(err).ToNot(HaveOccurred())
512+
513+
if c != nil {
514+
err = wait.Poll(pollInterval, pollDuration, func() (bool, error) {
515+
fetched, err := c.KubernetesInterface().CoreV1().Pods(catalogSource.GetNamespace()).List(context.TODO(), metav1.ListOptions{LabelSelector: "olm.catalogSource=" + catalogSource.GetName()})
516+
if err != nil {
517+
fmt.Println(err)
518+
return true, err
519+
}
520+
if len(fetched.Items) == 0 {
521+
return true, nil
522+
}
523+
return false, nil
524+
})
525+
}
512526
}
513527
}
514528

@@ -548,7 +562,7 @@ func createInvalidGRPCCatalogSource(crc versioned.Interface, name, namespace str
548562
catalogSource, err := crc.OperatorsV1alpha1().CatalogSources(namespace).Create(context.Background(), catalogSource, metav1.CreateOptions{})
549563
Expect(err).ToNot(HaveOccurred())
550564
ctx.Ctx().Logf("Catalog source %s created", name)
551-
return catalogSource, buildCatalogSourceCleanupFunc(crc, namespace, catalogSource)
565+
return catalogSource, buildCatalogSourceCleanupFunc(nil, crc, namespace, catalogSource)
552566
}
553567

554568
func createInternalCatalogSource(c operatorclient.ClientInterface, crc versioned.Interface, name, namespace string, manifests []registry.PackageManifest, crds []apiextensions.CustomResourceDefinition, csvs []operatorsv1alpha1.ClusterServiceVersion) (*operatorsv1alpha1.CatalogSource, cleanupFunc) {
@@ -579,7 +593,7 @@ func createInternalCatalogSource(c operatorclient.ClientInterface, crc versioned
579593

580594
cleanupInternalCatalogSource := func() {
581595
configMapCleanup()
582-
buildCatalogSourceCleanupFunc(crc, namespace, catalogSource)()
596+
buildCatalogSourceCleanupFunc(c, crc, namespace, catalogSource)()
583597
}
584598
return catalogSource, cleanupInternalCatalogSource
585599
}
@@ -616,7 +630,7 @@ func createInternalCatalogSourceWithPriority(c operatorclient.ClientInterface, c
616630

617631
cleanupInternalCatalogSource := func() {
618632
configMapCleanup()
619-
buildCatalogSourceCleanupFunc(crc, namespace, catalogSource)()
633+
buildCatalogSourceCleanupFunc(c, crc, namespace, catalogSource)()
620634
}
621635
return catalogSource, cleanupInternalCatalogSource
622636
}
@@ -650,7 +664,7 @@ func createV1CRDInternalCatalogSource(t GinkgoTInterface, c operatorclient.Clien
650664

651665
cleanupInternalCatalogSource := func() {
652666
configMapCleanup()
653-
buildCatalogSourceCleanupFunc(crc, namespace, catalogSource)()
667+
buildCatalogSourceCleanupFunc(c, crc, namespace, catalogSource)()
654668
}
655669
return catalogSource, cleanupInternalCatalogSource
656670
}

0 commit comments

Comments
 (0)