From: Loic Dachary <ldachary@redhat.com>
Date: Sun, 20 Sep 2015 21:42:45 +0000 (+0200)
Subject: tests: update to match crushmap validation message
X-Git-Tag: v0.94.4~15^2~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=abc5b5f139b05da0be0fbaa99282be16386980db;p=ceph.git

tests: update to match crushmap validation message

http://tracker.ceph.com/issues/13182 Fixes: #13182

Signed-off-by: Loic Dachary <ldachary@redhat.com>
(cherry picked from commit 21a1e75d8a7bad89a48cd9d36902c5d609be5015)

Conflicts:
	qa/workunits/cephtool/test.sh
	  the test structure in hammer is different, adapt
	  the relevant test.
---

diff --git a/qa/workunits/cephtool/test.sh b/qa/workunits/cephtool/test.sh
index b1bfeb3996ea..3c39a2412824 100755
--- a/qa/workunits/cephtool/test.sh
+++ b/qa/workunits/cephtool/test.sh
@@ -1483,6 +1483,7 @@ function test_mon_crushmap_validation()
 {
   local map=$TMPDIR/map
   ceph osd getcrushmap -o $map
+
   # crushtool validation timesout and is ignored
   cat > $TMPDIR/crushtool <<EOF
 #!/bin/sh
@@ -1492,7 +1493,19 @@ EOF
   chmod +x $TMPDIR/crushtool
   ceph tell mon.* injectargs --crushtool $TMPDIR/crushtool
   ceph osd setcrushmap -i $map 2>&1 | grep 'took too long'
+
+  # crushtool validation fails and is ignored
+  cat > $TMPDIR/crushtool <<EOF
+#!/bin/sh
+echo 'TEST FAIL' >&2
+exit 1 # failure
+EOF
+  chmod +x $TMPDIR/crushtool
+  ceph tell mon.* injectargs --crushtool $TMPDIR/crushtool
+  ceph osd setcrushmap -i $map 2>&1 | grep 'Failed crushmap test'
+
   ceph tell mon.* injectargs --crushtool crushtool
+
   # crushtool validation succeeds
   ceph osd setcrushmap -i $map
 }