Re: [libvirt] [PATCH] sanlock: Chown lease files as well

2012-12-31 Thread Eric Blake
On 12/28/2012 08:59 AM, Michal Privoznik wrote: > Since sanlock doesn't run under root:root, we have chown()'ed the > __LIBVIRT__DISKS__ lease file to the user:group defined in the > sanlock config. However, when writing the patch I've forgot about > lease files for each disk (this is the > /var/li

Re: [libvirt] [PATCH] sanlock: Chown lease files as well

2012-12-30 Thread Michal Privoznik
On 30.12.2012 09:46, Osier Yang wrote: > On 2012年12月28日 23:59, Michal Privoznik wrote: >> Since sanlock doesn't run under root:root, we have chown()'ed the >> __LIBVIRT__DISKS__ lease file to the user:group defined in the >> sanlock config. However, when writing the patch I've forgot about >> lease

Re: [libvirt] [PATCH] sanlock: Chown lease files as well

2012-12-30 Thread Osier Yang
On 2012年12月28日 23:59, Michal Privoznik wrote: Since sanlock doesn't run under root:root, we have chown()'ed the __LIBVIRT__DISKS__ lease file to the user:group defined in the sanlock config. However, when writing the patch I've forgot about lease files for each disk (this is the /var/lib/libvirt/

[libvirt] [PATCH] sanlock: Chown lease files as well

2012-12-28 Thread Michal Privoznik
Since sanlock doesn't run under root:root, we have chown()'ed the __LIBVIRT__DISKS__ lease file to the user:group defined in the sanlock config. However, when writing the patch I've forgot about lease files for each disk (this is the /var/lib/libvirt/sanlock/) file. --- src/locking/lock_driver_san