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

Adjusted hp_procurve_show_vlans.template to account for varying outputs. #148

Merged
merged 6 commits into from
Jan 22, 2018

Conversation

tsimson
Copy link
Contributor

@tsimson tsimson commented Sep 14, 2017

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT

hp_procurve_show_vlans.template

SUMMARY

@itdependsnetworks
Copy link
Contributor

I'm not sure how this is passing in regards to the vlan name? (\S+) matching Router Vlan

@itdependsnetworks
Copy link
Contributor

would this work?

Value VLAN_ID (\d+)
Value NAME (\S.+?)
Value STATUS (\S+)
Value VOICE (Yes|No)
Value JUMBO (Yes|No)

Start
  ^.*VLAN ID -> VLAN

VLAN
  ^\s+${VLAN_ID}\s+${NAME}\s+\|?\s?${STATUS}\s+${VOICE}\s+${JUMBO}? -> Record

@tsimson
Copy link
Contributor Author

tsimson commented Oct 16, 2017

My first attempt at fixing this looked like it included the pipe...and I think that your example might do that as well. I ended up with a look ahead that looked for but didn't include the pipe. It passed the CI tests this time.

@itdependsnetworks itdependsnetworks merged commit 62aa161 into networktocode:master Jan 22, 2018
jvanderaa pushed a commit that referenced this pull request Nov 10, 2021
Adjusted hp_procurve_show_vlans.template to account for varying outputs.
# 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