add dnslookup family to configmap case

Signed-off-by: Clay Kauzlaric <ckauzlaric@vmware.com>
projectcontour · Dec 8, 2023 · ccd8b46 · ccd8b46
1 parent c20f67d
commit ccd8b46
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/test/e2e/httpproxy/httpproxy_test.go b/test/e2e/httpproxy/httpproxy_test.go
@@ -373,18 +373,17 @@ var _ = Describe("HTTPProxy", func() {
 			Context("via Contour ConfigMap", func() {
 				BeforeEach(func() {
 					contourConfig.Cluster.UpstreamTLS.MinimumProtocolVersion = protocolVersion
+
+					// need to set this because it isn't set in the default config
+					contourConfig.Cluster.DNSLookupFamily = "auto"
 				})
 
 				testBackendTLSProtocolVersion(namespace, expectedProtocolVersion)
 			})
 			Context("via ContourConfiguration ", func() {
 				BeforeEach(func() {
-					contourConfiguration.Spec.Envoy.Cluster = &contour_api_v1alpha1.ClusterParameters{
-						UpstreamTLS: &contour_api_v1alpha1.EnvoyTLS{
-							MinimumProtocolVersion: protocolVersion,
-						},
-						// adding this here because it seems to get overwritten if I try to set contourConfiguration.Spec.Envoy.Cluster.UpstreamTLS
-						DNSLookupFamily: contour_api_v1alpha1.AutoClusterDNSFamily,
+					contourConfiguration.Spec.Envoy.Cluster.UpstreamTLS = &contour_api_v1alpha1.EnvoyTLS{
+						MinimumProtocolVersion: protocolVersion,
 					}
 				})
 				testBackendTLSProtocolVersion(namespace, expectedProtocolVersion)