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 to get_config for cli-engine classic and config-mode mixed devices #32

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

FranC1010
Copy link
Contributor

Ansible playbooks that are run against devices configured with cli-engine classic-cli md-cli and configuration-mode mixed, cause an Ansible 'command timeout' error at the get_config stage.

Notably, when the module sends the 'info running' command, the Ansible task fails because of the Nokia default prompt to continue (environment more true).

This was successfully tested against TiMOS 21.10.

…vices

Ansible playbooks that are run against devices configured with cli-engine classic-cli md-cli and configuration-mode mixed, cause an Ansible 'command timeout' error at the get_config stage.

Notably, when the module sends the 'info running' command, the Ansible task fails because of the Nokia default prompt to continue (environment more true).

This was successfully tested against TiMOS 21.10.
@nicoduck
Copy link

nicoduck commented Feb 9, 2025

This bug cost me over an hour, until I found this open pull request. Manually applying the patch fixed the issues for me.

@hellt
Copy link
Member

hellt commented Feb 9, 2025

/cc @wisotzky @jgcumming

@jgcumming jgcumming self-assigned this Feb 12, 2025
@jgcumming jgcumming added enhancement New feature or request in review Being reviewed be a maintainer labels Feb 12, 2025
@jgcumming jgcumming merged commit 184c616 into nokia:master Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request in review Being reviewed be a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants