[libvirt] [PATCH] virt-host-validate: require fuse for LXC if compiled in

Guido Günther posted 1 patch 6 years, 5 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20171012155321.d3mb42zzigz343ke@bogon.m.sigxcpu.org
tools/virt-host-validate-lxc.c | 7 +++++++
1 file changed, 7 insertions(+)
[libvirt] [PATCH] virt-host-validate: require fuse for LXC if compiled in
Posted by Guido Günther 6 years, 5 months ago
Domains fail to start without fuse like

  error: internal error: guest failed to start: fuse: device not found, try 'modprobe fuse' first
  Failure in libvirt_lxc startup: no error

so check for it too.

References: https://ci.debian.net/data/autopkgtest/unstable/amd64/libv/libvirt/20171012_105903/log.gz
---
 tools/virt-host-validate-lxc.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/tools/virt-host-validate-lxc.c b/tools/virt-host-validate-lxc.c
index 2b906cc88a..64d9279c30 100644
--- a/tools/virt-host-validate-lxc.c
+++ b/tools/virt-host-validate-lxc.c
@@ -93,5 +93,12 @@ int virHostValidateLXC(void)
                                         "BLK_CGROUP") < 0)
         ret = -1;
 
+#if WITH_FUSE
+    if (virHostValidateDeviceExists("LXC", "/sys/fs/fuse/connections",
+                                    VIR_HOST_VALIDATE_FAIL,
+                                    _("Load the 'fuse' module to enable /proc/ overrides")) < 0)
+        ret = -1;
+#endif
+
     return ret;
 }
-- 
2.14.2

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] virt-host-validate: require fuse for LXC if compiled in
Posted by Daniel P. Berrange 6 years, 5 months ago
On Thu, Oct 12, 2017 at 05:53:21PM +0200, Guido Günther wrote:
> Domains fail to start without fuse like
> 
>   error: internal error: guest failed to start: fuse: device not found, try 'modprobe fuse' first
>   Failure in libvirt_lxc startup: no error
> 
> so check for it too.
> 
> References: https://ci.debian.net/data/autopkgtest/unstable/amd64/libv/libvirt/20171012_105903/log.gz
> ---
>  tools/virt-host-validate-lxc.c | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/tools/virt-host-validate-lxc.c b/tools/virt-host-validate-lxc.c
> index 2b906cc88a..64d9279c30 100644
> --- a/tools/virt-host-validate-lxc.c
> +++ b/tools/virt-host-validate-lxc.c
> @@ -93,5 +93,12 @@ int virHostValidateLXC(void)
>                                          "BLK_CGROUP") < 0)
>          ret = -1;
>  
> +#if WITH_FUSE
> +    if (virHostValidateDeviceExists("LXC", "/sys/fs/fuse/connections",
> +                                    VIR_HOST_VALIDATE_FAIL,
> +                                    _("Load the 'fuse' module to enable /proc/ overrides")) < 0)
> +        ret = -1;
> +#endif
> +
>      return ret;
>  }

Reviewed-by: Daniel P. Berrange <berrange@redhat.com>


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 :|

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] virt-host-validate: require fuse for LXC if compiled in
Posted by Guido Günther 6 years, 5 months ago
Hi,
On Thu, Oct 12, 2017 at 05:25:34PM +0100, Daniel P. Berrange wrote:
> On Thu, Oct 12, 2017 at 05:53:21PM +0200, Guido Günther wrote:
> > Domains fail to start without fuse like
> > 
> >   error: internal error: guest failed to start: fuse: device not found, try 'modprobe fuse' first
> >   Failure in libvirt_lxc startup: no error
> > 
> > so check for it too.
> > 
> > References: https://ci.debian.net/data/autopkgtest/unstable/amd64/libv/libvirt/20171012_105903/log.gz
> > ---
> >  tools/virt-host-validate-lxc.c | 7 +++++++
> >  1 file changed, 7 insertions(+)
> > 
> > diff --git a/tools/virt-host-validate-lxc.c b/tools/virt-host-validate-lxc.c
> > index 2b906cc88a..64d9279c30 100644
> > --- a/tools/virt-host-validate-lxc.c
> > +++ b/tools/virt-host-validate-lxc.c
> > @@ -93,5 +93,12 @@ int virHostValidateLXC(void)
> >                                          "BLK_CGROUP") < 0)
> >          ret = -1;
> >  
> > +#if WITH_FUSE
> > +    if (virHostValidateDeviceExists("LXC", "/sys/fs/fuse/connections",
> > +                                    VIR_HOST_VALIDATE_FAIL,
> > +                                    _("Load the 'fuse' module to enable /proc/ overrides")) < 0)
> > +        ret = -1;
> > +#endif
> > +
> >      return ret;
> >  }
> 
> Reviewed-by: Daniel P. Berrange <berrange@redhat.com>

Pushed. thanks!
 -- Guido

> 
> 
> 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 :|
> 

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] virt-host-validate: require fuse for LXC if compiled in
Posted by Cedric Bosdonnat 6 years, 5 months ago

>>> Guido Günther <agx@sigxcpu.org> 10/12/17 6:12 PM >>>
Domains fail to start without fuse like

  error: internal error: guest failed to start: fuse: device not found, try 'modprobe fuse' first
  Failure in libvirt_lxc startup: no error

so check for it too.

References: https://ci.debian.net/data/autopkgtest/unstable/amd64/libv/libvirt/20171012_105903/log.gz
---
 tools/virt-host-validate-lxc.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/tools/virt-host-validate-lxc.c b/tools/virt-host-validate-lxc.c
index 2b906cc88a..64d9279c30 100644
--- a/tools/virt-host-validate-lxc.c
+++ b/tools/virt-host-validate-lxc.c
@@ -93,5 +93,12 @@ int virHostValidateLXC(void)
                                         "BLK_CGROUP") < 0)
         ret = -1;
 
+#if WITH_FUSE
+    if (virHostValidateDeviceExists("LXC", "/sys/fs/fuse/connections",
+                                    VIR_HOST_VALIDATE_FAIL,
+                                    _("Load the 'fuse' module to enable /proc/ overrides")) < 0)
+        ret = -1;
+#endif
+
     return ret;
 }

ACK

--
Cedric

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list