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 bgp neighbor #583

Closed

Conversation

kwrobert
Copy link
Contributor

ISSUE TYPE
  • New Template Pull Request
COMPONENT

Template: cisco_ios_show_ip_bgp_neighbor.textfsm
OS: Cisco IOS
Command: `show ip bgp neighbor

SUMMARY

Added support to parse output from show ip bgp neighbor because it contains some information not available in the output of show ip bgp summary.

A lot more output could be retrieved from this command, I only took what I needed for now.

Add support to parse output from `show ip bgp neighbor`.

A lot more output could be retrieved from this command, I only took what
I needed for now.
@FragmentedPacket FragmentedPacket added changes_requested Waiting on user to address feedback New Template labels Jan 29, 2020
@kwrobert kwrobert force-pushed the cisco_ios_show_ip_bgp_neighbor branch from f67d05c to 872213c Compare January 29, 2020 19:14
=============================================================================
BGP neighbor is 175.135.172.146, remote AS 65180, internal link
Description: nsxe-core-rtr1
Member of peer-group HMCO-core for session parameters
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would update any values that are using \w+ and change them to \S+. This will capture the peer_group: 'HMCO-core' within the parsed file.

@FragmentedPacket
Copy link
Contributor

@kwrobert Just an FYI you can pip install tox and then just run tox command for it to run the tests locally before pushing up and waiting for Travis CI

@FragmentedPacket
Copy link
Contributor

Closed by #590

Thanks for your work on this!

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.

2 participants