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

Fix command checking of echo in _gen_rollback_cfg #26

Merged
merged 1 commit into from
Aug 24, 2020

Conversation

tmshlvck
Copy link
Contributor

Fix generated pattern for command "copy running-config ..." in
_gen_rollback_cfg. The command has to be explicit in the full form
"copy running-configuration ..." to avoid breakage when the command
echo (which is full form in case of OS10 10.5.1.3) is compared with
the short form patern and fails the check.

The resulting exception was:
netmiko.ssh_exception.NetmikoTimeoutException: Timed-out reading
channel, data not available.

This fixes issue #24

Fix generated pattern for command "copy running-config ..." in
_gen_rollback_cfg. The command has to be explicit in the full form
"copy running-configuration ..." to avoid breakage when the command
echo (which is full form in case of OS10 10.5.1.3) is compared with
the short form patern and fails the check.

The resulting exception was:
netmiko.ssh_exception.NetmikoTimeoutException: Timed-out reading
channel, data not available.
@javeedf javeedf merged commit f718752 into napalm-automation-community:master Aug 24, 2020
@neuahouse
Copy link

@javeedf Can you release a new version that includes this fix?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants