]> git.ipfire.org Git - thirdparty/libvirt.git/commit
disk: Fixup error handling path for devmapper when part_separator='yes'
authorJohn Ferlan <jferlan@redhat.com>
Tue, 15 Nov 2016 19:37:01 +0000 (14:37 -0500)
committerJohn Ferlan <jferlan@redhat.com>
Wed, 18 Jan 2017 11:17:36 +0000 (06:17 -0500)
commit0d157b3fed835e20385150af549cfdc107ff4304
treedf67ffa644c43e6e40d2a8d5cd0f5ebce9075819
parent9508682ba090014f0fdf9399ca256b2fa5a606ec
disk: Fixup error handling path for devmapper when part_separator='yes'

https://bugzilla.redhat.com/show_bug.cgi?id=1346566

If libvirt_parthelper is erroneously told to append the partition
separator 'p' onto the generated output for a disk pool using device
mapper that has 'user_friendly_names' set to true, then the error
recovery path will fail to find volume resulting in the pool being
in an unusable state.

So, augment the documentation to provide the better hint that the
part_separator='yes' should be set when user_friendly_names are not
being used. Additionally, once we're in the error path where the
returned name doesn't match the expected partition name try to see
if the reason is because the 'p' was erroneosly added. If so alter
the about to be removed vol->target.path so that the DiskDeleteVol
code can find the partition that was created and remove it.
docs/formatstorage.html.in
src/storage/storage_backend_disk.c