Operator chart should consider deployment into existing namespaces

GoogleCloudPlatform / flink-on-k8s-operator

[DEPRECATED] Kubernetes operator for managing the lifecycle of Apache Flink and Beam applications.

Apache License 2.0

658 stars 266 forks source link

Operator chart should consider deployment into existing namespaces #347

Closed yaron-idan closed 3 years ago

yaron-idan commented 3 years ago

I'm trying to deploy the flink operator into an existing namespace within the cluster using the helm chart in this repository. Since it currently doesn't allow to skip namespace creation, I end up with a collision between the namespace I've already created and the one defined by the template.

WDYT about adding a flag allowing to skip namespace creation? I can create the PR if this feels like a valuable contribution.

functicons commented 3 years ago

This is definitely reasonable, looking forward to your PR! Thanks!

functicons commented 3 years ago

https://github.com/GoogleCloudPlatform/flink-on-k8s-operator/pull/348