Adding new templates for show (radius|tacacs) summary output #855
Conversation
mjbear
requested review from
FragmentedPacket,
itdependsnetworks,
jmcgill298 and
jvanderaa
as code owners
|
Similar to another of my PRs, this one will need rebased as well as a few modifications to the template (filldown). |
mjbear
force-pushed
the
cisco_wlc_auth_protocols
branch
from
6e327c2 to
b287059
Compare
…addresses in unique variables (since server lists could technically differ based on what is possible via CLI configuration)
…ses in unique variables
mjbear
force-pushed
the
cisco_wlc_auth_protocols
branch
from
b287059 to
a028369
Compare
| Value Filldown MAC_DELIM_ACCT_MSG (\S+) | ||
| Value Filldown AUTHC_FRAMED_MTU (\d+) | ||
| Value Required INDEX (\d+) | ||
| Value AUTHC_SRV_ADDR (\S+) |
There was a problem hiding this comment.
Can you change AUTHC and AUTHZ in these templates to the full names? e.g. AUTHORIZATION and AUTHENTICATION. It took me a few minutes to realize what it was.
There was a problem hiding this comment.
I would make ACCT long as well ACCOUNTING.
There was a problem hiding this comment.
Replaced abbreviations with entire word.
| Value AUTHC_SRV_ADDR (\S+) | ||
| Value ACCT_SRV_ADDR (\S+) | ||
| Value TYPE (\S+) | ||
| Value SRV_PORT (\d+) |
There was a problem hiding this comment.
I'm on the fence, but thinking these may not need the SRV_ prefix.
There was a problem hiding this comment.
Removed SRV_ from several variables.
…ing with the full words; removed SRV_ from several variables out of conciseness; removed _STR from a variable name out of conciseness; added state changes in several sections just in case the output should later be in a different order
ISSUE TYPE
COMPONENT
cisco_wlc_ssh_show_radius_summary.textfsm, cisco_wlc_ssh, show radius summary
cisco_wlc_ssh_show_tacacs_summary.textfsm, cisco_wlc_ssh, show tacacs summary
SUMMARY
Add parsing of Cisco WLC RADIUS and TACACS summary output so servers' addressing, statuses, and other settings can be programmatically accessed.