-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Adding Cisco AP to Device Types #3151
base: develop
Are you sure you want to change the base?
Conversation
To make the test go through change the base_prompt variable for cisco_ap to Please include |
I don't see why changing the base_prompt to something else will help. I changed it anyway and got two other tests include strip prompt to fail. I did deep dive the test and found that paramiko was duping the base prompt. So the strip is working just isn't striping twice. Here is the debug:
Pre strip
After strip
|
cisco_ap is set to no_config but here is the shorten results for test_netmiko_config:
|
Adding cisco_ap to supported device types. Tested on models 3700, 3800, and 9100 WAPs.
Example banner:
Updated test example and below is the results.
Test Results: