From Dan Kenigsberg <[email protected]>:

Dan Kenigsberg has submitted this change and it was merged.

Change subject: jsonrpcvdscli: pop timeout before preparing args
......................................................................


jsonrpcvdscli: pop timeout before preparing args

We get this error every time we add custom timeout:
Following parameters [u'_transport_timeout'] were not recognized.

_transport_timeout is no a command parameter but an indicator to
change default timeout to a new value.
So, it shouldn't be added to params and should be removed from kwargs
before that.

Change-Id: If89d2d96b183aaea13446c4bc6b940f4ea40cd62
Signed-off-by: Irit Goihman <[email protected]>
---
M lib/vdsm/jsonrpcvdscli.py
1 file changed, 2 insertions(+), 3 deletions(-)

Approvals:
  Piotr Kliczewski: Looks good to me, approved
  Nir Soffer: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Irit Goihman: Verified



-- 
To view, visit https://gerrit.ovirt.org/71404
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If89d2d96b183aaea13446c4bc6b940f4ea40cd62
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Irit Goihman <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Irit Goihman <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Piotr Kliczewski <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
_______________________________________________
vdsm-patches mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to