add var for remote system ip in lldp tpl #1577
Conversation
| @@ -17,7 +17,8 @@ Start | |||
| ^System Description: -> Description | |||
| ^System Description\s*-\s*${REMOTE_SYSTEM_DESCRIPTION} | |||
| ^System Capabilities\s*?[:-]\s+${REMOTE_SYSTEM_CAPAB} | |||
| ^Enabled Capabilities\s*?[:-]\s+${REMOTE_SYSTEM_ENABLE_CAPAB} -> Record | |||
| ^Enabled Capabilities\s*?[:-]\s+${REMOTE_SYSTEM_ENABLE_CAPAB} | |||
| ^\s+IP\:\s${REMPRT_SYSTEM_MGMT_IP} -> Record | |||
There was a problem hiding this comment.
What is remprt and where is this variable used?
There was a problem hiding this comment.
Actually correct name is REMOTE_SYSTEM_MGMT_IP. I will correct it.
This variable is being used to get remote system's management ip which can be used to build a network topology. Once you have remote system's management ip, then you can log into it and find more devices downstream.
There was a problem hiding this comment.
The tests are failing: https://github.com/napalm-automation/napalm/runs/6521163875?check_suite_focus=true#step:8:1199
Wrong format of TextFSM template show_lldp_neighbors_detail: Expected blank line after last Value entry. Line: 10.
Hi. Added a new line after var declaration - ratneshnagori@b5344e7 |
|
@ratneshnagori I'm afraid the tests are still failing... |
Current textfsm ios template for show lldp neighbors details doesnt return remote system's mgmt ip. Editing this template to parse output of command to include mgmt ip as well.