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

hp_procurve_show_vlans.template empty output #124

Closed
cyberdog478 opened this issue Jun 7, 2017 · 3 comments
Closed

hp_procurve_show_vlans.template empty output #124

cyberdog478 opened this issue Jun 7, 2017 · 3 comments

Comments

@cyberdog478
Copy link

The HP Procurve Show Vlans template regex is unable to handle the addition of the pipe separator used for separating the Name and Status colums (example below) which results in null returned. The following change to the regex resolves this issue (adding \S+\s+ to capture and ignore the pipe and white space):

Line 11:
^\s+${VLAN_ID}\s+${NAME}\s+\S+\s+${STATUS}\s+${VOICE}\s+${JUMBO} -> Record

Original Line 11:
^\s+${VLAN_ID}\s+${NAME}\s+${STATUS}\s+${VOICE}\s+${JUMBO} -> Record

  VLAN ID Name                             | Status     Voice Jumbo
  ------- -------------------------------- + ---------- ----- -----
  1       DEFAULT_VLAN                     | Port-based No    No  


@itdependsnetworks
Copy link
Contributor

@cyberdog478 I attempted to make the updates as you indicated, but it broke the existing test data. Can you provide full output (meaning cli prompt to cli prompt) of your show commands?

@itdependsnetworks
Copy link
Contributor

just a reminder @cyberdog478

@rkerr
Copy link

rkerr commented Sep 18, 2017

Looks like #148 would resolve this without breaking the test cases.

The | between the name and status appears on most of the newer hardware but is missing on the older stuff. The jumbo frames column won't always be present - depends if the hardware supports jumbo frames.

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

No branches or pull requests

3 participants