Fix: 002-test-reject-disk-with-lvm-signature.sh

Signed-off-by: Shishir Mahajan <[email protected]>

Author: Shishir Mahajan

tests/002-test-reject-disk-with-lvm-signature.sh

@@ -7,6 +7,8 @@ test_lvm_sig() {
   local test_status=1
   local testname=`basename "$0"`
   local vg_name="css-test-foo"
+  local tmplog=${WORKDIR}/tmplog
+  local errmsg="Wipe signatures using wipefs or use WIPE_SIGNATURES=true and retry."
 
   # Error out if any pre-existing volume group vg named css-test-foo
   if vg_exists "$vg_name"; then
@@ -25,17 +27,33 @@ EOF
   done
 
   # Run container-storage-setup
-  $CSSBIN >> $LOGS 2>&1
-
-  # Css should fail. If it did not, then test failed. This is very crude
-  # check though as css can fail for so many reasons. A more precise check
-  # would be too check for exact error message.
-  [ $? -ne 0 ] && test_status=0
+  $CSSBIN > $tmplog 2>&1
+  rc=$?
+  cat $tmplog >> $LOGS 2>&1
+
+  # Test failed.
+  if [ $rc -ne 0 ]; then
+      if grep --no-messages -q "$errmsg" $tmplog; then
+          test_status=0
+      else
+          echo "ERROR: $testname: $CSSBIN Failed for a reason other then \"$errmsg\"" >> $LOGS
+      fi
+  else
+      echo "ERROR: $testname: $CSSBIN Succeeded. Should have failed since LVM2_member signature exists on devices $devs" >> $LOGS
+  fi
 
-  cleanup $vg_name "$devs"
+  cleanup_pvs_sig $devs
   return $test_status
 }
 
+cleanup_pvs_sig() {
+  local devs=$1
+  for dev in $devs; do
+    pvremove -y ${dev} >> $LOGS 2>&1
+  done
+  wipe_signatures "$devs"
+}
+
 # Make sure a disk with lvm signature is rejected and is not overriden
 # by css. Returns 0 on success and 1 on failure.
 

tests/libtest.sh

@@ -72,11 +72,13 @@ lv_is_active() {
   return 1
 }
 
-remove_pvs() {
+remove_pvs_part() {
   local dev devs=$1 pv
   for dev in $devs; do
     pv=$(lsblk -npl -o NAME "$dev" | tail -n +2 | head -1)
-    pvremove -y ${pv} >> $LOGS 2>&1
+    if [ -n "$pv" ];then
+      pvremove -y ${pv} >> $LOGS 2>&1
+    fi
   done
 }
 
@@ -127,7 +129,7 @@ cleanup() {
 
 
   vgremove -y $vg_name >> $LOGS 2>&1
-  remove_pvs "$devs"
+  remove_pvs_part "$devs"
   remove_partitions "$devs"
   # After removing partitions let udev settle down. In some
   # cases it has been observed that udev rule kept the device