Change in vdsm[master]: kvm group can now r/w to all newly created files in /var/log...

amuller at redhat.com amuller at redhat.com
Fri Dec 13 08:37:06 UTC 2013


Assaf Muller has posted comments on this change.

Change subject: kvm group can now r/w to all newly created files in /var/log/vdsm
......................................................................


Patch Set 2: -Verified

My subconscious came up with the following solution during a night's sleep:

Remove the upgrade logger/handler from the VDSM logger conf file. During the start of an upgrade, configure logging using basicConfig (And not from a file), then use chown to make sure that upgrade.log is owned by vdsm:kvm. This change will make it so that upgrade.log is only created in one place, and that no matter if vdsm-tool was run by root or VDSM itself, upgrade.log will end up owned by vdsm:kvm.

I'll code it up Sunday.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie2fb887fa6d43f2b35f4a07927026baa746e7a41
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Assaf Muller <amuller at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Assaf Muller <amuller at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list