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

Adv 802dot11 channel #526

Merged
merged 8 commits into from
Nov 8, 2019
Merged

Adv 802dot11 channel #526

merged 8 commits into from
Nov 8, 2019

Conversation

timjsmith24
Copy link
Contributor

ISSUE TYPE
  • New Template Pull Request
COMPONENT

cisco_wlc_ssh_show_advanced_802.11a_channel.template, .*, cisco_wlc_ssh, show advanced 802.11a channel

SUMMARY

^\s+Maximum
^\s+Channel\sDwell\sTimes
^\s+802\.11a\s5\sGHz\sAuto\-RF Channel Lists*$$
^.*Allowed\sChannel\sList\.*\s${DCA_CHANNELS}s*$$ -> Channels
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This attempts to capture allowed channels on both 5 and 4.9Ghz.

I would say that we'd want to capture those in separate variables and then post process them in needed.

This would be to do state transitions on L37 instead of L38 and separate variables.

@FragmentedPacket FragmentedPacket added changes_requested Waiting on user to address feedback New Template labels Oct 31, 2019
@timjsmith24
Copy link
Contributor Author

Ok I separated out the 5GHz channels. I didn't create separate variables for the 4.9GHz channels as those are not used

@jmcgill298 jmcgill298 merged commit 9e0ce41 into networktocode:master Nov 8, 2019
thomasblass pushed a commit to thomasblass/ntc-templates that referenced this pull request Oct 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes_requested Waiting on user to address feedback New Template
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants