Skip to content
This repository has been archived by the owner on Feb 27, 2023. It is now read-only.

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

Open
skriss opened this issue Feb 15, 2022 · 2 comments
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@skriss
Copy link
Member

skriss commented Feb 15, 2022

Follow-up to #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.

@skriss skriss added the kind/feature Categorizes issue or PR as related to a new feature. label Feb 15, 2022
@sunjayBhatia
Copy link
Member

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
Copy link
Member Author

skriss commented Feb 15, 2022

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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

No branches or pull requests

2 participants