search

CiscoDevNet / terraform-provider-sdwan

Terraform Cisco SD-WAN Provider
https://registry.terraform.io/providers/CiscoDevNet/sdwan
Mozilla Public License 2.0
19 stars 11 forks source link

[UX2.0] transport_management_vpn_interface_ethernet_profile_parcel missing option to set ipv4/6_address as dynamic. #269

Open tzarski0 opened 3 months ago

tzarski0 commented 3 months ago

In transport_management_vpn_interface_ethernet_profile_parcel, we have option to set static ip with ipv4_address parameter, but I don't see any option to set ip address as dynamic. Am I missing something or do we need to add that?

seconroy commented 3 months ago

If you configure ipv4_dhcp_distance or enable_dhcpv6 (will rename to ipv6_enable_dhcpv6) the ip address should be set as dynamic

tzarski0 commented 3 months ago

OK, but what if user wants to configure dynamic ipv4 address and use default ipv4 DHCP distance? This is not possible in the current provider. Wouldn't it be better to create ipv4_address_dynamic bool? And use similar for IPv6?

tzarski0 commented 3 months ago

In sdwan_cisco_vpn_interface_feature_template, we've had "dhcp" bool variable that was allowing that. Maybe we should do it in similar way here?

seconroy commented 3 months ago

In sdwan_cisco_vpn_interface_feature_template the "dhcp" bool variable was needed by the API and in transport_management_vpn_interface_ethernet_profile_parcel there doesn't seem to be anything similar in the scheme/API. I will have a look and see how this can be improved though

tzarski0 commented 2 months ago

@seconroy it seems like we have same problem with sdwan_transport_wan_vpn_interface_ethernet_profile_parcel