Add timeout parameter #69
Conversation
|
Is this draft pull request still valid? I thought timeout was already configurable in testbed file. |
|
@miott This PR is still valid, but it's waiting on ncclient/ncclient#550 Although you can pass a timeout through the testbed it wasn't processed in yang in certain cases, and even once it is processed it's not being passed through properly in ncclient. So it would always use the default timeout. |
|
@GerriorL can we merge this? |
|
@GerriorL, does timeout work now or not? |
|
|
||
| # check timeout | ||
| try: | ||
| defaults['timeout'] = int(self.connection_info['timeout']) |
There was a problem hiding this comment.
https://github.com/CiscoTestAutomation/yang/blob/master/connector/src/yang/connector/netconf.py#L266
self.timeout already exist via __init__. you might want to use this instead to be simple.
and please be aware of that the timeout is already used here. https://github.com/CiscoTestAutomation/yang/blob/master/connector/src/yang/connector/netconf.py#L284
so the timeout value will be shared between this and your code.
6 participants
Allow timeout to be user configurable as netconf connections hits ncclient default timeout.
Updated ncclient to allow a custom parameter to be passed
ncclient/ncclient#543