-
Notifications
You must be signed in to change notification settings - Fork 743
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.
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
New Template: cisco_ios_show_ip_ospf_database_router #606
New Template: cisco_ios_show_ip_ospf_database_router #606
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ChristopherJHart Each state should end with
^\s*$$
^. -> Error
This ensures that we are not missing any data and incorrectly reporting back the data. This also means that each line in the output must be accounted for with a match (non-capturing).
@jmcgill298 Apologies! I've modified the TextFSM template states accordingly. Thank you! |
tests/cisco_ios/show_ip_ospf_database_router/cisco_ios_show_ip_ospf_database_router.yml
Show resolved
Hide resolved
Fix bug with detecting hexadecimal LSA sequence numbers Co-Authored-By: Mikhail Yohman <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After these changes, I think we're good to go. The parsed data matches what I would expect.
tests/cisco_ios/show_ip_ospf_database_router/cisco_ios_show_ip_ospf_database_router.yml
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome! Thanks for your contribution.
This isn't the easiest template to start off with either, but we appreciate your contribution!
ISSUE TYPE
COMPONENT
cisco_ios_show_ip_ospf_database_router
SUMMARY
show ip ospf database router
on Cisco IOS.Let me know if any changes are needed - thank you!