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 ubiquiti edgerouter show dhcp leases #1097

Merged
merged 2 commits into from
May 22, 2022

Conversation

jvanderaa
Copy link
Contributor

ISSUE TYPE
  • New Template Pull Request
COMPONENT

ubiquiti_edgerouter_show_dhcp_leases

SUMMARY

Replaces #1063 due to index file conflicts.

Before

ubiquiti_edgerouter_show_version.textfsm, .*, ubiquiti_edgerouter, sh[[ow]] ve[[rsion]]

After

ubiquiti_edgerouter_show_dhcp_leases.textfsm, .*, ubiquiti_edgerouter, sh[[ow]] dhcp leases
ubiquiti_edgerouter_show_version.textfsm, .*, ubiquiti_edgerouter, sh[[ow]] ve[[rsion]]

@diepes
Copy link
Contributor

diepes commented Mar 13, 2022

Do i understand correctly regarding the placing of the template match in ntc_templates/templates/index.
e.g.

ruckus_fastiron_show_arp.textfsm, .*, ruckus_fastiron, sh[[ow]] ar[[p]]

ubiquiti_edgerouter_show_dhcp_leases.textfsm, .*, ubiquiti_edgerouter, sh[[ow]] dhcp leases
ubiquiti_edgerouter_show_version.textfsm, .*, ubiquiti_edgerouter, sh[[ow]] ve[[rsion]]

ubiquiti_edgeswitch_show_version.textfsm, .*, ubiquiti_edgeswitch, sh[[ow]] ve[[rsion]]
  1. Alphabetically ubiquity is after ruckus
  2. length wise ubiquiti_edgerouter_show_dhcp_leases > ubiquiti_edgerouter_show_version thus comes before it.

@jmcgill298 jmcgill298 merged commit acf3684 into master May 22, 2022
@jmcgill298 jmcgill298 deleted the add-ubiquiti-edgerouter-show-dhcp-leases branch May 22, 2022 18:49
guillaume-mbali pushed a commit to unyc-io/ntc-templates that referenced this pull request Apr 12, 2023
cppmonkey pushed a commit to cppmonkey/ntc-templates that referenced this pull request Oct 25, 2023
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.

3 participants