From c6d7724b679554fe71b9044e40addc486940f697 Mon Sep 17 00:00:00 2001 From: Matt Moyer Date: Fri, 26 Mar 2021 16:28:33 -0500 Subject: [PATCH] In TestImpersonationProxy, instead of failing in this case just skip the test. Signed-off-by: Matt Moyer --- test/integration/concierge_impersonation_proxy_test.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/test/integration/concierge_impersonation_proxy_test.go b/test/integration/concierge_impersonation_proxy_test.go index 73738600..fb31fe30 100644 --- a/test/integration/concierge_impersonation_proxy_test.go +++ b/test/integration/concierge_impersonation_proxy_test.go @@ -226,9 +226,10 @@ func TestImpersonationProxy(t *testing.T) { //nolint:gocyclo // yeah, it's compl // Auto mode should have decided that the impersonator will be disabled. We need to manually enable it. // However, the cluster does not support load balancers so we should enable it without a load balancer // and use squid to make requests. (e.g. kind) - require.NotEmpty(t, env.Proxy, - "test cluster does not support load balancers but also doesn't have a squid proxy... "+ + if env.Proxy == "" { + t.Skip("test cluster does not support load balancers but also doesn't have a squid proxy... " + "this is not a supported configuration for test clusters") + } // Check that no load balancer has been created by the impersonator's "auto" mode. library.RequireNeverWithoutError(t, func() (bool, error) {