-
Task
-
Resolution: Done
-
Undefined
-
None
-
None
When rendering charts, the Sail operator (always) sets the following values (in maistra_30_strategy.go):
- global.istioNamespace
- istio_cni.enabled
- cni.privileged
The operator assumes that CNI should always be enabled, but this is wrong. Since the operator is meant to be moved upstream, it should support deploying the Istio control plane in any way the user wants.
Additionally, the name (maistra_30_strategy) no longer makes sense, since there is no "3.0" Istio version.
We should remove the code that sets the istio_cni.enabled and cni.privileged values, and move the code that sets global.istioNamespace somewhere else. Also, users shouldn't be able to override the namespace. It must always match the namespace of the Istio resource.
- mentioned on
(33 mentioned on)