search

equinix-labs / ansible-collection-equinix

Ansible content to help automate the management of Equinix resources
https://deploy.equinix.com/labs/ansible-collection-equinix/
GNU General Public License v3.0
2 stars 8 forks source link

fix: only allow one of vrfs or vlans for shared Metal connections #178

Closed ctreatma closed 3 months ago

ctreatma commented 3 months ago

The metal_connection module was only validating the value of vlans based on the selected redundancy. As a result, invalid values for vrfs would not be caught, and users creating a VRF-based Metal connection would be force to specify a stub value for vlans instead of omitting it entirely.

This updates the metal_connection module to enforce that the user specifies only one of vlans or vrfs, and that the parameter they specify only has 1 element if they are creating a non-redundant connection.

github-actions[bot] commented 3 months ago

This PR is included in version 0.4.1 :tada: