[libvirt] [PATCH for 5.0.0] qemu: Temporary disable owner remembering

[Peter Krempa]

On Mon, Jan 14, 2019 at 17:57:39 +0100, Michal Privoznik wrote:
> Turns out, that there are few bugs that are not that trivial to
> fix (e.g. around block jobs). Instead of rushing in not
> thoroughly tested fixes disable the feature temporarily for the
> release.
> 
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> ---
>  docs/news.xml                      | 13 -------------
>  src/qemu/libvirtd_qemu.aug         |  1 -
>  src/qemu/qemu.conf                 |  5 -----
>  src/qemu/qemu_conf.c               |  4 ----
>  src/qemu/test_libvirtd_qemu.aug.in |  1 -
>  5 files changed, 24 deletions(-)
> 
> diff --git a/docs/news.xml b/docs/news.xml
> index 90b7e8891e..c7a4dde463 100644
> --- a/docs/news.xml
> +++ b/docs/news.xml
> @@ -66,19 +66,6 @@
>            qemu: Add support for ARMv6l guests
>          </summary>
>        </change>
> -      <change>
> -        <summary>
> -          Remember original owners and SELinux labels of files
> -        </summary>
> -        <description>
> -          When a domain is starting up libvirt changes DAC and
> -          SELinux labels so that domain can access it. However,
> -          it never remembered the original labels and therefore
> -          the file was returned back to <code>root:root</code>.
> -          With this release, the original labels are remembered
> -          and restored properly.
> -        </description>
> -      </change>
>        <change>
>          <summary>
>            Support more NVDIMM configuration options
> diff --git a/src/qemu/libvirtd_qemu.aug b/src/qemu/libvirtd_qemu.aug
> index 8a5b39e568..ddc4bbfd1d 100644
> --- a/src/qemu/libvirtd_qemu.aug
> +++ b/src/qemu/libvirtd_qemu.aug
> @@ -71,7 +71,6 @@ module Libvirtd_qemu =
>                   | str_entry "user"
>                   | str_entry "group"
>                   | bool_entry "dynamic_ownership"
> -                 | bool_entry "remember_owner"
>                   | str_array_entry "cgroup_controllers"
>                   | str_array_entry "cgroup_device_acl"
>                   | int_entry "seccomp_sandbox"
> diff --git a/src/qemu/qemu.conf b/src/qemu/qemu.conf
> index 29093f6329..28e51b2c59 100644
> --- a/src/qemu/qemu.conf
> +++ b/src/qemu/qemu.conf
> @@ -450,11 +450,6 @@
>  # Set to 0 to disable file ownership changes.
>  #dynamic_ownership = 1
>  
> -# Whether libvirt should remember and restore the original
> -# ownership over files it is relabeling. Defaults to 1, set
> -# to 0 to disable the feature.
> -#remember_owner = 1
> -
>  # What cgroup controllers to make use of with QEMU guests
>  #
>  #  - 'cpu' - use for scheduler tunables
> diff --git a/src/qemu/qemu_conf.c b/src/qemu/qemu_conf.c
> index 20952e9607..b03e38b831 100644
> --- a/src/qemu/qemu_conf.c
> +++ b/src/qemu/qemu_conf.c
> @@ -145,7 +145,6 @@ virQEMUDriverConfigPtr virQEMUDriverConfigNew(bool privileged)
>          cfg->group = (gid_t)-1;
>      }
>      cfg->dynamicOwnership = privileged;
> -    cfg->rememberOwner = privileged;

Can't we just set this to false and add a note to the qemu.conf file
that it's currently disabled as being unstable so that you don't have to
delete the qemu_conf.c changes?

ACK for that option.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20190114/e4447b92/attachment-0001.sig>