Ondřej Svoboda has posted comments on this change. Change subject: tests: dummy: provide more info upon failure ......................................................................
Patch Set 1: Code-Review+1 I see no problem here :-) My nitting on the % operator says that in this patch it's actually nicely used, short and most importantly, the arguments are nonproblematic. (You know, the operator treats its arguments ambiguously if you don't wrap them in a tuple.) -- To view, visit http://gerrit.ovirt.org/37047 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4452ef5c03c14087b7395584cfa10d76ca8a3a88 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Ido Barkan <[email protected]> Gerrit-Reviewer: Ondřej Svoboda <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
