[RM-14355] Remove "filestore xattr use omap = true"

This patch removes option "filestore_xattr_use_omap"
from default ceph.conf global section,
as this option was removed in #7408.

Cleanup #14355

Signed-off-by: Vikhyat Umrao <vumrao@redhat.com>

diff --git a/ceph_deploy/new.py b/ceph_deploy/new.py
index 359f6bfc94b7b756786a4035dd739d6f3f462d2e..7f308d4e77cc7775764ee67b57429d5df55de249 100644 (file)
--- a/ceph_deploy/new.py
+++ b/ceph_deploy/new.py
@@ -190,9 +190,6 @@ def new(args):
     cfg.set('global', 'auth service required', 'cephx')
     cfg.set('global', 'auth client required', 'cephx')
 
-    # http://tracker.newdream.net/issues/3138
-    cfg.set('global', 'filestore xattr use omap', 'true')
-
     path = '{name}.conf'.format(
         name=args.cluster,
         )

diff --git a/ceph_deploy/tests/test_cli_new.py b/ceph_deploy/tests/test_cli_new.py
index cdd2ae647d0d1c6ebaa100c300c1a737efa5439a..809709c19b4b645e83482ff8eb91f20955d97744 100644 (file)
--- a/ceph_deploy/tests/test_cli_new.py
+++ b/ceph_deploy/tests/test_cli_new.py
@@ -68,4 +68,3 @@ def test_defaults(newcfg):
     assert cfg.get('global', 'auth cluster required') == 'cephx'
     assert cfg.get('global', 'auth service required') == 'cephx'
     assert cfg.get('global', 'auth client required') == 'cephx'
-    assert cfg.get('global', 'filestore_xattr_use_omap') == 'true'