more clean up

nebari-dev · Nov 4, 2024 · e05f143 · e05f143
1 parent 59daa0c
commit e05f143
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 9 deletions.
diff --git a/src/_nebari/stages/infrastructure/template/azure/modules/kubernetes/main.tf b/src/_nebari/stages/infrastructure/template/azure/modules/kubernetes/main.tf
@@ -36,8 +36,7 @@ resource "azurerm_kubernetes_cluster" "main" {
     min_count           = var.node_groups[0].min_size
     max_count           = var.node_groups[0].max_size
     max_pods            = var.max_pods
-    # TODO: I don't think it's possible to add node_taints to the default node pool so we should throw an error somewhere if people try to do this
-    # see https://github.com/hashicorp/terraform-provider-azurerm/issues/9183 for more info
+    # TODO: It's not possible to add node_taints to the default node pool. See https://github.com/hashicorp/terraform-provider-azurerm/issues/9183 for more info
 
     orchestrator_version = var.kubernetes_version
     node_labels = {

diff --git a/src/_nebari/stages/infrastructure/template/azure/modules/kubernetes/variables.tf b/src/_nebari/stages/infrastructure/template/azure/modules/kubernetes/variables.tf
@@ -29,12 +29,6 @@ variable "environment" {
   type        = string
 }
 
-
-# variable "node_groups" {
-#   description = "Node pools to add to Azure Kubernetes Cluster"
-#   type        = list(map(any))
-# }
-
 variable "node_groups" {
   description = "Node pools to add to Azure Kubernetes Cluster"
   type = list(object({

diff --git a/src/_nebari/stages/infrastructure/template/gcp/modules/kubernetes/variables.tf b/src/_nebari/stages/infrastructure/template/gcp/modules/kubernetes/variables.tf
@@ -58,7 +58,7 @@ variable "node_groups" {
       min_size      = 0
       max_size      = 2
       labels        = {}
-      node_taints   = [] # TODO: Do this for other cloud providers
+      node_taints   = []
     },
     {
       name          = "worker"