Change in vdsm[master]: net tests: give a more legible skip reason

danken at redhat.com danken at redhat.com
Tue Apr 12 05:43:36 UTC 2016


Dan Kenigsberg has submitted this change and it was merged.

Change subject: net tests: give a more legible skip reason
......................................................................


net tests: give a more legible skip reason

Current text

    SKIP: ('Tested host is not clean (running vs kernel):\nnetworks: %r != %r\nbonds: %r != %r\n', {}, {}, {}, {u'bond0': {u'nics': [], u'options': u'mode=0'}})

is not formatted.

Change-Id: Id641045f5cfc7e06d6afaf369e25adf021b889f9
Signed-off-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/55902
---
M tests/functional/networkTests.py
1 file changed, 6 insertions(+), 5 deletions(-)

Approvals:
  Dan Kenigsberg: Verified; Looks good to me, approved; Passed CI tests

Objections:
  Jenkins CI: Failed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id641045f5cfc7e06d6afaf369e25adf021b889f9
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh at redhat.com>
Gerrit-Reviewer: Ilia Meerovich <iliam at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Lihi Wishnitzer <lihiwish at gmail.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list