Skip to content

Commit

Permalink
Add Available IngressController Check to TestScopeChange
Browse files Browse the repository at this point in the history
  • Loading branch information
gcs278 committed Nov 11, 2024
1 parent 34c2e67 commit 4f8b44e
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion test/e2e/operator_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -1566,7 +1566,7 @@ func TestScopeChange(t *testing.T) {
Status: operatorv1.ConditionFalse,
}
if err := waitForIngressControllerCondition(t, kclient, 1*time.Minute, name, progressingFalse); err != nil {
t.Fatalf("failed to observe the ingresscontroller report Progressing=True: %v", err)
t.Fatalf("failed to observe the ingresscontroller report Progressing=False: %v", err)
}
case configv1.AzurePlatformType, configv1.GCPPlatformType:
err := wait.PollImmediate(5*time.Second, 5*time.Minute, func() (bool, error) {
Expand Down Expand Up @@ -1624,6 +1624,12 @@ func TestScopeChange(t *testing.T) {
if err != nil {
t.Fatalf("expected load balancer to become internal: %v", err)
}

// Verify that the IngressController provisions successfully to identify bugs like OCPBUGS-32776.
if err := waitForIngressControllerCondition(t, kclient, 5*time.Minute, name, availableNotProgressingConditionsForIngressControllerWithLoadBalancer...); err != nil {
t.Fatalf("failed to observe expected conditions: %v", err)
}

}

// TestNodePortServiceEndpointPublishingStrategy creates an ingresscontroller
Expand Down

0 comments on commit 4f8b44e

Please sign in to comment.