Change in vdsm[master]: pre-start: touch/configure vdsm logs

danken at redhat.com danken at redhat.com
Tue Mar 25 19:09:19 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

Change subject: pre-start: touch/configure vdsm logs
......................................................................


pre-start: touch/configure vdsm logs

During pre-start of vdsm we must create vdsm logs files
with proper perms/user/group/ if they doesn't exist.
This situation happens currently because during
ovirt node build it cleans /var/log dir so it will
remove all vdsm logs created during rpm install.

Change-Id: I6ca927cb447eeb24195cff6a1174e0f9a7295214
Signed-off-by: Douglas Schilling Landgraf <dougsland at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/25912
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M configure.ac
M init/vdsmd_init_common.sh.in
2 files changed, 18 insertions(+), 0 deletions(-)

Approvals:
  Douglas Schilling Landgraf: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6ca927cb447eeb24195cff6a1174e0f9a7295214
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Kiril Nesenko <knesenko at redhat.com>
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list