[Libguestfs] [PATCH 0/5] Improve inspection of /usr filesystems
Pino Toscano
ptoscano at redhat.com
Wed Dec 7 09:49:44 UTC 2016
On Wednesday, 7 December 2016 08:29:54 CET Richard W.M. Jones wrote:
> On Tue, Dec 06, 2016 at 04:29:17PM +0100, Pino Toscano wrote:
> > Hi,
> >
> > this patch series improves the way /usr filesystems are handled: tag
> > them appropriately, so later on we can find them and merge results they
> > contain directly back for the root filesystem.
> >
> > The series includes also a new private debug API, and its usage to fix
> > the resolution of /dev/mapper/.. devices found in fstab; without it,
> > LVM /usr filesystems are not recognized as belonging to their roots.
> > Maybe a better API for this could be added, but since it's something
> > only related to the appliance then can stay internal for now. (Better
> > suggestions always welcome, of course.)
>
> Firstly, I still think we need:
>
> [PATCH 1/2] inspect: fstab: Canonicalize paths appearing in fstab.
>
> although it's not related to the current patch series.
This was not an issue in the data of RHBZ#140147, do you have an example
of guest with such paths in fstab? So far I never seen one myself.
> In regards to this series:
>
> > Pino Toscano (5):
> > inspect: change is_root flag into enum
> > inspect: mark CoreOS /usr partitions with own USR role
>
> These are fine.
Pushed, thanks.
> > daemon: debug: new "exists" subcommand
> > inspect: fix existance check of /dev/mapper devices
>
> This is very ugly. We really shouldn't be calling a debug API from
> inspection code.
As mentioned above, any better suggestion is welcome. I didn't want to
add a new public API for inspecting files in the appliance itself --
what about something like internal_exists?
Thanks,
--
Pino Toscano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <http://listman.redhat.com/archives/libguestfs/attachments/20161207/d0a4eb12/attachment.sig>
More information about the Libguestfs
mailing list