]> sourceware.org Git - lvm2.git/commitdiff
tests: try to force remove higher minor first
authorZdenek Kabelac <zkabelac@redhat.com>
Mon, 4 Jul 2016 13:39:44 +0000 (15:39 +0200)
committerZdenek Kabelac <zkabelac@redhat.com>
Mon, 4 Jul 2016 15:40:24 +0000 (17:40 +0200)
When force removing thin-pool we loose 'real' access to hidden device,
and if such pool is in suspended state, any thin volume cannot be
dropped. It likely should be also checked by dmsetup, but meanwhile
apply simple logic -  try to force remove first all higher minors first
with assumption we first create thin-pool and then thin volume
and there are usually not being released lower dm numbers to
get the order wrong.

test/lib/aux.sh

index 8653149444d76f908d396ff9ffad21ea75feb9b0..66574df32e0685ba7f19e486b3e0c2b77f1a85c3 100644 (file)
@@ -419,7 +419,8 @@ teardown_devs_prefixed() {
                while num_devs=$(dm_table | grep "$prefix" | wc -l) && \
                    test $num_devs -lt $num_remaining_devs -a $num_devs -ne 0; do
                        test "$stray" -eq 0 || echo "Removing $num_devs stray mapped devices with names beginning with $prefix: "
-                       for dm in $(dm_info name --sort open | grep "$prefix") ; do
+                        # HACK: sort also by minors - so we try to close 'possibly later' created device first
+                       for dm in $(dm_info name --sort open,-minor | grep "$prefix") ; do
                                dmsetup remove -f "$dm" || true
                        done
                        num_remaining_devs=$num_devs
This page took 0.043388 seconds and 5 git commands to generate.