-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Update iosxr cliconf plugin #43837
Update iosxr cliconf plugin #43837
Conversation
Fixes ansible#39056 * Update iosxr cliconf plugin * Modify iosxr module_utils code to support refactored cliconf plugin api's * Other minor changes
Integration test run
|
lib/ansible/plugins/cliconf/iosxr.py
Outdated
if prompt.endswith(')#'): | ||
self.send_command('abort') | ||
if admin and 'admin-' in prompt: | ||
self.send_command('abort') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should be 'exit' instead of 'abort'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pushing the config on device in candidate datastore that is already present and after that fetching the diff won't result in output. However while existing (without commit since no diff) will result in a prompt on iosxr.
RP/0/0/CPU0:an-iosxr-01(config)#interface Loopback888
RP/0/0/CPU0:an-iosxr-01(config-if)#description test for ansible
RP/0/0/CPU0:an-iosxr-01(config-if)#exit
RP/0/0/CPU0:an-iosxr-01(config)#exit
Uncommitted changes found, commit them before exiting(yes/no/cancel)? [cancel]:
Since prompt is not handled it triggers command timeout. That's the reason abort is used
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
iosxr- lgtm
* devel: (30 commits) Prevent data being truncated over persistent connection socket (#43885) Fix eos_command integration test failures (#43922) Update iosxr cliconf plugin (#43837) win_domain modules: ensure Netlogon service is still running after promotion (#43703) openvswitch_db : Handle column value conversion and idempotency in no_key case (#43869) Fix typo Fix spelling of ansbile to ansible (#43898) added platform guides for NOS and VOSS (#43854) Fix download URL for yum integration test. New module for managing EMC VNX Block storage (#42945) Docker integration tests: factorize setup (#42306) VMware: datastore selection (#35812) Remove unnecessary features from cli_command (#43829) [doc] import_role: mention version from which behavior changed and fix some typos (#43843) Add source interface and use-vrf features (#43418) Fix unreferenced msg from vmware_host (#43872) set supports_generate_diff to False vyos (#43873) add group_by_os_family in azure dynamic inventory (#40702) ansible-test: Create public key creating Windows targets (#43760) azure_rm_loadbalancer_facts.py: list() takes at least 2 arguments fix (#29046) (#29050) ...
SUMMARY
Fixes #39056
refactored cliconf plugin api's
ISSUE TYPE
COMPONENT NAME
module_utils/network/ios/ios.py
module_utils/network/iosxr/iosxr.py
modules/network/iosxr/iosxr_command.py
modules/network/iosxr/iosxr_config.py
modules/network/iosxr/iosxr_facts.py
modules/network/iosxr/iosxr_interface.py
plugins/cliconf/init.py
plugins/cliconf/iosxr.py
plugins/cliconf/junos.py
ANSIBLE VERSION
ADDITIONAL INFORMATION