-
Notifications
You must be signed in to change notification settings - Fork 87
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create test for ingress for custom default backend. #145
Create test for ingress for custom default backend. #145
Conversation
5d5e174
to
d56a07d
Compare
c37101b
to
e3ef669
Compare
Add setup for network and subnet, and make sure the cluster is created in the given network and subnet. Add test/cleanup.sh to cleanup resouces. It uses the same value for env in test.
e3ef669
to
34624c1
Compare
I thought the purpose of the code is to test the YAMLs checked in, not to write more e2e tests? |
Do you mean we only want to check if the yaml files are syntactically correct? We don't need to create and validate the resources created by yaml? |
Yes it is getting into a more e2e test format, but we might fail to check many cases if we only check the yaml creates the corresponding objects, so validating traffic would be more a comprehensive check. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Create test for ingress for custom default backend.