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

Fixes #851 and add new template #857

Merged

Conversation

diepes
Copy link
Contributor

@diepes diepes commented Jan 7, 2021

add new test tests/cisco_nxos/show_ip_arp_detail/cisco_nxos_show_ip_arp_detail2.raw

fix compatibility of template templates/cisco_nxos_show_ip_arp_detail.textfsm

ISSUE TYPE
  • Bugfix Pull Request
  • Additional Testing
COMPONENT

templates/cisco_nxos_show_ip_arp_detail.textfsm

SUMMARY

add new test, time not just in "00:00:00" format , also in "0.000000" format.
add "&. -> Error" catch and match unused headers.
make time match more specific , looking for 3 types dot, colon, or just dash

previous test's still pass
added new test
tox pass

…os_show_ip_arp_detail2.raw

fix compatability of template templates/cisco_nxos_show_ip_arp_detail.textfsm
Copy link
Contributor

@jmcgill298 jmcgill298 left a comment

Choose a reason for hiding this comment

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

Thanks @diepes

Do you think my suggestions would work?

Pieter E Smit and others added 3 commits March 1, 2021 22:16
after matching header, jump to Data section and extract values.

Co-authored-by: Jacob McGill <9847006+jmcgill298@users.noreply.github.com>
simplify match as suggested in pr

Co-authored-by: Jacob McGill <9847006+jmcgill298@users.noreply.github.com>
@diepes
Copy link
Contributor Author

diepes commented Mar 9, 2021

@jmcgill298 please have a look again i think it is all in order now.

@jmcgill298 jmcgill298 merged commit 966392a into networktocode:master Mar 9, 2021
jvanderaa pushed a commit that referenced this pull request Nov 10, 2021
guillaume-mbali pushed a commit to unyc-io/ntc-templates that referenced this pull request Apr 12, 2023
cppmonkey pushed a commit to cppmonkey/ntc-templates that referenced this pull request Oct 25, 2023
# 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.

2 participants