Fix missing space in vitastor-cli rm-osd "still up" message

rm-osd printed "OSD 1is still up" and "OSDs 2, 3are still up" because
the verb was appended to the OSD list without a space. Also say "it"
instead of "them" when only one OSD is up.

src/cmd/cli_pg_ls.cpp

@@ -251,7 +251,7 @@ resume_1:
             { "title", "MISPLACED" },
         });
         cols.push_back(json11::Json::object{
-            { "key", "fmt_degraded" },
+            { "key", "fmt_misplaced" },
             { "title", "DEGRADED" },
         });
         cols.push_back(json11::Json::object{
@@ -273,8 +273,6 @@ resume_1:
         load_pg_stats();
         if (parent->waiting > 0)
             return;
-        if (state == 100)
-            return;
         format_pgs();
         state = 100;
     }

src/cmd/cli_rm_osd.cpp

@@ -170,7 +170,8 @@ struct rm_osd_t
             }
             if (still_up.size() && !allow_up)
                 error += (still_up.size() == 1 ? "OSD " : "OSDs ") + implode(", ", still_up) +
-                    (still_up.size() == 1 ? "is" : "are") + " still up. Use `vitastor-disk purge` to delete them.\n";
+                    (still_up.size() == 1 ? " is" : " are") + " still up. Use `vitastor-disk purge` to delete " +
+                    (still_up.size() == 1 ? "it" : "them") + ".\n";
             if (is_dataloss && !force_dataloss && !dry_run)
                 error += "OSDs not deleted. Please move data to other OSDs or bypass this check with --allow-data-loss if you know what you are doing.\n";
             else if (is_warning && !force_warning && !dry_run)

tests/run_tests.sh

@@ -61,12 +61,13 @@ OLD=1 SCHEME=ec ./test_snapshot.sh
 OLD=1 ./test_move_reappear.sh
 
 ./test_degraded.sh
-./test_pg_ls.sh
 
 ./test_rm.sh
 
 ./test_rm_degraded.sh
 
+./test_rm_osd_msg.sh
+
 ./test_snapshot_chain.sh
 SCHEME=ec ./test_snapshot_chain.sh
 ENCRYPTED=1 ./test_snapshot_chain.sh

tests/test_pg_ls.sh

@@ -1,47 +0,0 @@
-#!/bin/bash -ex
-
-# Test vitastor-cli ls-pgs on degraded PGs
-
-PG_COUNT=4
-PG_SIZE=3
-PG_MINSIZE=2
-# Keep degraded objects until the checks are done
-GLOBAL_CONFIG=',"no_recovery":true'
-. `dirname $0`/run_3osds.sh
-
-$VITASTOR_CLI create -s 32M testimg
-
-# Data written while OSD 3 is down is degraded when it comes back
-kill -9 $OSD3_PID
-$ETCDCTL del /vitastor/osd/state/3
-wait_condition 60 "$ETCDCTL get --prefix /vitastor/pg/state/ --print-value-only | jq -s -e '[ .[] | select(.state | index(\"degraded\")) ] | length == $PG_COUNT'" "PG peering without OSD 3"
-$VITASTOR_CLI dd if=/dev/urandom oimg=testimg bs=1M count=32
-start_osd 3
-wait_condition 60 "$ETCDCTL get --prefix /vitastor/pgstats/ --print-value-only | jq -s -e '[ .[] | select(.degraded_count > 0) ] | length == $PG_COUNT'" "PG degradation"
-
-$VITASTOR_CLI ls-pgs
-
-# NUM and DEGRADED of every PG from the table, and degraded_count from JSON
-$VITASTOR_CLI ls-pgs | awk 'NR == 1 { n = index($0, "NUM"); d = index($0, "DEGRADED"); e = index($0, "INCOMPLETE") }
-    NR > 1 { num = substr($0, n); sub(/ .*/, "", num); deg = substr($0, d, e-d); sub(/ +$/, "", deg); print num " " deg }' > ./testdata/pg_ls.txt
-$VITASTOR_CLI ls-pgs --json | jq -r '.[] | "\(.pg_num) \(.degraded_count // 0)"' > ./testdata/pg_ls_json.txt
-
-if [[ $(wc -l < ./testdata/pg_ls.txt) != $PG_COUNT ]]; then
-    format_error "ls-pgs should list $PG_COUNT PGs"
-fi
-while read num count; do
-    shown=$(awk '$1 == '$num' { $1 = ""; sub(/^ /, ""); print }' ./testdata/pg_ls.txt)
-    if [[ -z "$shown" || $count -gt 0 && "$shown" = "0 B" || $count -eq 0 && "$shown" != "0 B" ]]; then
-        format_error "ls-pgs shows DEGRADED=$shown for PG $num with $count degraded objects"
-    fi
-done < ./testdata/pg_ls_json.txt
-
-# An unknown pool is an error, not an empty table
-if out=$($VITASTOR_CLI ls-pgs --pool nosuchpool 2>&1); then
-    format_error "ls-pgs should fail for an unknown pool"
-fi
-if [[ "$out" != "Pool nosuchpool not found" ]]; then
-    format_error "ls-pgs prints '$out' instead of the unknown pool error"
-fi
-
-format_green OK

tests/test_rm_osd_msg.sh

@@ -0,0 +1,14 @@
+#!/bin/bash -ex
+
+# Test the rm-osd message for OSDs that are still up
+
+. `dirname $0`/run_3osds.sh
+
+$VITASTOR_CLI rm-osd --dry-run 1 > ./testdata/one.txt 2>&1 && format_error "rm-osd of an up OSD was not refused"
+cat ./testdata/one.txt
+grep -q '^OSD 1 is still up\. Use `vitastor-disk purge` to delete it\.$' ./testdata/one.txt || format_error "bad message for one OSD"
+$VITASTOR_CLI rm-osd --dry-run 1 2 > ./testdata/two.txt 2>&1 && format_error "rm-osd of up OSDs was not refused"
+cat ./testdata/two.txt
+grep -q '^OSDs 1, 2 are still up\. Use `vitastor-disk purge` to delete them\.$' ./testdata/two.txt || format_error "bad message for two OSDs"
+
+format_green OK