From eba4e767a64111ca5cb6dbb8d9494e21c89b88a3 Mon Sep 17 00:00:00 2001
From: hezijie <lonegunmanb@hotmail.com>
Date: Mon, 4 Mar 2024 17:15:04 +0800
Subject: [PATCH] remove gpu from default node pool, since it's weired to use
 gpu instance in system node pool

---
 README.md    | 1 -
 main.tf      | 2 --
 variables.tf | 6 ------
 3 files changed, 9 deletions(-)

diff --git a/README.md b/README.md
index 22aedd46..23244f9e 100644
--- a/README.md
+++ b/README.md
@@ -314,7 +314,6 @@ No modules.
 | <a name="input_create_role_assignment_network_contributor"></a> [create\_role\_assignment\_network\_contributor](#input\_create\_role\_assignment\_network\_contributor) | (Deprecated) Create a role assignment for the AKS Service Principal to be a Network Contributor on the subnets used for the AKS Cluster | `bool` | `false` | no |
 | <a name="input_create_role_assignments_for_application_gateway"></a> [create\_role\_assignments\_for\_application\_gateway](#input\_create\_role\_assignments\_for\_application\_gateway) | (Optional) Whether to create the corresponding role assignments for application gateway or not. Defaults to `true`. | `bool` | `true` | no |
 | <a name="input_default_node_pool_fips_enabled"></a> [default\_node\_pool\_fips\_enabled](#input\_default\_node\_pool\_fips\_enabled) | (Optional) Should the nodes in this Node Pool have Federal Information Processing Standard enabled? Changing this forces a new resource to be created. | `bool` | `null` | no |
-| <a name="input_default_node_pool_gpu_instance"></a> [default\_node\_pool\_gpu\_instance](#input\_default\_node\_pool\_gpu\_instance) | (Optional) Specifies the GPU MIG instance profile for supported GPU VM SKU for cluster's default node pool. The allowed values are `MIG1g`, `MIG2g`, `MIG3g`, `MIG4g` and `MIG7g`. Changing this forces a new resource to be created. | `string` | `null` | no |
 | <a name="input_disk_encryption_set_id"></a> [disk\_encryption\_set\_id](#input\_disk\_encryption\_set\_id) | (Optional) The ID of the Disk Encryption Set which should be used for the Nodes and Volumes. More information [can be found in the documentation](https://docs.microsoft.com/azure/aks/azure-disk-customer-managed-keys). Changing this forces a new resource to be created. | `string` | `null` | no |
 | <a name="input_ebpf_data_plane"></a> [ebpf\_data\_plane](#input\_ebpf\_data\_plane) | (Optional) Specifies the eBPF data plane used for building the Kubernetes network. Possible value is `cilium`. Changing this forces a new resource to be created. | `string` | `null` | no |
 | <a name="input_enable_auto_scaling"></a> [enable\_auto\_scaling](#input\_enable\_auto\_scaling) | Enable node pool autoscaling | `bool` | `false` | no |
diff --git a/main.tf b/main.tf
index 8a4037ad..1af3c989 100644
--- a/main.tf
+++ b/main.tf
@@ -59,7 +59,6 @@ resource "azurerm_kubernetes_cluster" "main" {
       enable_host_encryption       = var.enable_host_encryption
       enable_node_public_ip        = var.enable_node_public_ip
       fips_enabled                 = var.default_node_pool_fips_enabled
-      gpu_instance                 = var.default_node_pool_gpu_instance
       max_count                    = null
       max_pods                     = var.agents_max_pods
       min_count                    = null
@@ -162,7 +161,6 @@ resource "azurerm_kubernetes_cluster" "main" {
       enable_host_encryption       = var.enable_host_encryption
       enable_node_public_ip        = var.enable_node_public_ip
       fips_enabled                 = var.default_node_pool_fips_enabled
-      gpu_instance                 = var.default_node_pool_gpu_instance
       max_count                    = var.agents_max_count
       max_pods                     = var.agents_max_pods
       min_count                    = var.agents_min_count
diff --git a/variables.tf b/variables.tf
index bfc20618..32118034 100644
--- a/variables.tf
+++ b/variables.tf
@@ -436,12 +436,6 @@ variable "default_node_pool_fips_enabled" {
   description = " (Optional) Should the nodes in this Node Pool have Federal Information Processing Standard enabled? Changing this forces a new resource to be created."
 }
 
-variable "default_node_pool_gpu_instance" {
-  type        = string
-  default     = null
-  description = "(Optional) Specifies the GPU MIG instance profile for supported GPU VM SKU for cluster's default node pool. The allowed values are `MIG1g`, `MIG2g`, `MIG3g`, `MIG4g` and `MIG7g`. Changing this forces a new resource to be created."
-}
-
 variable "disk_encryption_set_id" {
   type        = string
   default     = null