[libvirt] [PATCH 2/2] fs: Fix probing on no-overwrite of target device

John Ferlan jferlan at redhat.com
Mon Nov 21 22:42:15 UTC 2016


[...]
> 
> [C] might be more user-friendly, but I'm not sure if we can change the
> meaning of the NO_OVERWRITE constant like that. On the bright side,
> if blkid can also identify partition tables, we could unify the probing
> code with the disk backend and stop parsing parted's output (also, catch
> those cases when someone puts the filesystem on the block device instead
> of its partition).

After some experimenting - the blkid_known_fstype would fail for every
one of the disk format types, so it'd be useless to help detect.

Furthermore, blkid_do_probe would fail for every type except when a
pvcreate has been performed on the device path.

Thus it seems we're stuck with parsing parted's output for disk.

And the fs pool usage of overwrite still makes no sense when compared to
disk (I think we agree that logical's lack of usage/support for
--overwrite is incorrect too).

John





More information about the libvir-list mailing list