Merge pull request #603 from DeviaVir/main

resource_group: remove data source + location required

Author: lonegunmanb

log_analytics.tf

@@ -1,7 +1,7 @@
 resource "azurerm_log_analytics_workspace" "main" {
   count = local.create_analytics_workspace ? 1 : 0
 
-  location                                = coalesce(var.location, data.azurerm_resource_group.main.location)
+  location                                = var.location
   name                                    = coalesce(var.cluster_log_analytics_workspace_name, trim("${var.prefix}-workspace", "-"))
   resource_group_name                     = coalesce(var.log_analytics_workspace_resource_group_name, var.resource_group_name)
   allow_resource_only_permissions         = var.log_analytics_workspace_allow_resource_only_permissions

main.tf

@@ -1,7 +1,3 @@
-data "azurerm_resource_group" "main" {
-  name = var.resource_group_name
-}
-
 moved {
   from = module.ssh-key.tls_private_key.ssh
   to   = tls_private_key.ssh[0]
@@ -15,9 +11,9 @@ resource "tls_private_key" "ssh" {
 }
 
 resource "azurerm_kubernetes_cluster" "main" {
-  location                            = coalesce(var.location, data.azurerm_resource_group.main.location)
+  location                            = var.location
   name                                = "${local.cluster_name}${var.cluster_name_random_suffix ? substr(md5(uuid()), 0, 4) : ""}"
-  resource_group_name                 = data.azurerm_resource_group.main.name
+  resource_group_name                 = var.resource_group_name
   azure_policy_enabled                = var.azure_policy_enabled
   cost_analysis_enabled               = var.cost_analysis_enabled
   disk_encryption_set_id              = var.disk_encryption_set_id

variables.tf

@@ -1,6 +1,6 @@
 variable "resource_group_name" {
   type        = string
-  description = "The resource group name to be imported"
+  description = "The existing resource group name to use"
 }
 
 variable "aci_connector_linux_enabled" {
@@ -671,7 +671,6 @@ variable "local_account_disabled" {
 
 variable "location" {
   type        = string
-  default     = null
   description = "Location of cluster, if not defined it will be read from the resource-group"
 }