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