Change in vdsm[master]: tests: Adding functional tests for networking

asegurap at redhat.com asegurap at redhat.com
Thu Jun 6 01:51:17 UTC 2013


Antoni Segura Puimedon has posted comments on this change.

Change subject: tests: Adding functional tests for networking
......................................................................


Patch Set 12: (1 inline comment)

....................................................
File tests/functional/utils.py
Line 88:             # cleanup
Line 89:             cmd_service = [service.cmd, "vdsm-restore-net-config", "start"]
Line 90:             utils.execCmd(cmd_service, sudo=True)
Line 91:             # keeps traceback
Line 92:             import sys
can't you just reraise doing:
raise
Line 93:             exc_info = sys.exc_info()
Line 94:             raise exc_info[0], exc_info[1], exc_info[2]
Line 95:     return wrapper
Line 96: 


--
To view, visit http://gerrit.ovirt.org/14840
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic3be71db9dc0b92c443b87e22fe06f920055c4d3
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Giuseppe Vallarelli <gvallare at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Giuseppe Vallarelli <gvallare at redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer at redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list