From: Venky Shankar <vshankar@redhat.com>
Date: Mon, 9 Oct 2023 05:06:49 +0000 (+0530)
Subject: Revert "mds: disable delegating inode ranges to clients"
X-Git-Tag: v19.0.0~303^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F53885%2Fhead;p=ceph.git

Revert "mds: disable delegating inode ranges to clients"

This isn't necessary -- the MDS handles delegating inode ranges
to clients from its preallocated inode set properly - the suspected
bug involving not persisting the sessionmap and causing asserts
during replay isn't an issue. The preallocated set is persisted
with the log event and the MDS correctly rebuild the set from
this information during replay.

Signed-off-by: Venky Shankar <vshankar@redhat.com>
---

diff --git a/PendingReleaseNotes b/PendingReleaseNotes
index eae5f22efcf7..1fc9c4510d3e 100644
--- a/PendingReleaseNotes
+++ b/PendingReleaseNotes
@@ -50,9 +50,6 @@
   recommend that users with versioned buckets, especially those that existed
   on prior releases, use these new tools to check whether their buckets are
   affected and to clean them up accordingly.
-CephFS: Disallow delegating preallocated inode ranges to clients. Config
-  `mds_client_delegate_inos_pct` defaults to 0 which disables async dirops
-  in the kclient.
 
 >=18.0.0
 
diff --git a/qa/suites/fs/workload/delegate_inos/default.yaml b/qa/suites/fs/workload/delegate_inos/default.yaml
deleted file mode 100644
index 219407f33fe0..000000000000
--- a/qa/suites/fs/workload/delegate_inos/default.yaml
+++ /dev/null
@@ -1 +0,0 @@
-# Use default (0)
diff --git a/qa/suites/fs/workload/delegate_inos/enabled.yaml b/qa/suites/fs/workload/delegate_inos/enabled.yaml
deleted file mode 100644
index 6c0e0da27a13..000000000000
--- a/qa/suites/fs/workload/delegate_inos/enabled.yaml
+++ /dev/null
@@ -1,5 +0,0 @@
-overrides:
-  ceph:
-    conf:
-      mds:
-        mds_client_delegate_inos_pct: 50
diff --git a/src/common/options/mds.yaml.in b/src/common/options/mds.yaml.in
index 4ce457234ba1..2599b6532b5d 100644
--- a/src/common/options/mds.yaml.in
+++ b/src/common/options/mds.yaml.in
@@ -454,7 +454,7 @@ options:
   type: uint
   level: advanced
   desc: percentage of preallocated inos to delegate to client
-  default: 0
+  default: 50
   services:
   - mds
   flags: