Azure / terraform-azurerm-avm-res-network-virtualnetwork

Azure Verified Module for Virtual Network
https://registry.terraform.io/modules/Azure/avm-res-network-virtualnetwork
MIT License
18 stars 14 forks source link

Redundant random_id resource created when enable_telemetry already set to false #18

Closed tanvp112 closed 7 months ago

tanvp112 commented 8 months ago

There's a random_id resource here that doesn't refer to the user specified enable_telemetry but a local enable_telemetry that is always true.

Can the code here be updated to the following so that a redundant random_id won't be created.

locals {
  enable_telemetry = var.enable_telemetry
}

Tq.

herms14 commented 8 months ago

Hi @tanvp112 ,

Thanks for checking. Will look into it and provide update this week

herms14 commented 7 months ago

Hi @tanvp112 ,

This will be fixed in the next PR.

herms14 commented 7 months ago

Hi @tanvp112

This has been fixed in PR #20