feature: Oxidized basic config search #5333

Merged
merged 3 commits into from Jan 9, 2017

Projects

None yet

5 participants

@Rosiak
Contributor
Rosiak commented Jan 6, 2017 edited

DO NOT DELETE THIS TEXT

Please note

Please read this information carefully. You can run ./scripts/pre-commit.php to check your code before submitting.

Hacked this together this eve.

Depending on the progress of ytti/oxidized-web#111

This enables you to leverage the Oxidized's API Config search function, giving you the ability to search through configs stored in Oxidized. Example could be that you want to find all the Cisco devices with "no service password-encryption" command set.

This is VERY basic, code should be refactored and prettified some.
Output should also be better as it only yields matches for now.
Feel free to take it from here, at least this is a start.

screen shot 2017-01-07 at 00 10 04

Rosiak added some commits Jan 6, 2017
@Rosiak Rosiak feature: Basic Oxidized Config Search 307319f
@Rosiak Rosiak welcome to 2017
309b66d
@Rosiak Rosiak changed the title from Oxidized basic config search to feature: Oxidized basic config search Jan 6, 2017
@laf laf used helper functions for input / output
df0cd97
@scrutinizer-notifier

The inspection completed: 1 updated code elements

@laf
Member
laf commented Jan 8, 2017

@Rosiak can you give this small update a test, it's just to use the helper functions to clean and display code.

@Rosiak
Contributor
Rosiak commented Jan 9, 2017

@laf Works fine, thanks!

@laf laf merged commit 92aab16 into librenms:master Jan 9, 2017

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@DonovanEsterhuizen

Where in th ui can I find this?

@DonovanEsterhuizen

Has this been released yet?

@laf
Member
laf commented Jan 13, 2017

It's already live in the dev / daily branch but relies on an update from oxidized which afaik isn't yet released.

@Rosiak Rosiak deleted the Rosiak:oxidized-basic-config-search branch Feb 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment