Dan Kenigsberg has submitted this change and it was merged. Change subject: jsonrpc: default args values in wrong order ......................................................................
jsonrpc: default args values in wrong order Default arguments values of a method were passed in wrong order due to wrong if condition. Change-Id: I4e256468833851cd4ce866297de916332a560f2f Signed-off-by: pkliczewski <[email protected]> Bug-Url: https://bugzilla.redhat.com/1148499 Reviewed-on: http://gerrit.ovirt.org/33861 Reviewed-by: Oved Ourfali <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M tests/bridgeTests.py M vdsm/rpc/Bridge.py 2 files changed, 34 insertions(+), 1 deletion(-) Approvals: Piotr Kliczewski: Verified Dan Kenigsberg: Looks good to me, approved Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/33861 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4e256468833851cd4ce866297de916332a560f2f Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
