Skip to content

Commit 756fc5d

Browse files
authored
update the traffic test for ingress (#3725)
* update the traffic test for ingress * make crds
1 parent f5dee3f commit 756fc5d

File tree

2 files changed

+10
-4
lines changed

2 files changed

+10
-4
lines changed

config/webhook/manifests.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
---
12
apiVersion: admissionregistration.k8s.io/v1
23
kind: MutatingWebhookConfiguration
34
metadata:

test/e2e/ingress/vanilla_ingress_test.go

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -766,10 +766,15 @@ var _ = Describe("vanilla ingress tests", func() {
766766

767767
// test traffic
768768
ExpectLBDNSBeAvailable(ctx, tf, lbARN, lbDNS)
769-
httpExp := httpexpect.New(tf.LoggerReporter, fmt.Sprintf("http://%v", lbDNS))
770-
httpExp.GET("/path").Expect().
771-
Status(http.StatusOK).
772-
Body().Equal("Hello World!")
769+
770+
//TODO: update the traffic test for dualstack-without-public-ipv4 ALB
771+
// as it may need additional setup compared to dualstack ALB
772+
if annotation["alb.ingress.kubernetes.io/ip-address-type"] != "dualstack-without-public-ipv4" {
773+
httpExp := httpexpect.New(tf.LoggerReporter, fmt.Sprintf("http://%v", lbDNS))
774+
httpExp.GET("/path").Expect().
775+
Status(http.StatusOK).
776+
Body().Equal("Hello World!")
777+
}
773778
})
774779
})
775780
})

0 commit comments

Comments
 (0)