Change in vdsm[master]: BZ#731750 - handle missing libvirt lograte config

Saggi Mizrahi smizrahi at redhat.com
Mon Aug 29 09:31:53 UTC 2011


Saggi Mizrahi has submitted this change and it was merged.

Change subject: BZ#731750 - handle missing libvirt lograte config
......................................................................


BZ#731750 - handle missing libvirt lograte config

* I also added restorecon in case there is a configuration where
  logrotate conf file should have a special context.

VDSM failed to start if it couldn't find the original libvirt logrotate
configurations. Because we just replace the configuration there is no
need to actually make sure the old one existed. If it didn't we can just
create a new config with default permissions.

Change-Id: If909667a96a75df9070baca4054406a465cb11e0
---
M vdsm/vdsmd.in
1 file changed, 4 insertions(+), 1 deletion(-)

Approvals:
  Igor Lvovsky: Looks good to me, but someone else must approve
  David Naori: Verified
  Ayal Baron: Looks good to me, but someone else must approve
  Saggi Mizrahi: Looks good to me, approved


--
To view, visit http://gerrit.usersys.redhat.com/856
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If909667a96a75df9070baca4054406a465cb11e0
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Ayal Baron
Gerrit-Reviewer: David Naori <dnaori at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list