search

percona / percona-helm-charts

Collection of Helm charts for Percona Kubernetes Operators.
https://www.percona.com/software/percona-kubernetes-operators
Other
120 stars 165 forks source link

Make namespace creation as optional #292

Open sumeet-zuora opened 8 months ago

sumeet-zuora commented 8 months ago

Can we make the creation of namespace as optional at https://github.com/percona/percona-helm-charts/blob/main/charts/psmdb-operator/templates/namespace.yaml as we might already have namespace created from different automation

spron-in commented 8 months ago

Hello @sumeet-zuora ,

yeah, we have it in our plans and it might be delivered under https://perconadev.atlassian.net/browse/K8SPSMDB-930

sumeet-zuora commented 8 months ago

Can this be also added to pxc, pg and ps operator seems all have same issue

tplavcic commented 8 months ago

@sumeet-zuora From what I see pxc is affected and I will create a PR for it, ps and pg don't seem to be affected since they don't create namespaces (but correct me if I'm wrong and don't see something).

sumeet-zuora commented 8 months ago

In ps the watch namespace is only for where it is deployed, pg watch namespace have proper variable.

Role and rolebindings have mentioned the watch namespace, but values.yaml don't have it defined.

In both, it is expected as string or single namespace. Where as I think in psmdb it is multiple namespaces separated by comma.

abh commented 1 month ago

in ps-operator it'd be nice if it'd be possible to set the WATCH_NAMESPACE variable through the chart, instead of having it fixed to the namespace of the operator.