Change in ovirt-dwh[master]: packaging: services: remove pid when service start fails

ydary at redhat.com ydary at redhat.com
Thu Jan 9 11:16:08 UTC 2014


Yaniv Dary has submitted this change and it was merged.

Change subject: packaging: services: remove pid when service start fails
......................................................................


packaging: services: remove pid when service start fails

the service is running under non privileged user, so it cannot remove
the pid file only clear it.

if service is dead at validation stage its status is printed as:

  dead but pid file exists

to remedy that, if service fails to start we remove the pid if service
start fails and pid file is zero length.

Change-Id: I11a29e58e0fd673eb1df79983b0c09799f9ef0f3
Signed-off-by: Alon Bar-Lev <alonbl at redhat.com>
---
M packaging/services/ovirt-engine-dwhd/ovirt-engine-dwhd.sysv.in
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Alon Bar-Lev: Verified
  Sandro Bonazzola: Looks good to me, but someone else must approve
  Yaniv Dary: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I11a29e58e0fd673eb1df79983b0c09799f9ef0f3
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-dwh
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Yaniv Dary <ydary at redhat.com>



More information about the Engine-commits mailing list