rollback appProtocol change

Kong · Feb 2, 2024 · 4c4c8b1 · 4c4c8b1
1 parent f14aeb0
commit 4c4c8b1
Showing 1 changed file with 0 additions and 6 deletions.
diff --git a/test/e2e/helpers_test.go b/test/e2e/helpers_test.go
@@ -411,12 +411,6 @@ func deployIngressWithEchoBackends(ctx context.Context, t *testing.T, env enviro
 
 	t.Logf("exposing deployment %s via service", deployment.Name)
 	service := generators.NewServiceForDeployment(deployment, corev1.ServiceTypeClusterIP)
-	for i := range service.Spec.Ports {
-		// Set Service's appProtocol to http so that Kuma load-balances requests on HTTP-level instead of TCP.
-		// TCP load-balancing proved to cause issues with not distributing load evenly in short time spans like in this test.
-		// Ref: https://github.com/Kong/kubernetes-ingress-controller/issues/5498
-		service.Spec.Ports[i].AppProtocol = lo.ToPtr("http")
-	}
 	_, err = env.Cluster().Client().CoreV1().Services(corev1.NamespaceDefault).Create(ctx, service, metav1.CreateOptions{})
 	require.NoError(t, err)
 	t.Cleanup(func() {