-
Notifications
You must be signed in to change notification settings - Fork 931
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
Backup fails after updating to 0.17 #583
Comments
I'm traveling until 21st or so, currently on somewhere over greenland with terrible Internet. Could this be caused by e638e7c - can you change the 'empty' ::: to :nil:nil: |
Yes that works. But it is confusing to be honest. I though it would be a wise idea to put nil in all unused parts of my config to make it readable. But then stuff breaks as well. So, some fields need nil and some don't. See:
While I do understand this field wants 'ssh' or 'telnet' it would be great if it would just do the defaults when it has nil in it. Since using :: in that part works that way. Also, just removing the ::::: from the end of the router definition works as well. |
That looks weird, it looks like we're getting string 'nil', not type nil. If it were type nil, then it should return ssh, telnet: https://github.com/ytti/oxidized/blob/master/lib/oxidized/node.rb#L203 I can't immediately figure out, why it's string "nil" not nil, like it should. This should turn it into type nil, because of: Somehow it looks like that's not working, and string 'nil' gets returned. |
Please understand that I'm not the best coder and Ruby is definitely not my cup of tea, but, if I look at the following:
Doesn't that mean that it only sends vars of vars_map to node_var_interpolate and not the vars that are in map? So, with the following config:
the settings 0 to 5 will not be send to node_var_interpolate and will just be processed as strings. That would explain what I am seeing. AFAIK only setting 0 is mandatory and all others might have default settings which are inherited from the config and may have 'nil' in the router.db. |
Good catch, that's it! |
Woah, you're fast ;-) |
Lemme know if it works and I'll release 0.17.1 |
Yup, this fixes it. |
* 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
Today I've updated oxidized to version 0.17 and somehow my smartedges will not backup anymore. Debug output gives:
My routerdb looks like:
I'm a bit lost why this is happening. Especially with the ssh error right in the beginning. Any pointers or ideas?
Sander
The text was updated successfully, but these errors were encountered: