Skip to content

Commit

Permalink
increase retry count for TestAPIGateway_GatewayClassConfig test
Browse files Browse the repository at this point in the history
  • Loading branch information
NiniOak committed May 16, 2024
1 parent b547902 commit b7ecab4
Showing 1 changed file with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -133,12 +133,14 @@ func TestAPIGateway_GatewayClassConfig(t *testing.T) {
k8sClient.DeleteAllOf(context.Background(), &gwv1beta1.Gateway{}, client.InNamespace(namespace))
})


Check failure on line 136 in acceptance/tests/api-gateway/api_gateway_gatewayclassconfig_test.go

View workflow job for this annotation

GitHub Actions / golangci-lint

File is not `gofmt`-ed with `-s` (gofmt)

// Ensure it exists.
logger.Log(t, "checking that gateway is synchronized to Consul")
checkConsulExists(t, consulClient, api.APIGateway, gatewayName)

// Scenario: Gateway deployment should match the default instances defined on the gateway class config
logger.Log(t, "checking that gateway instances match defined gateway class config")
// checking that gateway instances match defined gateway class config
checkNumberOfInstances(t, k8sClient, consulClient, gateway.Name, gateway.Namespace, defaultInstances, gateway)

// Scenario: Updating the GatewayClassConfig should not affect gateways that have already been created
Expand Down Expand Up @@ -177,7 +179,7 @@ func scale(t *testing.T, client client.Client, name, namespace string, scaleTo *
func checkNumberOfInstances(t *testing.T, k8client client.Client, consulClient *api.Client, name, namespace string, wantNumber *int32, gateway *gwv1beta1.Gateway) {
t.Helper()

retryCheckWithWait(t, 30, 10*time.Second, func(r *retry.R) {
retryCheckWithWait(t, 40, 10*time.Second, func(r *retry.R) {
logger.Log(t, "checking that gateway instances match defined gateway class config")
logger.Log(t, fmt.Sprintf("want: %d", *wantNumber))

Expand Down

0 comments on commit b7ecab4

Please sign in to comment.