ansible-junos-stdlib icon indicating copy to clipboard operation
ansible-junos-stdlib copied to clipboard

Published 20 hours ago • verified publisher icon Juniper

implement rpc timeout

Open apurvaraghu opened this issue 1 year ago • 1 comments

fix issue #607

apurvaraghu avatar Feb 13 '24 05:02 apurvaraghu

I just investigated this issue. It seems that timeout behaviour of commit_confirmed got broken with this commit: https://github.com/Juniper/ansible-junos-stdlib/commit/0cdae0e4d0274c5826da5ba74dec0ecbf7c1e999 .

It introduced timeout param for commit_confirmed with default value of 30.

However, the only place where commit_configuration is called is https://github.com/Juniper/ansible-junos-stdlib/blob/9e39e0023f25ffa4e4d430d5905e60299018ad9a/ansible_collections/juniper/device/plugins/modules/config.py#L1229 and timeout parameter is not passed here, so default value is always used.

I think more proper solution would be to pass "timeout" parameter to this call if it was passed to module.

n3wtype avatar Apr 18 '24 13:04 n3wtype