search

projectcontour / contour-operator

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

drop contour.spec.namespace, create resources in the same namespace as the Contour CR #494

Open skriss opened 2 years ago

skriss commented 2 years ago

Follow-up to https://github.com/projectcontour/contour-operator/pull/382

We have clarified our policy around alpha APIs to allow fields to be dropped without revving the API version (this diverges slightly from upstream). As such, we are free to go ahead and drop the namespace field from the Contour spec and always create resources in the same namespace as the Contour CR itself, to simplify namespace management.

sunjayBhatia commented 2 years ago

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 the required fields over, etc.

skriss commented 2 years ago

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 the required fields over, etc.

Good question, not sure of the answer but agree that if we are going to do that migration then we should make this change as part of that.