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

Add INPUT_PACKETS, INPUT_ERRORS, OUTPUT_PACKETS, OUTPUT_ERRORS fields to cisco_ios_show_interfaces & cisco_nxos_show_interface templates #307

Conversation

wvandeun
Copy link
Contributor

ISSUE TYPE
  • Update Template Pull Request
COMPONENT

cisco_ios_show_interfaces.template, cisco_ios, show interfaces
cisco_nxos_show_interface.template, cisco_nxos, show interface

SUMMARY
  • Add INPUT_PACKETS, INPUT_ERRORS, OUTPUT_PACKETS, OUTPUT_ERRORS fields to cisco_ios_show_interfaces & cisco_nxos_show_interface templates
  • Updated cisco_ios_show_interfaces & cisco_nxos_show_interface tests, to take the new fields into account.

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.

@Enzzzy It is easy for extra space characters to be missed, so I like to be more generous with matching spaces. \s should be \s+ and end-of-line should allow for spaces, \s*.

templates/cisco_ios_show_interfaces.template Outdated Show resolved Hide resolved
templates/cisco_ios_show_interfaces.template Outdated Show resolved Hide resolved
templates/cisco_ios_show_interfaces.template Outdated Show resolved Hide resolved
templates/cisco_ios_show_interfaces.template Outdated Show resolved Hide resolved
…isco_ios_show_interfaces & cisco_nxos_show_interface templates

* Updated cisco_ios_show_interfaces & cisco_nxos_show_interface tests, to take the new fields into account.
@wvandeun wvandeun force-pushed the feature_add_interface_input_out_packets_errors_ios_nxos branch from 8a8c196 to 9926451 Compare January 7, 2019 21:44
@wvandeun
Copy link
Contributor Author

wvandeun commented Jan 7, 2019

@jmcgill298 I have made all the requested changes.

@jmcgill298 jmcgill298 merged commit 6a0ab67 into networktocode:master Jan 7, 2019
jvanderaa pushed a commit that referenced this pull request Nov 10, 2021
…isco_ios_show_interfaces & cisco_nxos_show_interface templates (#307)

* Updated cisco_ios_show_interfaces & cisco_nxos_show_interface tests, to take the new fields into account.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants