From f7adcb5f6db7e7c1131b5dfb9114235a908292b1 Mon Sep 17 00:00:00 2001
From: Zdenek Kabelac <zkabelac@redhat.com>
Date: Sun, 18 Nov 2018 21:38:56 +0100
Subject: [PATCH] tests: skip portion of test for lvmpolld

lvmpolld ATM is not desingned to preserve interval checking
in the same way the 'lvconvert' tool is doing - so the passed
'-i 40' is not respected and lvmpolld autonomously checks
state of conversion and updates lvm2 metadata and dm tables
when needed.

So skip portion of test that relayed on this and preserve this logic
only for command line invocation and forking of polling process
where the interval of checking is under full control.
---
 test/shell/mirror-names.sh | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/test/shell/mirror-names.sh b/test/shell/mirror-names.sh
index 6d461807a..1019bd3f3 100644
--- a/test/shell/mirror-names.sh
+++ b/test/shell/mirror-names.sh
@@ -112,6 +112,17 @@ check_and_cleanup_lvs_
 lvcreate -aey -l2 --type mirror -m1 -n $lv1 $vg
 # Use large enough polling interval so mirror is keeping mimagetmp
 LVM_TEST_TAG="kill_me_$PREFIX" lvconvert -m+1 -i+40 -b $vg/$lv1
+
+#
+# TODO: lvmpolld is not 'preserving' -i  interval setting from
+# lvconvert initiating command - so there is not much to test
+# if the lvconvert is already finished at this point
+# and lvmpolld cleaned metadata and refreshed DM table
+#
+# It' unclear if this is undocumented feature of bug.
+#
+if test ! -f LOCAL_LVMPOLLD ; then
+
 for i in $(seq 1 10) ; do
 	# check if background process already started
 	# this is recognized by presence of LV1_mimage_2
@@ -139,6 +150,7 @@ convlv=$(lv_convert_lv_ $vg/$lv2)
 lv_devices_ $vg/$lv2 $convlv ${lv2}_mimage_2
 lv_devices_ $vg/$convlv ${lv2}_mimage_0 ${lv2}_mimage_1
 lv_mirror_log_ $vg/$convlv ${lv2}_mlog
+fi  # ! -f LOCAL_LVMPOLLD
 
 #COMM "cleanup"
 check_and_cleanup_lvs_