-
Notifications
You must be signed in to change notification settings - Fork 745
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
Bugfix: cisco_ios_show_running_partition_access_list #808
Conversation
@jmcgill298 thanks for looking into this PR. I never learned to rebase without screwing the log :( |
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.
Same as #809
@jmcgill298 I believe they should be fine now. |
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.
Looks like there is still something a little off.
ISSUE TYPE
COMPONENT
cisco_ios_show_running_partition_access-list.textfsm
show running partition access-list
SUMMARY
Cisco IOS-XE outputs ACE line number on later versions.
Added Value LINE_NUM and made it backwards compatible.
Introduced Value ICMP_TYPE to amend capturing the icmp type eg: 'echo=reply'