[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Libguestfs] [PATCH] v2v: ova: Make OVA directory public readable to work around libvirt bug (RHBZ#1375157).



On Mon, Sep 12, 2016 at 01:51:42PM +0200, Pino Toscano wrote:
> On Monday, 12 September 2016 11:53:26 CEST Richard W.M. Jones wrote:
> > Only do this if running as root and if the backend is set
> > to libvirt.
> > ---
> >  v2v/input_ova.ml | 13 +++++++++++++
> >  1 file changed, 13 insertions(+)
> > 
> > diff --git a/v2v/input_ova.ml b/v2v/input_ova.ml
> > index 4f848e2..bb9cb1b 100644
> > --- a/v2v/input_ova.ml
> > +++ b/v2v/input_ova.ml
> > @@ -104,6 +104,19 @@ object
> >      (* Exploded path must be absolute (RHBZ#1155121). *)
> >      let exploded = absolute_path exploded in
> >  
> > +    (* If virt-v2v is running as root, and the backend is libvirt, then
> > +     * we have to chmod the directory to 0755 and files to 0644
> > +     * so it is readable by qemu.qemu.  This is libvirt bug RHBZ#890291.
> > +     *)
> > +    if Unix.geteuid () = 0 then (
> > +      let libguestfs_backend = (open_guestfs ())#get_backend () in
> > +      if libguestfs_backend = "libvirt" then (
> > +        warning (f_"making OVA directory public readable to workaround libvirt bug https://bugzilla.redhat.com/890291";);
> > +        let cmd = [ "chmod"; "-R"; "go=u,go-w"; exploded ] in
> > +        ignore (run_command cmd)
> > +      )
> > +    );
> > +
> 
> The problem is that if the ova is specified as directory, this will
> change its permissions, which is not expected.

Yeah I thought about that too, hence the visible warning.

The problem is that this failure also occurs if we are passed an
exploded OVA which has the wrong permissions.  It's not just something
which happens with the tmpdir that we create.

Really the answer is for libvirt to just fix this ...

Rich.

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
virt-top is 'top' for virtual machines.  Tiny program with many
powerful monitoring features, net stats, disk stats, logging, etc.
http://people.redhat.com/~rjones/virt-top


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]