Change in vdsm[master]: Removing support in sysv and upstart

dkuznets at redhat.com dkuznets at redhat.com
Mon Jun 15 11:49:21 UTC 2015


Dima Kuznetsov has posted comments on this change.

Change subject: Removing support in sysv and upstart
......................................................................


Patch Set 8: Code-Review+1

(1 comment)

https://gerrit.ovirt.org/#/c/40726/8/vdsm/clientIF.py
File vdsm/clientIF.py:

Line 520:                 vmId, fileType = f.split(".", 1)
Line 521:                 exts = ["guest.socket", "monitor.socket",
Line 522:                         "stdio.dump", "recovery"]
Line 523:                 if fileType in exts and vmId not in self.vmContainer:
Line 524:                     self.log.debug("removing old file " + f)
+1 looks much better
Line 525:                     utils.rmFile(constants.P_VDSM_RUN + f)
Line 526:             except:
Line 527:                 pass
Line 528: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib9af14b3d78badc5250042508d25f294dc514a2d
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dima Kuznetsov <dkuznets at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list