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

Added Avaya ERS templates #155

Merged
merged 17 commits into from
Jan 29, 2018
Merged

Conversation

kadecole
Copy link
Contributor

ISSUE TYPE
  • New Template Pull Request
COMPONENT
  • avaya_ers_show_sys-info, avaya-ers, show sys-info
  • avaya_ers_show_vlan, avaya-ers, show vlan
SUMMARY

This has added support for the commands listed above for Avaya ERS switches.

@kadecole kadecole changed the title Avaya ers Added Avaya ERS templates Sep 28, 2017
@itdependsnetworks
Copy link
Contributor

I prefer to see \S instead of \w, as it included dashes and other characters. Also can you work out VLANPORTMEMBERS to be a proper list, meaning ['1/2-8' , '1/10-19', '1/21-26', '4/47-49'] It is a pain I know, but it is doable. Let me know on slack if you need any help.

@kadecole
Copy link
Contributor Author

Now that #146 has merged, I have updated this PR. I was planning on working on the VLANPORTMEMBERS to be a proper list on a new merge as my first attempt was taking too much time.

@itdependsnetworks
Copy link
Contributor

This should work, as I every other column has to be there, so you can just match everything up to the last space.

Value VLANID (\d+)
Value VLANNAME (\S.*)
Value VLANTYPE (\S+)
Value VLANPROTOCOL (\S+)
Value VLANPID (\S+)
Value VLANACTIVE (\S+)
Value VLANIVLSVL (\S+)
Value VLANMGMT (\S+)
Value List VLANPORTMEMBERS (\S+)

Start
  ^Total\s+VLANs: -> Done
  ^\d+ -> Continue.Record
  ^${VLANID}\s+${VLANNAME}\s+${VLANTYPE}\s+${VLANPROTOCOL}\s+${VLANPID}\s+${VLANACTIVE}\s+${VLANIVLSVL}\s+${VLANMGMT}$$
  ^\t+Port Members:\s+${VLANPORTMEMBERS}$$
  ^\t+\s{14}${VLANPORTMEMBERS}$$


Done

@itdependsnetworks itdependsnetworks merged commit 02edadd into networktocode:master Jan 29, 2018
@kadecole kadecole deleted the avaya-ers branch May 10, 2018 19:37
jvanderaa pushed a commit that referenced this pull request Nov 10, 2021
# 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