Change in vdsm[master]: Nit fixes of supervdsmServer

danken at redhat.com danken at redhat.com
Mon Dec 10 07:16:18 UTC 2012


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Nit fixes of supervdsmServer
......................................................................


Nit fixes of supervdsmServer

use utils.rmFile instead of unlink
Raise err in supervdsmServer final exit is useless,
remove it.

Change-Id: Ie87c2332ca6342044e0c157231ea8c076ad622cc
Signed-off-by: Royce Lv<lvroyce at linux.vnet.ibm.com>
---
M vdsm/supervdsmServer.py
1 file changed, 1 insertion(+), 2 deletions(-)

Approvals:
  Royce Lv: Verified
  Yaniv Bronhaim: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved
  Zhou Zheng Sheng: Looks good to me, but someone else must approve


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie87c2332ca6342044e0c157231ea8c076ad622cc
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Royce Lv <lvroyce at linux.vnet.ibm.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Royce Lv <lvroyce at linux.vnet.ibm.com>
Gerrit-Reviewer: Shu Ming <shuming at linux.vnet.ibm.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.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