network-insight-sdk-generic-datasources icon indicating copy to clipboard operation
network-insight-sdk-generic-datasources copied to clipboard

added secret parameter for enable

Open tnTano opened this issue 6 years ago • 10 comments

With this mod we can access the switch enable mode with the same login password

tnTano avatar Oct 08 '19 13:10 tnTano

@tnTano, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <[email protected]> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

vmwclabot avatar Oct 08 '19 13:10 vmwclabot

@tnTano Do we require switch enable mode ? SDK will only fetch configuration data for reading. We don't want to modify anything on switch. @sbhagwatvnera Your thoughts.

piyushvmware avatar Nov 13 '19 08:11 piyushvmware

@piyushvmware nope, we do not need to modify something but in our switch we have the enable mode with password and without entering in enable mode we cannot execute the scripts command...

tnTano avatar Nov 13 '19 08:11 tnTano

@tnTano Can the enable password be different from existing ssh password ? If yes, then we need to have mechanism for passing enable password as parameter.

For which switch and what commands are you firing? if it is okay to answer. :)

piyushvmware avatar Nov 13 '19 08:11 piyushvmware

@piyushvmware yea, of course it can be different, for our infrastructure the password is the same and of course if more practice if the script prompt for the enable password...

we have a dell N1140 and Dell S5048 (not compatible with the script) and all the script not working, we made this because the script not working and after this we have the integration in vrealize.....

tnTano avatar Nov 13 '19 09:11 tnTano

@tnTano Please add capability to accept enable mode secret password from user like other parameters. If secret password not provided then old logic of login approach should be there. May be setting secret=None would help but need to test.

piyushvmware avatar Nov 13 '19 11:11 piyushvmware

sure, when i have free time i'll update the script

tnTano avatar Nov 13 '19 11:11 tnTano

@tnTano, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <[email protected]> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

vmwclabot avatar Nov 14 '19 15:11 vmwclabot

@tnTano, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <[email protected]> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

vmwclabot avatar Nov 15 '19 14:11 vmwclabot

@tnTano Please merge latest master in to your codebase. Imports are now fixed and is in-sync with Python3. Please use export PYTHONPATH="$PYTHONPATH:../network-insight-sdk-generic-datasources" Existing users are using same. Please refer to README.md for more information.

piyushvmware avatar Feb 26 '20 08:02 piyushvmware