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.

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

Merged
merged 9 commits into from
Mar 8, 2020
Merged

New Template: cisco_ios_show_ip_ospf_database_router #606

merged 9 commits into from
Mar 8, 2020

Conversation

ChristopherJHart
Copy link
Contributor

ISSUE TYPE
  • New Template Pull Request
COMPONENT

cisco_ios_show_ip_ospf_database_router

SUMMARY
  • Parse relevant information from output of show ip ospf database router on Cisco IOS.
  • Output pulled from an IOSv node running IOS 15.7(3)M3.

Let me know if any changes are needed - thank you!

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.

@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).

@ChristopherJHart
Copy link
Contributor Author

@jmcgill298 Apologies! I've modified the TextFSM template states accordingly.

Thank you!

@FragmentedPacket FragmentedPacket added changes_requested Waiting on user to address feedback New Template labels Mar 6, 2020
Copy link
Contributor

@FragmentedPacket FragmentedPacket left a 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.

Copy link
Contributor

@FragmentedPacket FragmentedPacket left a 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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes_requested Waiting on user to address feedback New Template
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants