Skip to content

Add dhcp servers list #621

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from
Open

Add dhcp servers list #621

Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
ea7187f
Expand dhcp_servers to 16
Sep 26, 2025
6452c1f
Merge branch 'develop' into add_dhcp_servers_list
Sep 26, 2025
c1961f2
Fix issue 492 - Add dhcp_servers list
Oct 1, 2025
49ceaaa
Merge branch 'develop' into add_dhcp_servers_list
Oct 1, 2025
e574d0f
Merge branch 'develop' into add_dhcp_servers_list
Nov 5, 2025
496e73b
Remove dependency on dhcp_srvrX_ip and dhcp_srvrX_vrf
Nov 11, 2025
ed1155e
Merge branch 'develop' into add_dhcp_servers_list
Nov 11, 2025
db5528f
Normalized jinja template logic for dhcp_servers definition
Nov 14, 2025
7469920
Merge branch 'develop' into add_dhcp_servers_list
Nov 14, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions plugins/action/dtc/manage_child_fabric_networks.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@
# Currently supported Network template config keys and their mapping to data model keys
NETWORK_TEMPLATE_CONFIG_MAP = {
'loopbackId': {'dm_key': 'dhcp_loopback_id', 'default': ''},
'dhcpServers': {'dm_key': 'dhcp_servers', 'default': ''},
'ENABLE_NETFLOW': {'dm_key': 'netflow_enable', 'default': False},
'VLAN_NETFLOW_MONITOR': {'dm_key': 'vlan_netflow_monitor', 'default': ''},
'trmEnabled': {'dm_key': 'trm_enable', 'default': False},
Expand Down
21 changes: 6 additions & 15 deletions roles/dtc/common/templates/ndfc_networks/dc_vxlan_fabric/dc_vxlan_fabric_networks.j2
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,21 +34,12 @@
arp_suppress: {{ net['arp_suppress'] | default(defaults.vxlan.overlay.networks.arp_supress) }}
dhcp_loopback_id: {{ net['dhcp_loopback_id'] | default(omit) }}
{% if net.dhcp_servers is defined %}
{% if net.dhcp_servers | length == 1 %}
dhcp_srvr1_ip: {{ net['dhcp_servers'][0]['ip_address'] }}
dhcp_srvr1_vrf: {{ net['dhcp_servers'][0]['vrf'] }}
{% elif net.dhcp_servers | length == 2 %}
dhcp_srvr1_ip: {{ net['dhcp_servers'][0]['ip_address'] }}
dhcp_srvr1_vrf: {{ net['dhcp_servers'][0]['vrf'] }}
dhcp_srvr2_ip: {{ net['dhcp_servers'][1]['ip_address'] }}
dhcp_srvr2_vrf: {{ net['dhcp_servers'][1]['vrf'] }}
{% elif net.dhcp_servers | length == 3 %}
dhcp_srvr1_ip: {{ net['dhcp_servers'][0]['ip_address'] }}
dhcp_srvr1_vrf: {{ net['dhcp_servers'][0]['vrf'] }}
dhcp_srvr2_ip: {{ net['dhcp_servers'][1]['ip_address'] }}
dhcp_srvr2_vrf: {{ net['dhcp_servers'][1]['vrf'] }}
dhcp_srvr3_ip: {{ net['dhcp_servers'][2]['ip_address'] }}
dhcp_srvr3_vrf: {{ net['dhcp_servers'][2]['vrf'] }}
{% if net.dhcp_servers | length > 0 and net.dhcp_servers | length <= 16 %}
dhcp_servers:
{% for dhcp_server in net.dhcp_servers[:16] %}
- srvr_ip: {{ dhcp_server['ip_address'] }}
srvr_vrf: {{ dhcp_server['vrf'] }}
{% endfor %}
{% endif %}
{% endif %}
gw_ipv6_subnet: {{ net['gw_ipv6_address'] | default(omit) }}
Expand Down
20 changes: 14 additions & 6 deletions ...tc/common/templates/ndfc_networks/msd_fabric/child_fabric/3.1/msd_child_fabric_network.j2
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,17 @@
{# Auto-generated NDFC VXLAN EVPN MSD Child Fabric Network config data structure for fabric {{ fabric_name }} #}
{# Build dhcpServers as a JSON dict (or empty string) #}
{%- set _dhcp_pairs = [] -%}
{%- if dm is defined and dm['dhcp_servers'] is defined and dm['dhcp_servers'] -%}
Copy link
Collaborator

@mtarking mtarking Nov 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dm should always be defined, no?

Copy link
Collaborator

@mtarking mtarking Nov 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dm['dhcp_servers'] is defined and dm['dhcp_servers'] is a different check than in roles/dtc/common/templates/ndfc_networks/dc_vxlan_fabric/dc_vxlan_fabric_networks.j2. Can we normalize?

{%- for srvr in dm['dhcp_servers'] -%}
{%- if srvr['ip_address'] is defined and srvr['ip_address'] -%}
{%- set _ = _dhcp_pairs.append({
"srvrAddr": srvr['ip_address'], "srvrVrf": srvr['vrf']
}) -%}
{%- endif -%}
{%- endfor -%}
{%- endif -%}
{%- set _dhcp_obj = {'dhcpServers': _dhcp_pairs} -%}
{%- set dhcpServers = _dhcp_obj if (_dhcp_pairs|length > 0) else '' -%}
{
"fabric": "{{ fabric_name }}",
"networkName": "{{ network_name }}",
Expand All @@ -24,12 +37,7 @@
"secondaryGW3": ndfc.secondaryGW3,
"secondaryGW4": ndfc.secondaryGW4,
"loopbackId": dm.dhcp_loopback_id | default(ndfc.loopbackId),
"dhcpServerAddr1": dm['dhcp_servers'][0]['ip_address'] if dm['dhcp_servers'][0]['ip_address'] is defined else "",
"vrfDhcp": dm['dhcp_servers'][0]['vrf'] if dm['dhcp_servers'][0]['ip_address'] is defined else ndfc.vrfDhcp,
"dhcpServerAddr2": dm['dhcp_servers'][1]['ip_address'] if dm['dhcp_servers'][1]['ip_address'] is defined else ndfc.dhcpServerAddr2,
"vrfDhcp2": dm['dhcp_servers'][1]['vrf'] if dm['dhcp_servers'][1]['ip_address'] is defined else "",
"dhcpServerAddr3": dm['dhcp_servers'][2]['ip_address'] if dm['dhcp_servers'][2]['ip_address'] is defined else ndfc.dhcpServerAddr3,
"vrfDhcp3": dm['dhcp_servers'][2]['vrf'] if dm['dhcp_servers'][2]['vrf'] is defined else "",
"dhcpServers": dhcpServers,
"ENABLE_NETFLOW": dm.netflow_enable if dm.netflow_enable is defined else ndfc.ENABLE_NETFLOW,
"SVI_NETFLOW_MONITOR": ndfc.SVI_NETFLOW_MONITOR,
"VLAN_NETFLOW_MONITOR": dm.vlan_netflow_monitor if (dm.netflow_enable is defined and dm.netflow_enable) else ndfc.VLAN_NETFLOW_MONITOR,
Expand Down
20 changes: 14 additions & 6 deletions ...tc/common/templates/ndfc_networks/msd_fabric/child_fabric/3.2/msd_child_fabric_network.j2
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,17 @@
{# Auto-generated NDFC VXLAN EVPN MSD Child Fabric Network config data structure for fabric {{ fabric_name }} #}
{# Build dhcpServers as a JSON dict (or empty string) #}
{%- set _dhcp_pairs = [] -%}
{%- if dm is defined and dm['dhcp_servers'] is defined and dm['dhcp_servers'] -%}
{%- for srvr in dm['dhcp_servers'] -%}
{%- if srvr['ip_address'] is defined and srvr['ip_address'] -%}
{%- set _ = _dhcp_pairs.append({
"srvrAddr": srvr['ip_address'], "srvrVrf": srvr['vrf']
}) -%}
{%- endif -%}
{%- endfor -%}
{%- endif -%}
{%- set _dhcp_obj = {'dhcpServers': _dhcp_pairs} -%}
{%- set dhcpServers = _dhcp_obj if (_dhcp_pairs|length > 0) else '' -%}
{
"fabric": "{{ fabric_name }}",
"networkName": "{{ network_name }}",
Expand All @@ -24,12 +37,7 @@
"secondaryGW3": ndfc.secondaryGW3,
"secondaryGW4": ndfc.secondaryGW4,
"loopbackId": dm.dhcp_loopback_id | default(ndfc.loopbackId),
"dhcpServerAddr1": dm['dhcp_servers'][0]['ip_address'] if dm['dhcp_servers'][0]['ip_address'] is defined else "",
"vrfDhcp": dm['dhcp_servers'][0]['vrf'] if dm['dhcp_servers'][0]['ip_address'] is defined else ndfc.vrfDhcp,
"dhcpServerAddr2": dm['dhcp_servers'][1]['ip_address'] if dm['dhcp_servers'][1]['ip_address'] is defined else ndfc.dhcpServerAddr2,
"vrfDhcp2": dm['dhcp_servers'][1]['vrf'] if dm['dhcp_servers'][1]['ip_address'] is defined else "",
"dhcpServerAddr3": dm['dhcp_servers'][2]['ip_address'] if dm['dhcp_servers'][2]['ip_address'] is defined else ndfc.dhcpServerAddr3,
"vrfDhcp3": dm['dhcp_servers'][2]['vrf'] if dm['dhcp_servers'][2]['vrf'] is defined else "",
"dhcpServers": dhcpServers,
"ENABLE_NETFLOW": dm.netflow_enable if dm.netflow_enable is defined else ndfc.ENABLE_NETFLOW,
"SVI_NETFLOW_MONITOR": ndfc.SVI_NETFLOW_MONITOR,
"VLAN_NETFLOW_MONITOR": dm.vlan_netflow_monitor if (dm.netflow_enable is defined and dm.netflow_enable) else ndfc.VLAN_NETFLOW_MONITOR,
Expand Down
20 changes: 14 additions & 6 deletions ...tc/common/templates/ndfc_networks/msd_fabric/child_fabric/4.1/msd_child_fabric_network.j2
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,17 @@
{# Auto-generated NDFC VXLAN EVPN MSD Child Fabric Network config data structure for fabric {{ fabric_name }} #}
{# Build dhcpServers as a JSON dict (or empty string) #}
{%- set _dhcp_pairs = [] -%}
{%- if dm is defined and dm['dhcp_servers'] is defined and dm['dhcp_servers'] -%}
{%- for srvr in dm['dhcp_servers'] -%}
{%- if srvr['ip_address'] is defined and srvr['ip_address'] -%}
{%- set _ = _dhcp_pairs.append({
"srvrAddr": srvr['ip_address'], "srvrVrf": srvr['vrf']
}) -%}
{%- endif -%}
{%- endfor -%}
{%- endif -%}
{%- set _dhcp_obj = {'dhcpServers': _dhcp_pairs} -%}
{%- set dhcpServers = _dhcp_obj if (_dhcp_pairs|length > 0) else '' -%}
{
"fabric": "{{ fabric_name }}",
"networkName": "{{ network_name }}",
Expand All @@ -24,12 +37,7 @@
"secondaryGW3": ndfc.secondaryGW3,
"secondaryGW4": ndfc.secondaryGW4,
"loopbackId": dm.dhcp_loopback_id | default(ndfc.loopbackId),
"dhcpServerAddr1": dm['dhcp_servers'][0]['ip_address'] if dm['dhcp_servers'][0]['ip_address'] is defined else "",
"vrfDhcp": dm['dhcp_servers'][0]['vrf'] if dm['dhcp_servers'][0]['ip_address'] is defined else ndfc.vrfDhcp,
"dhcpServerAddr2": dm['dhcp_servers'][1]['ip_address'] if dm['dhcp_servers'][1]['ip_address'] is defined else ndfc.dhcpServerAddr2,
"vrfDhcp2": dm['dhcp_servers'][1]['vrf'] if dm['dhcp_servers'][1]['ip_address'] is defined else "",
"dhcpServerAddr3": dm['dhcp_servers'][2]['ip_address'] if dm['dhcp_servers'][2]['ip_address'] is defined else ndfc.dhcpServerAddr3,
"vrfDhcp3": dm['dhcp_servers'][2]['vrf'] if dm['dhcp_servers'][2]['vrf'] is defined else "",
"dhcpServers": dhcpServers,
"ENABLE_NETFLOW": dm.netflow_enable if dm.netflow_enable is defined else ndfc.ENABLE_NETFLOW,
"SVI_NETFLOW_MONITOR": ndfc.SVI_NETFLOW_MONITOR,
"VLAN_NETFLOW_MONITOR": dm.vlan_netflow_monitor if (dm.netflow_enable is defined and dm.netflow_enable) else ndfc.VLAN_NETFLOW_MONITOR,
Expand Down
19 changes: 13 additions & 6 deletions roles/dtc/common/templates/ndfc_networks/msd_fabric/child_fabric/msd_child_fabric_network.j2
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,16 @@
{# Auto-generated NDFC VXLAN EVPN MSD Child Fabric Network config data structure for fabric {{ fabric_name }} #}
{%- set _dhcp_pairs = [] -%}
{%- if dm is defined and dm['dhcp_servers'] is defined and dm['dhcp_servers'] -%}
{%- for srvr in dm['dhcp_servers'] -%}
{%- if srvr['ip_address'] is defined and srvr['ip_address'] -%}
{%- set _ = _dhcp_pairs.append({
"srvrAddr": srvr['ip_address'], "srvrVrf": srvr['vrf'] | default("")
}) -%}
{%- endif -%}
{%- endfor -%}
{%- endif -%}
{%- set _dhcp_obj = {'dhcpServers': _dhcp_pairs} -%}
{%- set dhcpServers = _dhcp_obj if (_dhcp_pairs|length > 0) else '' -%}
{
"fabric": "{{ fabric_name }}",
"networkName": "{{ network_name }}",
Expand All @@ -25,12 +37,7 @@
"secondaryGW3": ndfc.secondaryGW3,
"secondaryGW4": ndfc.secondaryGW4,
"loopbackId": dm.dhcp_loopback_id | default(ndfc.loopbackId),
"dhcpServerAddr1": dm['dhcp_servers'][0]['ip_address'] if dm['dhcp_servers'][0]['ip_address'] is defined else "",
"vrfDhcp": dm['dhcp_servers'][0]['vrf'] if dm['dhcp_servers'][0]['ip_address'] is defined else ndfc.vrfDhcp,
"dhcpServerAddr2": dm['dhcp_servers'][1]['ip_address'] if dm['dhcp_servers'][1]['ip_address'] is defined else ndfc.dhcpServerAddr2,
"vrfDhcp2": dm['dhcp_servers'][1]['vrf'] if dm['dhcp_servers'][1]['ip_address'] is defined else "",
"dhcpServerAddr3": dm['dhcp_servers'][2]['ip_address'] if dm['dhcp_servers'][2]['ip_address'] is defined else ndfc.dhcpServerAddr3,
"vrfDhcp3": dm['dhcp_servers'][2]['vrf'] if dm['dhcp_servers'][2]['vrf'] is defined else "",
"dhcpServers": dhcpServers,
"ENABLE_NETFLOW": dm.netflow_enable if dm.netflow_enable is defined else ndfc.ENABLE_NETFLOW,
"SVI_NETFLOW_MONITOR": ndfc.SVI_NETFLOW_MONITOR,
"VLAN_NETFLOW_MONITOR": dm.vlan_netflow_monitor if (dm.netflow_enable is defined and dm.netflow_enable) else ndfc.VLAN_NETFLOW_MONITOR,
Expand Down