Skip to content

Commit d4cd456

Browse files
committed
Rename ingress_controller_type to ingress_controller_provider.
Signed-off-by: Mateusz Paluszkiewicz <theaifam5@gmail.com>
1 parent 7f9cba0 commit d4cd456

File tree

4 files changed

+10
-10
lines changed

4 files changed

+10
-10
lines changed

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -502,7 +502,7 @@ For access to Talos and the Kubernetes API, please refer to the [Cluster Access]
502502

503503
The ingress controller uses a default load balancer service to manage external traffic. For geo-redundancy and high availability, `ingress_load_balancer_pools` can be configured as an alternative, replacing the default load balancer with the specified pool of load balancers.
504504

505-
This project has support for Cilium and NGINX ingress controllers. The default is NGINX, but Cilium can be selected by setting `ingress_controller_type` to `cilium`.
505+
This project has support for Cilium and NGINX ingress controllers. The default is NGINX, but Cilium can be selected by setting `ingress_controller_provider` to `cilium`.
506506

507507
##### Configuring Load Balancer Pools
508508
To replace the default load balancer, use `ingress_load_balancer_pools` in the Terraform configuration. This setup ensures high availability and geo-redundancy by distributing traffic from various locations across all targets in all regions.
@@ -528,7 +528,7 @@ Configuring local traffic handling enhances network efficiency by reducing laten
528528

529529
Example `kubernetes.tf` configuration:
530530
```hcl
531-
ingress_nginx_kind = "DaemonSet" # Only for ingress_controller_type = "nginx"
531+
ingress_nginx_kind = "DaemonSet" # Only for ingress_controller_provider = "nginx"
532532
ingress_service_external_traffic_policy = "Local"
533533
534534
ingress_load_balancer_pools = [

cilium.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
locals {
2-
ingress_cilium_enabled = var.ingress_controller_enabled && var.ingress_controller_type == "cilium"
2+
ingress_cilium_enabled = var.ingress_controller_enabled && var.ingress_controller_provider == "cilium"
33

44
# Cilium IPSec Configuration
55
cilium_ipsec_enabled = var.cilium_encryption_enabled && var.cilium_encryption_type == "ipsec"

ingress_nginx.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
locals {
2-
ingress_nginx_enabled = var.ingress_controller_enabled && var.ingress_controller_type == "nginx"
2+
ingress_nginx_enabled = var.ingress_controller_enabled && var.ingress_controller_provider == "nginx"
33
ingress_nginx_namespace = local.ingress_nginx_enabled ? {
44
apiVersion = "v1"
55
kind = "Namespace"

variables.tf

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1385,23 +1385,23 @@ variable "ingress_controller_enabled" {
13851385
description = "Enables the deployment of an ingress controller."
13861386
}
13871387

1388-
variable "ingress_controller_type" {
1388+
variable "ingress_controller_provider" {
13891389
type = string
13901390
default = "nginx"
1391-
description = "Specifies the type of ingress controller to be used. Valid options are 'nginx' or 'cilium'."
1391+
description = "Specifies the ingress controller provider to use. Valid options are 'nginx' or 'cilium'."
13921392

13931393
validation {
1394-
condition = contains(["nginx", "cilium"], var.ingress_controller_type)
1395-
error_message = "Invalid ingress kind. Allowed values are 'nginx' or 'cilium'."
1394+
condition = contains(["nginx", "cilium"], var.ingress_controller_provider)
1395+
error_message = "Invalid ingress provider. Allowed values are 'nginx' or 'cilium'."
13961396
}
13971397

13981398
validation {
1399-
condition = var.ingress_controller_type != "nginx" || var.cert_manager_enabled
1399+
condition = var.ingress_controller_provider != "nginx" || var.cert_manager_enabled
14001400
error_message = "Ingress NGINX can only be enabled if cert-manager is also enabled."
14011401
}
14021402

14031403
validation {
1404-
condition = var.ingress_controller_type != "cilium" || var.cilium_enabled
1404+
condition = var.ingress_controller_provider != "cilium" || var.cilium_enabled
14051405
error_message = "Ingress Cilium can only be enabled if Cilium is also enabled."
14061406
}
14071407
}

0 commit comments

Comments
 (0)