Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix template to properly handle hyphens in 'show lldp neighbors' for arista eos #1843

Merged
merged 4 commits into from
Sep 18, 2024

Conversation

ktbyers
Copy link
Contributor

@ktbyers ktbyers commented Sep 18, 2024

The hyphens in this following output:

netmiko-show --cmd "show lldp neighbors" arista1
Last table change time   : 5 days, 8:50:53 ago
Number of table inserts  : 9
Number of table deletes  : 3
Number of table drops    : 0
Number of table age-outs : 3
Port          Neighbor Device ID       Neighbor Port ID    TTL
---------- ------------------------ ---------------------- ---
Et2           arista2                  Ethernet2           120
Et3           arista2                  Ethernet3           120
Et4           arista2                  Ethernet4           120
Et5           arista2                  Ethernet5           120
Et6           arista2                  Ethernet6           120
Et7           arista2                  Ethernet7           120

Was showing up as an entry in the parsed LLDP output table.

This fix causes that hyphen line to be skipped over.

@ktbyers ktbyers changed the title Fix template to properly handle hyphens in 'show lldp neighbors' for arista eos Fix template to properly handle hyphens in 'show lldp neighbors' for arista eos [don't merge yet...more updates] Sep 18, 2024
@ktbyers ktbyers changed the title Fix template to properly handle hyphens in 'show lldp neighbors' for arista eos [don't merge yet...more updates] Fix template to properly handle hyphens in 'show lldp neighbors' for arista eos Sep 18, 2024
@itdependsnetworks
Copy link
Contributor

Good to see that arista has been around long enough to do the same things Cisco does with show output :)

@jmcgill298 jmcgill298 merged commit 3532582 into networktocode:master Sep 18, 2024
14 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants