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 for cisco_ios_show_running-config_partition_access-list template #598

Merged
merged 7 commits into from
Feb 27, 2020
Merged

Fixes for cisco_ios_show_running-config_partition_access-list template #598

merged 7 commits into from
Feb 27, 2020

Conversation

jpobeda
Copy link
Contributor

@jpobeda jpobeda commented Feb 23, 2020

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT

cisco_ios_show_running-config_partition_access-list.textfsm
cisco_ios
show running-config | section access-list

SUMMARY
  • PROTOCOL capture group wouldn't match numbers.
  • SRC_PORT_RANGE_START capture group was incorrect.
  • TCP flags weren't captured.
  • Comment typo on "Record numerd ACL Remark"
  • COMMENT is now a List, allowing for consecutive remarks.

@jpobeda
Copy link
Contributor Author

jpobeda commented Feb 24, 2020

Hi @jmcgill298 ,

Blame github reviewers suggestions hehe

How do we get around this one? I added a missing/ignored capture group for tcp_flags. So even though it works backwards, people will get a new key/value pair on their dictionaries.

@jmcgill298 jmcgill298 merged commit b34587d into networktocode:master Feb 27, 2020
@jmcgill298
Copy link
Contributor

Thanks @jpobeda

@jpobeda jpobeda deleted the ios_partition_acl branch February 27, 2020 18:48
thomasblass pushed a commit to thomasblass/ntc-templates that referenced this pull request Oct 25, 2020
* Add Capture Group `TCP_FLAG`
* Convert `COMMENT` to a List
* Update `PROTOCOL` to include numbered protocols
jvanderaa pushed a commit that referenced this pull request Nov 10, 2021
* Add Capture Group `TCP_FLAG`
* Convert `COMMENT` to a List
* Update `PROTOCOL` to include numbered protocols
# 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