[PATCH] qemu-shim: add a --with-config option

[Daniel P. Berrangé]

On Sat, Mar 28, 2020 at 12:38:57AM +0100, marcandre.lureau at redhat.com wrote:
> From: Marc-André Lureau <marcandre.lureau at redhat.com>
> 
> Add an option to ease setting up a VM with existing libvirt qemu.conf.
> 
> Fixes:
> https://bugzilla.redhat.com/show_bug.cgi?id=1817776
> 
> Signed-off-by: Marc-André Lureau <marcandre.lureau at redhat.com>
> ---
>  docs/manpages/virt-qemu-run.rst |  6 ++++++
>  src/libvirt_private.syms        |  1 -
>  src/qemu/qemu_shim.c            | 36 +++++++++++++++++++++++++++++++++
>  3 files changed, 42 insertions(+), 1 deletion(-)
> 
> diff --git a/docs/manpages/virt-qemu-run.rst b/docs/manpages/virt-qemu-run.rst
> index b06c311b1d..3196b4c8f4 100644
> --- a/docs/manpages/virt-qemu-run.rst
> +++ b/docs/manpages/virt-qemu-run.rst
> @@ -71,6 +71,12 @@ is a path to the XML description of the secret, whose UUID should
>  match a secret referenced in the guest domain XML. The ``VALUE-FILE``
>  is a path containing the raw value of the secret.
>  
> +``-c``, ``--with-config``
> +
> +Copy the libvirt qemu.conf configuration to the root directory.  If
> +there is already a qemu.conf file in the $ROOT/etc directory, exit
> +with failure.

As discussed in the bugzilla, we really don't want to be using the
global qemu.conf file in combination with the embedded driver. We
are not intending to guarantee that the embedded driver will share
the same defaults as the global drivers.

In general we want to eliminate the need to touch qemu.conf at all.

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|