[libvirt] [PATCH] sanlock: Introduce 'user' and 'group' conf variables
Jiri Denemark
jdenemar at redhat.com
Thu Oct 25 07:40:32 UTC 2012
On Wed, Oct 24, 2012 at 19:05:01 +0200, Michal Privoznik wrote:
> through which user set under what permissions does sanlock
> daemon run so libvirt will set the same permissions for
> files exposed to it.
> ---
> docs/locking.html.in | 22 +++++++++
> src/locking/libvirt_sanlock.aug | 2 +
> src/locking/lock_driver_sanlock.c | 76 ++++++++++++++++++++++++++++++-
> src/locking/sanlock.conf | 11 ++++-
> src/locking/test_libvirt_sanlock.aug.in | 2 +
> 5 files changed, 111 insertions(+), 2 deletions(-)
>
...
> diff --git a/src/locking/sanlock.conf b/src/locking/sanlock.conf
> index efc35ee..40ece65 100644
> --- a/src/locking/sanlock.conf
> +++ b/src/locking/sanlock.conf
> @@ -29,7 +29,8 @@
> #
> # Recommendation is to just mount this default location as
> # an NFS volume. Uncomment this, if you would prefer the mount
> -# point to be somewhere else.
> +# point to be somewhere else. Moreover, please make sure
> +# sanlock daemon can access the specified path.
> #
> #disk_lease_dir = "/var/lib/libvirt/sanlock"
Since this directory is created by libvirt.spec in lock-sanlock subpackage, I
think the specfile should make sure the directory is created in a way that
default installation of sanlock is able to access it, i.e., it should be
accessible for group sanlock.
Jirka
More information about the libvir-list
mailing list