Aruba Mobility Access Switch does not provide "show slots" or "show license" commands #669
Comments
|
If we can reasonably avoid duplicate models, we should. If there are some commands which are only valid in other platform, it should be somewhat simple matter of just regexping the complaint out of the output, leaving no output for systems which don't support the commands. We can also in model gather stuff like version, and store information about the platform in variable, and then run different commands for different hw models. It's ultimately up-to-you which solution is cleanest, all new model or more complexity to existing model. |
|
I though about using regex to clean the error, but worry about a legitimate error not getting written. I think handling different devices within the model may be the best approach as Aruba currently uses ArubaOS for different devices with different capabilities. For example the latest stable ArubaOS version for the Access Mobility Switches is 7.4.1.7. For Aruba Instant the latest stable ArubaOS version is 6.4.4.8-4.2. Can you recommend a model to review that gathers information and runs different commands based on that information? I want to see how it has been done before so I can try to be consistent. From our hardware both are reported as ArubaOS |
|
If the error is like 'invalid command', then what is there to worry if you match exactly to that line and remove that line? Then if that command complains about invalid command, it's gone but it'll only complain on boxes not supporting it? I think there are some legit models doing conditional commands, but there is still day1 contrived example in junos.rb (it's not actually that anyone needed that, i just wanted to add to the model example how to do it without having use case). I know some model uses that for real use-case, just can't remember which. |
|
Ok I'll go down that route until another problem facilitates a bigger solution or it causes problems. For now it will be the easiest and cleanest solution. I'm not sure when I will get to write and test, but this is a minor annoyance at this point so it should not hinder the use of the current model. The two error lines that I will try to remove are: Thank you. |
|
|
This is the proposed fix for issue ytti#669. Let me know if there is anything that I need to fix. This appears to be working correctly for the switch I have in house (S2400-24P). However the model is not working for the Aruba Instant AP (IAP) (virtual) controller. I hope to submit a fix for those once I have it working. The problems with that device appear to be unrelated to these changes as the prompt is incorrect.
* Remove extra new lines added by export command * Updated Dockerfile to use ruby 2.3 * ruby2.1 behaves oddly with this style of dependcy fixed ytti#565 * rollback `alvarion` model wrongly removed in 56bac97 * interpolate node variable values Empty value was variably considered "" or nil, now it will always be "" string. Now "nil", "false", "true" strings are converted to their respective types nil, false and true. This also means we cannot have verbatim strings by that name, like if your password was "false", you're shit out of luck. If this is the case, we can add some __false__ hack or consider other similar solutions. Fixes ytti#500 Fixes ytti#534 May break stuff with ""/nil changing now, but as it was not consistent to begin with, I find it acceptable. Users now may need to manually enter nil in some fields to regain old behaviour. * update CHANGELOG * bump up version * Redoing the modifications on a up to date base * Redoing the modifications on a up to date base * update model names * add links to models * add comware link name contained -, which my replace regexp didn't allow, thus didn't generate link for that model * regexp missed this model * prompt not captured after sending enabe PW fixes ytti#577 * The \n is not seen as a lineline with ' it need " * Update README.md Added correction related to issue ytti#445 and docker-compose file example * Interpolate also node variables, not just vars map Fixes ytti#583 * Add nginx reverse proxy example * Add apache2 reverse proxy example * Add key authentication to ssh input * bump up version * Use variable vars(:ssh_keys) for private key authentication * Fix blank pfSense configs being saved if regex doesn't match * Update regex to match newer pfSense config files * Add missing dependency for Rugged * Added support for fujitsu blade switches * Added Support for MRV FiberDriver Linux based management module * fetch should return string, not array Also if group was explicitly given, we referred to wrong directory * return nil if we can't find file unsure if we really should, or just raise the error and let consumer of fetch decide what to do with the error * add support for the HPE BladeSystem * add Ruby.gitignore from github.com/github/gitignore * Allow model to specify SSH PTY options * it shouldn't be a private method * add Planet SGS switches * add to README * Update dnos.rb Handles old DNOS switches that do not allow term length 0 prior to enable, and double exits in case of enable.. * expect prompt after sending enable password * Added support for Trango/Trangolink-45 I only have the Trangolink-45 so I'm not sure which other radios this might support. * This adds support for the Casa C1G CMTS This will probably work for the other Casa CMTS as well. * This adds support for D-Link switches This specifically supports DES-3526/3550 series switches. * This adds support for Hatteras Networks devices * renamed alvarion -> alvarion.rb I believe from this: require File.join dir, file+'.rb' That the only way this model would work is if it has a .rb extension. I don't have any of these devices to test if this works. * add support for PLANET SG switches So far, only SGS switches were supported. Now, we check the model type during the 'show version' command, and only execute the 'show transceiver details' command in case an SGS switch was detected (as its not supported on SG models). We will also strip lines containing the current System Time and Uptime. These only appear on SG models, but it's a good idea to strip them anyways. * expect prompt after entering enable password * Update ciscosmb.rb Adapted for SG series, not sure about SF. Please test. * Update ciscosmb.rb Forgot the username prompt change. * Update ciscosmb.rb Added old commands in case of different CLI variations. * Added option to disable ssl verification checks for http source * Updated config options * New hook: awssns - Publish messages to AWS SNS topics * exclude time from output * Update eos.rb * Remove trailing whitespace and enable prompt detection Fixes: ytti#630 * Fix suggested by ytti for issue ytti#610 * Recursively search from one dir above specified Fixes ytti#626 * update changelogs * bump up version * delete secret password if it is called secret * documention: debug inside input is now boolean * Update ios.rb Do the terminal configure after enable since it might be prohibited in non ena * Hide remaining passwords * adding zhone OLT/MX GPON/MetroE/DSL gear * Ignore undefined UTF8 in ironware.rb We have some Brocade MLX devices that are triggering 'raised Encoding::UndefinedConversionError with msg ""\xFF" from ASCII-8BIT to UTF-8"'. Update the ironware.rb file to ignore undefined UTF8 as well as invalid UTF8 * Add support for telnet * Mask out configuration date for fiberdriver Our fiberdriver devices report a timestamp during 'show running-config', which results in a config diff on every run. This patch removes the '+! Configuration saved on 2017/01/10 14:21:20' line from the config, as well as a couple other useless status messages * Ensure config gets returned even if no replacements are done * Ignore fan speed changes for ironware Brocade ICX/FCX log fan speed changes like this: Fan ok, speed (auto): 1<->[[2]]<->3 Fan ok, speed (auto): [[1]]<->2<->3 Remove the current fan speed from the configuration to prevent lots of useless changes from being logged. * Add suggestion to set `ip` variable in CSV reader We have around ~1000 nodes, and oxidized was taking 30s to do the initial config load. After adding the 'ip' variable to our CSV, load time dropped to less than 1 second. * Update ironware.rb Some Brocade devices only have two possible fan speeds. Previously, this only handled three levels of fan speeds. Updated regex will handle both of these cases: Fan 1 ok, speed (auto): 1<->[[2]] Fan 1 ok, speed (auto): 1<->[[2]]<->3 * Tweak the white space for cleaning. * Untested GPG work * Syntax error * Update * GPG support * GPG support working * Requested changes * Update * Don't show error is unsupported This is the proposed fix for issue ytti#669. Let me know if there is anything that I need to fix. This appears to be working correctly for the switch I have in house (S2400-24P). However the model is not working for the Aruba Instant AP (IAP) (virtual) controller. I hope to submit a fix for those once I have it working. The problems with that device appear to be unrelated to these changes as the prompt is incorrect. * Filter out rogue client alerts from AireOS * Added git build * Adjustments * feature: support IP Addressed provided with a netmask * Updated split to be more efficient, thanks ytti! * Update fabricos.rb Ignore Uptime/Power/Date fields in chassisShow and configShow * Restore compatibility with older versions of IronWare Remove \r from the "enable" command. Both \r and \n are interpreted the same on older versions, causing a blank line to be sent as the enable password. Add a delay between the sending of the "logout" command and each "exit" command. Older devices don't seem to be able to keep up with the speed the commands are sent without a delay. * Cleaner way to fix older IronWare devices This works on older IronWare devices without needing a sleep between each command. * added secret removal for fortiOS * nicer regex for both password variants of fortiOS
|
I think this can be closed now. |
In working through an Aruba S2500-24P switch I found that the "show slots" and "show license" command result in a simple error in the output file because the commands do not exists in the firmware. There errors are not really hurting anything so it may be a non-issue, but I like things clean.
I am more than willing to fix this and submit a pull request, but I am rather new to the Oxidized system and do not know how best to handle this within the framework.
Should this be a new model file?
Possibly AOSMAS (Aruba OS Mobility Access Switch) to complement AOSW (Aruba OS Wireless controller)
If not how should this be structured?
--
I currently have a proof of concept installation of Oxidized that I am working with and loving. Thank you for making this.
I hope to spend some time making the Aruba and possibly the Aruba/HPE models better.
The text was updated successfully, but these errors were encountered: