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

net-ssh 3.1.0.rc2 issue with Debian/Ruby2.1 #565

Closed
ijg0 opened this issue Sep 15, 2016 · 5 comments
Closed

net-ssh 3.1.0.rc2 issue with Debian/Ruby2.1 #565

ijg0 opened this issue Sep 15, 2016 · 5 comments
Labels

Comments

@ijg0
Copy link

ijg0 commented Sep 15, 2016

Similar issue to #472 but we are using Debian Jessie (8.5) without Docker. As such we don't have an easy upgrade path to Ruby 2.2 or 2.3 to resolve the net-ssh version issue.

@ytti
Copy link
Owner

ytti commented Sep 15, 2016

You've installed oxidized via gem? Which release? And it installed net-ssh 3.1.0rc2? And if you remove net-ssh completely with gem, and reintall oxidized, it'll fetch net-ssh 3.1.0rc2?

@ijg0
Copy link
Author

ijg0 commented Sep 15, 2016

All correct. We are using oxidized-0.16.3.gem

@ytti
Copy link
Owner

ytti commented Sep 15, 2016

It might be that I don't understand how dependency is set correctly.

If you don't mind, try this:

  1. clone the repo
  2. edit oxidized.gemspec
  3. move the dependencies in an array, [ '>= 3.0.0', '<3.1' ]
  4. build the package rake build
  5. install it gem install pkg/oxidized-0.16.3.gem

Of course uninstall net-ssh and oxidized first.

Will that still install incorrect net-ssh or correct?

@ijg0
Copy link
Author

ijg0 commented Sep 16, 2016

Oddly it still pulls net-ssh 3.1.0rc2.

As a work around installing net-ssh 3.0.2 before oxidized solves the issue.

@ytti
Copy link
Owner

ytti commented Sep 16, 2016

I don't understand what's going on here :(. Need to spend some time testing, but don't have time for it right now. I'll keep the bug open.

@ytti ytti added bug help_wanted No one is working on this issue. If no one fixes it, it will be closed after some time. and removed help_wanted No one is working on this issue. If no one fixes it, it will be closed after some time. labels Sep 16, 2016
@ytti ytti closed this as completed in 5b09674 Sep 16, 2016
MajesticFalcon added a commit to MajesticFalcon/oxidized that referenced this issue Feb 6, 2017
* 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants