From: Venky Shankar <vshankar@redhat.com>
Date: Sun, 25 Oct 2020 07:01:03 +0000 (-0400)
Subject: cephfs-mirror: remove `cephfs_mirror_directory_choose_policy` option
X-Git-Tag: v16.1.0~51^2~7
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=493a050155c8e7e519f5c2b1de853dcf0ea860ee;p=ceph.git

cephfs-mirror: remove `cephfs_mirror_directory_choose_policy` option

Basically, the mirror daemon would process directory list in in a
round-robin fashion. Randomly choosing directories probably does not
make much sense right now.

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

diff --git a/src/common/options.cc b/src/common/options.cc
index 2affa3e38c3c..1f15939a6783 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -8898,11 +8898,6 @@ std::vector<Option> get_cephfs_mirror_options() {
     .set_description("maximum number of concurrent snapshot synchronization threads")
     .set_long_description("maximum number of directory snapshots that can be synchronized concurrently by cephfs-mirror daemon. Controls the number of synchronization threads."),
 
-    Option("cephfs_mirror_directory_choose_policy", Option::TYPE_STR, Option::LEVEL_ADVANCED)
-    .set_default("random")
-    .set_description("policy for choosing directories to mirror snapshots")
-    .set_long_description("policy used by cephfs-mirror daemon to choose directories for snapshot mirroring"),
-
     Option("cephfs_mirror_action_update_interval", Option::TYPE_SECS, Option::LEVEL_ADVANCED)
     .set_default(2)
     .set_min(1)