Change in vdsm[master]: tests: dummy: provide more info upon failure

osvoboda at redhat.com osvoboda at redhat.com
Mon Jan 19 12:37:35 UTC 2015


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 <danken at redhat.com>
Gerrit-Reviewer: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list