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

refactor(module/vnet_peering)!: module refactor and adjusted examples #355

Closed acelebanski closed 1 year ago

acelebanski commented 1 year ago

Description

PR delivers changes for vnet_peering:

README.md adjusted to new style with additional header file
minimum Terraform version set to 1.3
refactor variables:
- descriptions
- types with optionals
- default values
- validations
simplify main.tf by removing not required try
remove name generation logic from the module
add terraform registry reference

Motivation and Context

336

How Has This Been Tested?

I checked that module using "test_infrastructure" example.

Types of changes

Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes if appropriate.
[x] All new and existing tests passed.