From 3f7eadb79f4bb0a2fe25b229c24b652e7dcbaf00 Mon Sep 17 00:00:00 2001
From: Sage Weil <sage@redhat.com>
Date: Sat, 4 Nov 2017 09:32:51 -0500
Subject: [PATCH] qa/suites/rados/thrash: combine balancer with
 require-luminous

We can only do the balancer stuff if we've done the mkfs-time
require luminous because otherwise the mgr commands aren't proxied via
the mon.

Only do balancer for thrash tests; not for basic, monthrash, verify.

Signed-off-by: Sage Weil <sage@redhat.com>
---
 qa/suites/rados/basic/d-require-luminous      |  1 -
 .../basic/d-require-luminous/at-end.yaml      | 33 +++++++++++++++++++
 .../d-require-luminous/at-mkfs.yaml}          |  0
 qa/suites/rados/monthrash/d-require-luminous  |  2 +-
 .../at-mkfs-balancer-crush-compat.yaml}       |  0
 .../at-mkfs-balancer-upmap.yaml}              |  0
 qa/suites/rados/verify/d-require-luminous     |  2 +-
 7 files changed, 35 insertions(+), 3 deletions(-)
 delete mode 120000 qa/suites/rados/basic/d-require-luminous
 create mode 100644 qa/suites/rados/basic/d-require-luminous/at-end.yaml
 rename qa/suites/rados/{thrash/d-balancer/off.yaml => basic/d-require-luminous/at-mkfs.yaml} (100%)
 rename qa/suites/rados/thrash/{d-balancer/crush-compat.yaml => d-require-luminous/at-mkfs-balancer-crush-compat.yaml} (100%)
 rename qa/suites/rados/thrash/{d-balancer/upmap.yaml => d-require-luminous/at-mkfs-balancer-upmap.yaml} (100%)

diff --git a/qa/suites/rados/basic/d-require-luminous b/qa/suites/rados/basic/d-require-luminous
deleted file mode 120000
index 737aee82415f3..0000000000000
--- a/qa/suites/rados/basic/d-require-luminous
+++ /dev/null
@@ -1 +0,0 @@
-../thrash/d-require-luminous/
\ No newline at end of file
diff --git a/qa/suites/rados/basic/d-require-luminous/at-end.yaml b/qa/suites/rados/basic/d-require-luminous/at-end.yaml
new file mode 100644
index 0000000000000..ef998cc89143d
--- /dev/null
+++ b/qa/suites/rados/basic/d-require-luminous/at-end.yaml
@@ -0,0 +1,33 @@
+# do not require luminous osds at mkfs time; only set flag at
+# the end of the test run, then do a final scrub (to convert any
+# legacy snapsets), and verify we are healthy.
+tasks:
+- full_sequential_finally:
+  - exec:
+      mon.a:
+        - ceph osd require-osd-release luminous
+        - ceph osd pool application enable base rados || true
+# make sure osds have latest map
+        - rados -p rbd bench 5 write -b 4096
+  - ceph.healthy:
+  - ceph.osd_scrub_pgs:
+      cluster: ceph
+  - exec:
+      mon.a:
+        - sleep 15
+        - ceph osd dump | grep purged_snapdirs
+        - ceph pg dump -f json-pretty
+        - "ceph pg dump sum -f json-pretty | grep num_legacy_snapsets | head -1 | grep ': 0'"
+overrides:
+  ceph:
+    conf:
+      global:
+        mon debug no require luminous: true
+
+# setting luminous triggers peering, which *might* trigger health alerts
+    log-whitelist:
+      - overall HEALTH_
+      - \(PG_AVAILABILITY\)
+      - \(PG_DEGRADED\)
+  thrashosds:
+    chance_thrash_cluster_full: 0
diff --git a/qa/suites/rados/thrash/d-balancer/off.yaml b/qa/suites/rados/basic/d-require-luminous/at-mkfs.yaml
similarity index 100%
rename from qa/suites/rados/thrash/d-balancer/off.yaml
rename to qa/suites/rados/basic/d-require-luminous/at-mkfs.yaml
diff --git a/qa/suites/rados/monthrash/d-require-luminous b/qa/suites/rados/monthrash/d-require-luminous
index 737aee82415f3..82036c67f956f 120000
--- a/qa/suites/rados/monthrash/d-require-luminous
+++ b/qa/suites/rados/monthrash/d-require-luminous
@@ -1 +1 @@
-../thrash/d-require-luminous/
\ No newline at end of file
+../basic/d-require-luminous
\ No newline at end of file
diff --git a/qa/suites/rados/thrash/d-balancer/crush-compat.yaml b/qa/suites/rados/thrash/d-require-luminous/at-mkfs-balancer-crush-compat.yaml
similarity index 100%
rename from qa/suites/rados/thrash/d-balancer/crush-compat.yaml
rename to qa/suites/rados/thrash/d-require-luminous/at-mkfs-balancer-crush-compat.yaml
diff --git a/qa/suites/rados/thrash/d-balancer/upmap.yaml b/qa/suites/rados/thrash/d-require-luminous/at-mkfs-balancer-upmap.yaml
similarity index 100%
rename from qa/suites/rados/thrash/d-balancer/upmap.yaml
rename to qa/suites/rados/thrash/d-require-luminous/at-mkfs-balancer-upmap.yaml
diff --git a/qa/suites/rados/verify/d-require-luminous b/qa/suites/rados/verify/d-require-luminous
index 737aee82415f3..82036c67f956f 120000
--- a/qa/suites/rados/verify/d-require-luminous
+++ b/qa/suites/rados/verify/d-require-luminous
@@ -1 +1 @@
-../thrash/d-require-luminous/
\ No newline at end of file
+../basic/d-require-luminous
\ No newline at end of file
-- 
2.39.5