Azure / terraform-azurerm-avm-template

Template repo for Azure Verified Modules using Terraform
MIT License
47 stars 30 forks source link

fix: bug when location is specified #49

Closed kewalaka closed 11 months ago

kewalaka commented 11 months ago

hi @matt-FFFFFF - missed this one - we need a 'try' around the location otherwise in the case a location is specified, the data block errors. I think this proposal is the best.

I've E2E tested it on my container registry which has a mix of locations specified & not:

https://github.com/kewalaka/terraform-azurerm-avm-res-containerregistry-registry/actions/runs/7065336984

kewalaka commented 11 months ago

hi @matt-FFFFFF I think what I've added is reasonable, I've ran it against the container registry too just to give it some exercise 😊

https://github.com/kewalaka/terraform-azurerm-avm-res-containerregistry-registry/actions/runs/7146061224/job/19462991978