enhancement: Bump nginx ingress default version

Author: bmonkman

modules/kubernetes/ingress_nginx/README.md

@@ -22,7 +22,7 @@ No requirements.
 |------|-------------|------|---------|:--------:|
 | additional\_configmap\_options | Map of additional options to add to the configmap. Merged into a default set defined by the module. | `map(string)` | `{}` | no |
 | apply\_pod\_anti\_affinity | If true, will instruct k8s to try not to schedule multiple nginx controller pods on the same node if there are other nodes available. This helps redundancy, as it is less likely all your controller pods are on the same node, which could cause problems if that node were terminated unexepectedly. | `bool` | `true` | no |
-| chart\_version | The version of helm chart to use. | `string` | `"3.25.0"` | no |
+| chart\_version | The version of helm chart to use. | `string` | `"3.34.0"` | no |
 | connection\_idle\_timeout | The amount of time the load balancer will keep an idle connection open for. The value of nginx upstream-keepalive-timeout will also be set to this value + 5. If it were shorter than the LB timeout it could cause intermittent 502s. | `number` | `55` | no |
 | enable\_metrics | Enable prometheus metrics support, including adding a ServiceMonitor. | `bool` | n/a | yes |
 | external\_traffic\_policy | The external traffic policy to apply to the ingress service. Cluster will open a valid NodePort on all nodes even if they aren't running an ingress pod and kubernetes will handle sending the traffic to the correct pod. Local will only have valid NodePorts on the nodes running ingress pods. | `string` | `"Cluster"` | no |

modules/kubernetes/ingress_nginx/variables.tf

@@ -1,7 +1,7 @@
 variable "chart_version" {
   description = "The version of helm chart to use."
   type        = string
-  default     = "3.25.0"
+  default     = "3.34.0"
 }
 
 variable "replica_count" {