Sunjay Bhatia

/approve

What version of Contour are you using @qts0312 ? In Contour 1.29.0+ we merged/released some changes to fix the conflict handling when there are identical matches between routes: https://github.com/projectcontour/contour/pull/6188

Yep, I haven't had a lot of bandwidth to implement contour support in ingress2gateway, but if there is a willing contributor I would definitely help shepherd the changes!

this is important to do for 1.19: https://github.com/projectcontour/contour-operator/issues/386#issuecomment-862618737 will also allow us to be smarter about how we handle multiple contours with gateway api

do we still want to transition to using the contour ContourDeployment CRD? if so could be a related change needed to stop using the Contour CRD from the operator, move...

I would love to see some abstraction of the test suite to make setup/teardown etc. really easy @skriss' work on https://github.com/projectcontour/contour/pull/3622 was quite nice

+1 fan of that pattern and reducing duplication

Looks like `HTTPRouteHTTPSListener` is failing, it expects a 404 for request to a HTTPS listener with an unknown hostname Seems like we should either push back on this (expect a...

> Looks like `HTTPRouteHTTPSListener` is failing, it expects a 404 for request to a HTTPS listener with an unknown hostname > > Seems like we should either push back on...

> > Looks like `HTTPRouteHTTPSListener` is failing, it expects a 404 for request to a HTTPS listener with an unknown hostname > > Seems like we should either push back...