Two variables for one function

[Dok-dev]

Two variables are used to select the CNI plugin. And how not to install the plugin at all is also not obvious:

terraform-yc-kubernetes/variables.tf

Lines 132 to 142 in d736b1f

	variable "network_policy_provider" {
	description = "Network policy provider for Kubernetes cluster"
	type = string
	default = "CALICO"
	}
	variable "enable_cilium_policy" {
	description = "Flag for enabling or disabling Cilium CNI."
	type = bool
	default = false
	}

I used a solution with a single variable "network_policy_provider" for all three cases. Do you think it is appropriate to make a pull request for this?

[romati88]

We use Calico only for network policy functionality (if Cilium isn't used) and not as CNI. So Cilium CNI is called tunnel mode in managed k8s and must be separated variable. You can turn of NP at all with :

network_policy_provider = ""
enable_cilium_policy=false

[Dok-dev]

Thanks for quoting the documentation, although I didn't ask anything. I think everyone here understands how it works. The problem is the illogical switching of network plug-ins, which complicates automation when using the module. We use a different solution to switch network plug-ins of a single variable.