-
Notifications
You must be signed in to change notification settings - Fork 28
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
Kingress controller with Gateway APIs #12
Comments
You mean that the controller to create from Gateway API from KIngress, correct? If so, we will not create it.
|
I read the thread again. sounds like we don't need the ingress v2 controller. Then about the phases, here is what I read from the chat thread:
I am not very sure what we plan to do about "Create new networking conformance suite by Gateway API resources" mentioned in above comment. Do we plan to add conformance test suites within net-ingressv2 to cover all kingress scenarios, or only basic test suites like https://github.com/knative-sandbox/net-ingressv2/pull/10/files#diff-dd6583ae8832b447f778694426e7a1a3f9da8de2ce0991ebfaad75935c87d6d5 for basic sanity check? Could we just skip this phase considering that we may want to run all E2E tests with makeIngressv2 enabled for all impls? |
Thought about this again. I think the value of the work |
Yes, not only handy conformance check for the ingress v2 providers, but also serving's e2e tests do not cover all networking features so we need the "Create new networking conformance suite by Gateway API resources" which can cover current KIngress tests by ingress v2. Also, we plan to add all conformance tests. I just added basic test only here #10 because creating all test suites needs "man-power". I think I just add a basic test and common utils in #10, then will ask some volunteers to add other tests. |
This issue is stale because it has been open for 90 days with no |
We need to implement Kingress controller with Gateway APIs so that Knative could consume Gateway APIs.
/cc @nak3 I think we still need it as the intermediate step?
The text was updated successfully, but these errors were encountered: