From 660abda21163bf4297b325ea2ac42a28661d5f62 Mon Sep 17 00:00:00 2001
From: Casey Bodley <cbodley@redhat.com>
Date: Tue, 24 Oct 2017 17:55:34 -0400
Subject: [PATCH] rgw: more documentation for bilog trim config

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 4d4442168c15775f8346ebc6295a6de912250b8f)
---
 src/common/options.cc | 17 ++++++++++++++---
 1 file changed, 14 insertions(+), 3 deletions(-)

diff --git a/src/common/options.cc b/src/common/options.cc
index bda153c32c3..25638d0d2b3 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -4933,15 +4933,26 @@ std::vector<Option> get_rgw_options() {
 
     Option("rgw_sync_log_trim_max_buckets", Option::TYPE_INT, Option::LEVEL_ADVANCED)
     .set_default(16)
-    .set_description("Maximum number of buckets to trim per interval"),
+    .set_description("Maximum number of buckets to trim per interval")
+    .set_long_description("The maximum number of buckets to consider for bucket index log trimming each trim interval, regardless of the number of bucket index shards. Priority is given to buckets with the most sync activity over the last trim interval.")
+    .add_see_also("rgw_sync_log_trim_interval")
+    .add_see_also("rgw_sync_log_trim_min_cold_buckets")
+    .add_see_also("rgw_sync_log_trim_concurrent_buckets"),
 
     Option("rgw_sync_log_trim_min_cold_buckets", Option::TYPE_INT, Option::LEVEL_ADVANCED)
     .set_default(4)
-    .set_description("Minimum number of cold buckets to trim per interval"),
+    .set_description("Minimum number of cold buckets to trim per interval")
+    .set_long_description("Of the `rgw_sync_log_trim_max_buckets` selected for bucket index log trimming each trim interval, at least this many of them must be 'cold' buckets. These buckets are selected in order from the list of all bucket instances, to guarantee that all buckets will be visited eventually.")
+    .add_see_also("rgw_sync_log_trim_interval")
+    .add_see_also("rgw_sync_log_trim_max_buckets")
+    .add_see_also("rgw_sync_log_trim_concurrent_buckets"),
 
     Option("rgw_sync_log_trim_concurrent_buckets", Option::TYPE_INT, Option::LEVEL_ADVANCED)
     .set_default(4)
-    .set_description("Maximum number of buckets to trim in parallel"),
+    .set_description("Maximum number of buckets to trim in parallel")
+    .add_see_also("rgw_sync_log_trim_interval")
+    .add_see_also("rgw_sync_log_trim_max_buckets")
+    .add_see_also("rgw_sync_log_trim_min_cold_buckets"),
 
     Option("rgw_sync_data_inject_err_probability", Option::TYPE_FLOAT, Option::LEVEL_DEV)
     .set_default(0)
-- 
2.47.3