PaloAltoNetworks / terraform-azurerm-vmseries-modules

Terraform Reusable Modules for VM-Series on Azure
https://registry.terraform.io/modules/PaloAltoNetworks/vmseries-modules/azurerm/latest
MIT License
49 stars 59 forks source link

for_each and count not usable with vmseries module #231

Closed mazwettler closed 1 year ago

mazwettler commented 1 year ago

Is your feature request related to a problem?

Not sure if bug or feature request is more accurate, therefore i will file this as a feature request. Currently it is not possible to use the vmseries module with for_each or count to create multiple similar VMs. However i think this should be possible since in most deployments at least at my customers we use load balancers and therefore want to easily increasre/decrease the number of VMs. This seems to be related to the provider configuration beeing present in the module according to the error message: The module at module.vmseries is a legacy module which contains its own local provider configurations, and so calls to it may not use the count, for_each, │ or depends_on arguments.

I am wondering - is there anything that the provider configuration is needed for in the module, since its basically empty?

Describe the solution you'd like

Remove the provider configuration from the module.

welcome-to-palo-alto-networks[bot] commented 1 year ago

:tada: Thanks for opening your first issue here! Welcome to the community!

mazwettler commented 1 year ago

my mistake, realised i used the example...