projectcontour / contour-operator

Experimental repository to explore an operator for deploying Contour
Apache License 2.0
43 stars 34 forks source link

Should Operator check for Gateway API CRDs before configuring a Contour with Gateway Controller Name? #426

Open sunjayBhatia opened 3 years ago

sunjayBhatia commented 3 years ago

This logic existed before b/c the operator had GatewayClass/Gateway controllers but they no longer exist.

Options:

The second option seems like an option if there is a good way for users to see why their Contour isn't being created, beyond having to trawl through Operator logs. Seems like it might get complicated and cause conflict if the Contour config CRD is moved to the Contour repo/API.

nak3 commented 3 years ago

+1 for "Set status on Contour config object if they don't exist".

And not only the existing check, but also setting status for the unsupported gateway-api CRDs version might be useful.

youngnick commented 3 years ago

I agree, setting status on the Contour object seems like the go here.