]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
qa/suites/fs/workload: enable snap_schedule early 56424/head
authorPatrick Donnelly <pdonnell@redhat.com>
Thu, 21 Mar 2024 01:52:11 +0000 (21:52 -0400)
committerPatrick Donnelly <pdonnell@redhat.com>
Sat, 23 Mar 2024 13:13:56 +0000 (09:13 -0400)
Otherwise it can cause client evictions when the volumes plugin has mounted
cephfs.

Fixes: https://tracker.ceph.com/issues/64988
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit eb7b5c800fd03c3e6e699c3281ed897fa48c9085)

qa/cephfs/begin/3-modules.yaml [new file with mode: 0644]
qa/suites/fs/workload/begin/3-modules.yaml [new symlink]
qa/suites/fs/workload/tasks/3-snaps/yes.yaml

diff --git a/qa/cephfs/begin/3-modules.yaml b/qa/cephfs/begin/3-modules.yaml
new file mode 100644 (file)
index 0000000..2594734
--- /dev/null
@@ -0,0 +1,19 @@
+# Enable mgr modules now before any CephFS mounts are created by the mgr.  This
+# avoids the potential race of the mgr mounting CephFS and then getting failed
+# over by the monitors before the monitors have a chance to note the new client
+# session from the mgr beacon. In that case, the monitors will not blocklist
+# that client mount automatically so the MDS will eventually do the eviction
+# (and create a cluster log warning which we want to avoid).
+#
+# Note: ideally the mgr would gently stop mgr modules before respawning so that
+# the client mounts can be unmounted but this caused issues historically with
+# modules like the dashboard so an abrupt restart was chosen instead.
+
+mgrmodules:
+  sequential:
+    - print: "Enabling mgr modules"
+    # other fragments append to this
+
+tasks:
+  - sequential:
+      - mgrmodules
diff --git a/qa/suites/fs/workload/begin/3-modules.yaml b/qa/suites/fs/workload/begin/3-modules.yaml
new file mode 120000 (symlink)
index 0000000..1eba706
--- /dev/null
@@ -0,0 +1 @@
+.qa/cephfs/begin/3-modules.yaml
\ No newline at end of file
index 1a6a6cc60142f582f9ae85e9d1827ca11536ec2f..69f53768df812c65233f5ae56111aa46290abb88 100644 (file)
@@ -1,3 +1,10 @@
+mgrmodules:
+  sequential:
+    - exec:
+        mon.a:
+          - ceph mgr module enable snap_schedule
+          - ceph config set mgr mgr/snap_schedule/allow_m_granularity true
+          - ceph config set mgr mgr/snap_schedule/dump_on_update true
 overrides:
   ceph:
     conf:
@@ -12,9 +19,6 @@ overrides:
 tasks:
 - exec:
     mon.a:
-      - ceph mgr module enable snap_schedule
-      - ceph config set mgr mgr/snap_schedule/allow_m_granularity true
-      - ceph config set mgr mgr/snap_schedule/dump_on_update true
       - ceph fs snap-schedule add --fs=cephfs --path=/ --snap_schedule=1m
       - ceph fs snap-schedule retention add --fs=cephfs --path=/ --retention-spec-or-period=6m3h
       - ceph fs snap-schedule status --fs=cephfs --path=/