Skip to content

Commit 15b7dc1

Browse files
committed
Fix failing webhook e2e test
This commit introduces a change to the webhook tests that rely on a grpc catalogSource that is stored in an off cluster registry. These tests originally required that the following happen within a minute: 1. The catalogSource is installed and ready to serve. 2. The contents of the bundle are unpacked and placed in an installPlan. 3. The CSV is installed and in the ready state. The tests will now wait for the catalogSource to be ready to serve before moving onto steps 2 and 3.
1 parent 23d84fa commit 15b7dc1

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

Diff for: test/e2e/webhook_e2e_test.go

+4
Original file line numberDiff line numberDiff line change
@@ -673,6 +673,10 @@ var _ = Describe("CSVs with a Webhook", func() {
673673
require.NoError(GinkgoT(), crc.OperatorsV1alpha1().CatalogSources(source.GetNamespace()).Delete(context.TODO(), source.GetName(), metav1.DeleteOptions{}))
674674
}
675675

676+
// Wait for the CatalogSource to be ready
677+
_, err = fetchCatalogSourceOnStatus(crc, source.GetName(), source.GetNamespace(), catalogSourceRegistryPodSynced)
678+
require.NoError(GinkgoT(), err)
679+
676680
// Create a Subscription for the webhook-operator
677681
subscriptionName := genName("sub-")
678682
cleanupSubscription := createSubscriptionForCatalog(crc, testNamespace, subscriptionName, source.GetName(), packageName, channelName, "", operatorsv1alpha1.ApprovalAutomatic)

0 commit comments

Comments
 (0)