Replace data source lookup for "azurerm_resource_group" with variables to avoid resources recreation issue when re-applying the module

[nitinnbisht]

FIxes #105
Changes proposed in the pull request:
When testing in terraform v13,14,15 - The current module results in recreation of cluster in consecutive apply due to the unwanted diff generated by data source lookup of location from the resource group. The fix in the PR proposes using variable for location in-place of a data source lookup to resolve this issue.

The data source lookup responsible:

 data "azurerm_resource_group" "main" {  
  name = var.resource_group_name
  }

Terraform plan using the module results in certain resources to force replacement, PFB

# module.aks.azurerm_kubernetes_cluster.main must be replaced
-/+ resource "azurerm_kubernetes_cluster" "main" {
      - api_server_authorized_ip_ranges = [] -> null
      - enable_pod_security_policy      = false -> null
---
      ~ location                        = "westeurope" -> (known after apply) # forces replacement
---
    }

  # module.aks.azurerm_log_analytics_solution.main[0] must be replaced
-/+ resource "azurerm_log_analytics_solution" "main" {
---
      ~ location              = "westeurope" -> (known after apply) # forces replacement
---
        }
    }

  # module.aks.azurerm_log_analytics_workspace.main[0] must be replaced
-/+ resource "azurerm_log_analytics_workspace" "main" {
 ---
      ~ location                   = "westeurope" -> (known after apply) # forces replacement
 ---
    }

[ghost]

All CLA requirements met.

[fplantinga-guida]

+1. We experience resource recreation as well when using data source lookups

[nitinnbisht]

Any updates for this?

[github-actions]

MAIN BRANCH PUSH DETECTED DUE TO #241, THIS PR WILL BE UPDATED.

[github-actions]

MAIN BRANCH PUSH DETECTED DUE TO #, THIS PR WILL BE UPDATED.

[github-actions]

MAIN BRANCH PUSH DETECTED DUE TO #, THIS PR WILL BE UPDATED.

[github-actions]

MAIN BRANCH PUSH DETECTED DUE TO #, THIS PR NEED TO BE UPDATED TO TRIGGER CI.

[lonegunmanb]

I'm closing this pr since we've implemented this feature in #172. Thanks folks.