<div dir="ltr">Greetings from an ancient thread (but the most recent on to openstack-dev about localfs that I can find).<div><br></div><div>In 2014, which were heady times, we decided that we couldn't deprecate nova.virt.disk.vfs.localfs because we theorised that FreeBSD and containers would soon need it. Oh, and xenapi uses it a little too.</div><div><br></div><div>It is now three years later, and there has been some recent thought given to deprecating localfs again, which I bumped into when I chose this code as the first candidate for a prototype privsep user inside the nova codebase [1].</div><div><br></div><div>I think we therefore need to have this conversation again. Do we still have any clear use cases for localfs? Where does guestfs not run at the moment? Or did I waste my time privsep'ing localfs?</div><div><br></div><div>Thanks,</div><div>Michael</div><div><br></div><div>1: <a href="https://review.openstack.org/#/c/459166/">https://review.openstack.org/#/c/459166/</a></div><div class="gmail_extra"><br></div><div class="gmail_extra"><br></div><div class="gmail_extra"><br></div><div class="gmail_extra"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 24, 2014 at 7:04 PM, Daniel P. Berrange <span dir="ltr"><<a href="mailto:berrange@redhat.com" target="_blank">berrange@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail-HOEnZb"><div class="gmail-h5">On Wed, Sep 24, 2014 at 08:26:44AM +1000, Michael Still wrote:<br>
> On Tue, Sep 23, 2014 at 8:58 PM, Daniel P. Berrange <<a href="mailto:berrange@redhat.com">berrange@redhat.com</a>> wrote:<br>
> > On Tue, Sep 23, 2014 at 02:27:52PM +0400, Roman Bogorodskiy wrote:<br>
> >>   Michael Still wrote:<br>
> >><br>
> >> > Hi.<br>
> >> ><br>
> >> > I know we've been talking about deprecating nova.virt.disk.vfs.localfs<br>
> >> > for a long time, in favour of wanting people to use libguestfs<br>
> >> > instead. However, I can't immediately find any written documentation<br>
> >> > for when we said we'd do that thing.<br>
> >> ><br>
> >> > Additionally, this came to my attention because Ubuntu 14.04 is<br>
> >> > apparently shipping a libguestfs old enough to cause us to emit the<br>
> >> > "falling back to localfs" warning, so I think we need Ubuntu to catch<br>
> >> > up before we can do this thing.<br>
> >> ><br>
> >> > So -- how about we remove localfs early in Kilo to give Canonical a<br>
> >> > release to update libguestfs?<br>
> >> ><br>
> >> > Thoughts appreciated,<br>
> >> > Michael<br>
> >><br>
> >> If at some point we'd start moving into getting FreeBSD supported as a<br>
> >> host OS for OpenStack, then it would make sense to keep localfs for that<br>
> >> configuration.<br>
> >><br>
> >> libguestfs doesn't work on FreeBSD yet. On the other hand, localfs<br>
> >> code in Nova doesn't look like it'd be hard to port.<br>
> ><br>
> > Yep, that's a good point and in fact applies to Linux too when considering<br>
> > the non-KVM/QEMU drivers libvirt supports. eg if your host does not have<br>
> > virtualization and you're using LXC for container virt, then we need to<br>
> > have localfs still be present. Likewise if running Xen.<br>
> ><br>
> > So we definitely cannot delete or even deprecate it unconditionally. We<br>
> > simply want to make sure localfs isn't used when Nova is configured to<br>
> > run QEMU/KVM via libvirt.<br>
> ><br>
> > So if we take the config option approach I suggested, then we'd set a<br>
> > default value for the vfs_impl parameter according to which libvirt<br>
> > driver you have enabled.<br>
><br>
> I'm glad we've had this thread, because I hadn't thought of the<br>
> FreeBSD case at all. In that case I wonder if we want to water down<br>
> the warning we currently log in this case:<br>
><br>
>                 LOG.warn(_LW("Unable to import guestfs"<br>
>                              "falling back to VFSLocalFS"))<br>
><br>
> If feel like it should be an info if we know some platforms will<br>
> always have this occur. I know this is a minor thing, but this came to<br>
> my attention because at lease one operator was concerned by seeing<br>
> that warning in their logs.<br>
<br>
</div></div>If we take my suggested approach of using a fixed impl based on libvirt<br>
driver type, then we wouldn't have fallback & so not see this warning.<br>
Even when we do have fallback, we should only warn if libguestfs is<br>
installed, but not working.<br>
<div class="gmail-HOEnZb"><div class="gmail-h5"><br>
<br>
Regards,<br>
Daniel<br>
--<br>
|: <a href="http://berrange.com" rel="noreferrer" target="_blank">http://berrange.com</a>      -o-    <a href="http://www.flickr.com/photos/dberrange/" rel="noreferrer" target="_blank">http://www.flickr.com/photos/<wbr>dberrange/</a> :|<br>
|: <a href="http://libvirt.org" rel="noreferrer" target="_blank">http://libvirt.org</a>              -o-             <a href="http://virt-manager.org" rel="noreferrer" target="_blank">http://virt-manager.org</a> :|<br>
|: <a href="http://autobuild.org" rel="noreferrer" target="_blank">http://autobuild.org</a>       -o-         <a href="http://search.cpan.org/~danberr/" rel="noreferrer" target="_blank">http://search.cpan.org/~<wbr>danberr/</a> :|<br>
|: <a href="http://entangle-photo.org" rel="noreferrer" target="_blank">http://entangle-photo.org</a>       -o-       <a href="http://live.gnome.org/gtk-vnc" rel="noreferrer" target="_blank">http://live.gnome.org/gtk-vnc</a> :|<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Rackspace Australia</div>
</div></div>