clean up of resources

Author: riragh

main.tf

@@ -199,8 +199,6 @@ module "node_pools" {
   machine_type   = each.value.machine_type
   fips_enabled   = var.fips_enabled
   os_disk_size   = each.value.os_disk_size
-  # TODO: enable with azurerm v2.37.0
-  #  os_disk_type                 = each.value.os_disk_type
   auto_scaling_enabled         = each.value.min_nodes == each.value.max_nodes ? false : true
   node_count                   = each.value.min_nodes
   min_nodes                    = each.value.min_nodes == each.value.max_nodes ? null : each.value.min_nodes

modules/aks_node_pool/main.tf

@@ -14,8 +14,6 @@ resource "azurerm_kubernetes_cluster_node_pool" "autoscale_node_pool" {
   proximity_placement_group_id = var.proximity_placement_group_id == "" ? null : var.proximity_placement_group_id
   vm_size                      = var.machine_type
   os_disk_size_gb              = var.os_disk_size
-  # TODO: enable after azurerm v2.37.0
-  # os_disk_type                 = var.os_disk_type
   os_type                = var.os_type
   auto_scaling_enabled   = var.auto_scaling_enabled
   node_public_ip_enabled = var.node_public_ip_enabled
@@ -44,8 +42,6 @@ resource "azurerm_kubernetes_cluster_node_pool" "static_node_pool" {
   proximity_placement_group_id = var.proximity_placement_group_id == "" ? null : var.proximity_placement_group_id
   vm_size                      = var.machine_type
   os_disk_size_gb              = var.os_disk_size
-  # TODO: enable after azurerm v2.37.0
-  # os_disk_type                 = var.os_disk_type
   os_type              = var.os_type
   auto_scaling_enabled = var.auto_scaling_enabled
   node_count           = var.node_count

modules/aks_node_pool/variables.tf

@@ -46,13 +46,6 @@ variable "os_disk_size" {
   default     = 100
 }
 
-# TODO: enable after azurerm v2.37.0
-# variable "os_disk_type" {
-#   description = "The type of disk which should be used for the Operating System. Possible values are Ephemeral and Managed. Defaults to Managed. Changing this forces a new resource to be created"
-#   type = string
-#   default = "Managed"
-# }
-
 variable "os_type" {
   description = "The Operating System which should be used for this Node Pool. Changing this forces a new resource to be created. Possible values are Linux and Windows. Defaults to Linux"
   type        = string

modules/azurerm_netapp/main.tf

@@ -5,13 +5,6 @@
 # Terraform Registry - https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/netapp_volume
 # Azure docs - https://docs.microsoft.com/en-us/azure/azure-netapp-files/
 
-
-# TODO: changer after upgrading to v0.13 
-# https://github.com/terraform-providers/terraform-provider-azurerm/issues/5233
-# data "azurerm_resource_group" "anf" {
-#   name = var.resource_group_name
-# }
-
 resource "azurerm_netapp_account" "anf" {
   name                = "${var.prefix}-netappaccount"
   location            = var.location

modules/azurerm_vm/main.tf

@@ -28,7 +28,6 @@ resource "azurerm_network_interface" "vm_nic" {
   tags = var.tags
 }
 
-# TODO : requires specific permissions
 resource "azurerm_network_interface_security_group_association" "vm_nic_sg" {
   network_interface_id      = azurerm_network_interface.vm_nic.id
   network_security_group_id = var.azure_nsg_id