search

Azure / terraform-azurerm-avm-res-cognitiveservices-account

This Terraform module is designed to manage Azure Cognitive Services. It provides a comprehensive set of variables and resources to configure and deploy Cognitive Services in Azure.
https://github.com/Azure/terraform-azurerm-avm-res-cognitiveservices-account
MIT License
6 stars 10 forks source link

[AVM Module Issue]: diagnostic.tf #48

Closed acauret closed 3 months ago

acauret commented 3 months ago

Check for previous/existing GitHub issues

Issue Type?

Bug

(Optional) Module Version

No response

(Optional) Correlation Id

No response

Description

This is incorrect: name = coalesce(each.value.name != null, "diag-${var.name}") - This will either return true or false as coalesce takes any number of arguments and returns the first one that isn't null or an empty string.

should be name = coalesce(each.value.name , "diag-${var.name}")

+ "logAnalyticsDestinationType": null so terraform is always trying to set the logAnalyticsDestinationType

microsoft-github-policy-service[bot] commented 3 months ago

[!IMPORTANT] The "Needs: Triage :mag:" label must be removed once the triage process is complete!

[!NOTE] This label was added as per ITA06.

microsoft-github-policy-service[bot] commented 3 months ago

[!NOTE] The "Type: Bug :bug:" label was added as per ITA21.

acauret commented 3 months ago

"logAnalyticsDestinationType": null so terraform is always trying to set the logAnalyticsDestinationType

lonegunmanb commented 3 months ago

@acauret Thanks for reporting this bug to us! I'm working on a patch for it.