rbd: update default image features

Exclusive lock, object map, fast-diff, and deep-flatten have been
enabled by default for all new images.

Signed-off-by: Jason Dillaman <dillaman@redhat.com>

diff --git a/PendingReleaseNotes b/PendingReleaseNotes
index c4140307b7030592cdfae60d79b1c16bcfb14e84..0b94b198f0e1305e9d991d060692f871ccd4bd76 100644 (file)
--- a/PendingReleaseNotes
+++ b/PendingReleaseNotes
@@ -12,6 +12,14 @@ v10.0.0
   limit max waiting time of monitor election process, which was previously
   restricted by 'mon_lease'.
 
+* The default RBD image features for new images have been updated to enable
+  the following: exclusive lock, object map, fast-diff, and deep-flatten.
+  These features are not currently supported by the RBD kernel driver nor older
+  RBD clients. These features can be disabled on a per-image basis via the RBD
+  CLI or the default features can be updated to the pre-Jewel setting by adding
+  the following to the client section of the Ceph configuration file:
+    rbd default features = 1
+
 v9.3.0
 ======
 * Some symbols wrongly exposed by librados in v9.1.0 and v9.2.0 were removed.

diff --git a/src/common/config_opts.h b/src/common/config_opts.h
index 03d4c6abf00844b41054e75ffb6df109afa58d39..427fdddb199919c325638b7aa463e0ba965b2dd7 100644 (file)
--- a/src/common/config_opts.h
+++ b/src/common/config_opts.h
@@ -1118,9 +1118,11 @@ OPTION(rbd_default_format, OPT_INT, 2)
 OPTION(rbd_default_order, OPT_INT, 22)
 OPTION(rbd_default_stripe_count, OPT_U64, 0) // changing requires stripingv2 feature
 OPTION(rbd_default_stripe_unit, OPT_U64, 0) // changing to non-object size requires stripingv2 feature
-OPTION(rbd_default_features, OPT_INT, 3) // only applies to format 2 images
-                                        // +1 for layering, +2 for stripingv2,
-                                        // +4 for exclusive lock, +8 for object map
+OPTION(rbd_default_features, OPT_INT, 61)   // only applies to format 2 images
+                                           // +1 for layering, +2 for stripingv2,
+                                           // +4 for exclusive lock, +8 for object map
+                                           // +16 for fast-diff, +32 for deep-flatten,
+                                           // +64 for journaling
 
 OPTION(rbd_default_map_options, OPT_STR, "") // default rbd map -o / --options
 

diff --git a/src/test/pybind/test_rbd.py b/src/test/pybind/test_rbd.py
index df98dee8c09c9fa6135787138914c29e862c2c9d..e77a0269203009d8e989318422cc15467b460f7c 100644 (file)
--- a/src/test/pybind/test_rbd.py
+++ b/src/test/pybind/test_rbd.py
@@ -143,7 +143,7 @@ def check_default_params(format, order=None, features=None, stripe_count=None,
 
                     expected_features = features
                     if expected_features is None or format == 1:
-                        expected_features = 0 if format == 1 else 3
+                        expected_features = 0 if format == 1 else 61
                     eq(expected_features, image.features())
 
                     expected_stripe_count = stripe_count