Skip to content
This repository has been archived by the owner on May 10, 2019. It is now read-only.

Add support for Cisco Small Business switches #88

Open
wants to merge 38 commits into
base: master
Choose a base branch
from

Conversation

jnorell
Copy link
Contributor

@jnorell jnorell commented Jan 26, 2018

No description provided.

@dotwaffle
Copy link
Owner

Jesse,

I just want to say: this is fantastic work, thank you... However, I'm trying to deprecate this fork. Upstream has diverged significantly from this code, and has implemented git in a different way.

I'd strongly recommend posting this patch to the mailing list at http://www.shrubbery.net/mailman/listinfo/rancid-discuss and trying to get it included in the next version.

Thanks again, and best of luck!

@jnorell
Copy link
Contributor Author

jnorell commented Jan 30, 2018

Yes, I know/agree we should move to the official version; it's just a lot less work to update the one we're using (I'll need to port numerous device support to the new version when we finally do). But I could post the patch and see if someone wants to convert it to version 3...

Thanks

samdoran and others added 28 commits May 16, 2018 10:37
This more accurately reflects the expanded capabilities of RANCID. It's not just for
routers anymore.
Update specfile and changelog.
Also update commit messages and log messages.
The Socket module included with the version of Perl in CentOS 6 does not include
inet_pton (or the version is too old). Using inet_pton from Socket6 solves this issue.

See this bug report for more details https://bugzilla.redhat.com/show_bug.cgi?id=1224143
Modify the regexp so it will match point release like '3.2' and not just an integer.
This change is from the 3.2 upstream.
On certain routers, Load and Time showup in the output and change every time.
This creates noisy diffs.
The main additions are additional error checking for invalid commands that
caused the run to fail on certain Nexus devices.

Remove commented out and unnecessary code.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants