]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
qa: fix cephfs-mirror unwinding and 'fs volume create/rm' order 52600/head
authorJos Collin <jcollin@redhat.com>
Mon, 24 Jul 2023 08:46:52 +0000 (14:16 +0530)
committerJos Collin <jcollin@redhat.com>
Wed, 26 Jul 2023 04:30:41 +0000 (10:00 +0530)
* Fixes the 'fs volume create' happens before the cephfs-mirror daemon start.
* Fixes the 'fs volume rm' happen only after the cephfs-mirror daemon unwinding.
  - This prevents the issue of mirror-daemon not returning from a libcephfs call, as
    the volumes were deleted during the cephfs_mirror_thrash ing.

Fixes: https://tracker.ceph.com/issues/61182
Signed-off-by: Jos Collin <jcollin@redhat.com>
qa/suites/fs/mirror-ha/cephfs-mirror/+ [new file with mode: 0644]
qa/suites/fs/mirror-ha/cephfs-mirror/1-volume-create-rm.yaml [new file with mode: 0644]
qa/suites/fs/mirror-ha/cephfs-mirror/2-three-per-cluster.yaml [new file with mode: 0644]
qa/suites/fs/mirror-ha/cephfs-mirror/three-per-cluster.yaml [deleted file]
qa/suites/fs/mirror-ha/workloads/cephfs-mirror-ha-workunit.yaml

diff --git a/qa/suites/fs/mirror-ha/cephfs-mirror/+ b/qa/suites/fs/mirror-ha/cephfs-mirror/+
new file mode 100644 (file)
index 0000000..e69de29
diff --git a/qa/suites/fs/mirror-ha/cephfs-mirror/1-volume-create-rm.yaml b/qa/suites/fs/mirror-ha/cephfs-mirror/1-volume-create-rm.yaml
new file mode 100644 (file)
index 0000000..4ee16e1
--- /dev/null
@@ -0,0 +1,14 @@
+meta:
+- desc: create/rm volumes and set configs
+
+tasks:
+  - exec:
+      mon.a:
+        - "ceph fs volume create dc"
+        - "ceph fs volume create dc-backup"
+  - full_sequential_finally:
+    - exec:
+        mon.a:
+          - ceph config set mon mon_allow_pool_delete true
+          - ceph fs volume rm dc --yes-i-really-mean-it
+          - ceph fs volume rm dc-backup --yes-i-really-mean-it
diff --git a/qa/suites/fs/mirror-ha/cephfs-mirror/2-three-per-cluster.yaml b/qa/suites/fs/mirror-ha/cephfs-mirror/2-three-per-cluster.yaml
new file mode 100644 (file)
index 0000000..095f089
--- /dev/null
@@ -0,0 +1,12 @@
+meta:
+- desc: run one cephfs-mirror daemon on primary cluster
+tasks:
+- cephfs-mirror:
+    client: client.mirror1
+    run_in_foreground: True
+- cephfs-mirror:
+    client: client.mirror2
+    run_in_foreground: True
+- cephfs-mirror:
+    client: client.mirror3
+    run_in_foreground: True
diff --git a/qa/suites/fs/mirror-ha/cephfs-mirror/three-per-cluster.yaml b/qa/suites/fs/mirror-ha/cephfs-mirror/three-per-cluster.yaml
deleted file mode 100644 (file)
index 095f089..0000000
+++ /dev/null
@@ -1,12 +0,0 @@
-meta:
-- desc: run one cephfs-mirror daemon on primary cluster
-tasks:
-- cephfs-mirror:
-    client: client.mirror1
-    run_in_foreground: True
-- cephfs-mirror:
-    client: client.mirror2
-    run_in_foreground: True
-- cephfs-mirror:
-    client: client.mirror3
-    run_in_foreground: True
index f43a9a4ba7b258769b8ecbeb861c28b8f8405680..ce4dddf78d51e4a68a41d0d71ab8fee045f512f1 100644 (file)
@@ -8,17 +8,6 @@ overrides:
         debug client: 10
 
 tasks:
-  - exec:
-      mon.a:
-        - "ceph fs volume create dc"
-        - "ceph fs volume create dc-backup"
-  # Remove volumes during unwind to avoid MDS replacement warnings:
-  - full_sequential_finally:
-    - exec:
-        mon.a:
-          - ceph config set mon mon_allow_pool_delete true
-          - ceph fs volume rm dc --yes-i-really-mean-it
-          - ceph fs volume rm dc-backup --yes-i-really-mean-it
   - ceph-fuse:
       client.1:
         cephfs_name: dc