Skip to content
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

Jump to bottom

Add verification IPv4 address on LLDP conf Jinja2 Template #5699

Merged
merged 9 commits into from
Nov 7, 2020
Merged

Add verification IPv4 address on LLDP conf Jinja2 Template #5699

Show file tree
Hide file tree
Changes from 8 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
d42ae43
Add verification IPv4 address on LLDP conf Jinja2 Template
bratashX Oct 22, 2020
bda9e7d
Code improvement
bratashX Oct 23, 2020
ad2001f
Fix config file for correct work test_j2files.py(test_lldp)
bratashX Oct 29, 2020
de774cb
Revert "Fix config file for correct work test_j2files.py(test_lldp)"
bratashX Oct 29, 2020
a5f32d2
Merge branch 'master' into fix_lldp_jinja2_template
bratashX Nov 5, 2020
ad58479
Add test for lldp.conf
bratashX Nov 5, 2020
0e81cc3
Code refactoring
bratashX Nov 5, 2020
8464938
Add new case to test_lldp and remove link to output files
bratashX Nov 6, 2020
3281d2f
Use json-file instead of minigraph
bratashX Nov 6, 2020
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
19 changes: 13 additions & 6 deletions dockers/docker-lldp/lldpd.conf.j2
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,13 +1,20 @@
{% set mgmt_if = {} %}
{% if MGMT_INTERFACE %}
{% for (mgmt_name, mgmt_prefix) in MGMT_INTERFACE|pfx_filter %}
{% if mgmt_prefix|ipv4 %}
{% if mgmt_if.update({'port_name' : mgmt_name}) %} {% endif %}
{% if mgmt_if.update({'ipv4' : mgmt_prefix|ip}) %} {% endif %}
{% endif %}
{% endfor %}
{% endif %}
{% if mgmt_if %}
{# If MGMT port alias is available, use it for port ID subtype, otherwise use port name #}
{% set mgmt_port_name = (MGMT_INTERFACE.keys()|list)[0][0] %}
{% set ipv4 = (MGMT_INTERFACE.keys()|list)[0][1].split('/')[0] %}
{% if MGMT_PORT and MGMT_PORT[mgmt_port_name] and MGMT_PORT[mgmt_port_name].alias %}
configure ports eth0 lldp portidsubtype local {{ MGMT_PORT[mgmt_port_name].alias }}
{% if MGMT_PORT and MGMT_PORT[mgmt_if.port_name] and MGMT_PORT[mgmt_if.port_name].alias %}
configure ports eth0 lldp portidsubtype local {{ MGMT_PORT[mgmt_if.port_name].alias }}
{% else %}
configure ports eth0 lldp portidsubtype local {{ mgmt_port_name }}
configure ports eth0 lldp portidsubtype local {{ mgmt_if.port_name }}
{% endif %}
configure system ip management pattern {{ ipv4 }}
configure system ip management pattern {{ mgmt_if.ipv4 }}
{% endif %}
configure system hostname {{ DEVICE_METADATA['localhost']['hostname'] }}
{# pause lldpd operations until all interfaces are well configured, resume command will run in lldpmgrd #}
Expand Down
0 ...engine/tests/sample_output/py2/lldpd.conf → ...utput/py2/lldp_conf/lldpd-ipv4-iface.conf
View file
Open in desktop
File renamed without changes.
2 changes: 2 additions & 0 deletions src/sonic-config-engine/tests/sample_output/py2/lldp_conf/lldpd-ipv6-iface.conf
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
configure system hostname switch-t0
pause
0 ...engine/tests/sample_output/py3/lldpd.conf → ...utput/py3/lldp_conf/lldpd-ipv4-iface.conf
View file
Open in desktop
File renamed without changes.
2 changes: 2 additions & 0 deletions src/sonic-config-engine/tests/sample_output/py3/lldp_conf/lldpd-ipv6-iface.conf
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
configure system hostname switch-t0
pause
Loading