Bug 1591873

Summary: Set default filestore_merge_threshold = -10
Product: [Red Hat Storage] Red Hat Ceph Storage Reporter: Douglas Fuller <dfuller>
Component: RADOSAssignee: Douglas Fuller <dfuller>
Status: CLOSED ERRATA QA Contact: Tiffany Nguyen <tunguyen>
Severity: high Docs Contact: Aron Gunn <agunn>
Priority: high    
Version: 3.1CC: agunn, cbodley, ceph-eng-bugs, ceph-qe-bugs, dfuller, dzafman, jharriga, kbader, kchai, kdreyer, mbenjamin, owasserm, sweil, vakulkar
Target Milestone: rc   
Target Release: 3.1   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: RHEL: ceph-12.2.5-28.el7cp Ubuntu: ceph_12.2.5-14redhat1 Doc Type: Enhancement
Doc Text:
.The `filestore_merge_threshold` option default has changed Subdirectory merging has been disabled by default. The default value of the `filestore_merge_threshold` option has changed to -10 from 10. It has been observed to improve performance significantly on larger systems with a minimal performance impact to smaller systems. To take advantage of this performance increase set the `expected-num-objects` value when creating new data pools. See the link:{object-gw-production}#creating-a-data-pool-rgw-adv[Object Gateway for Production Guide] for more information.
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-09-26 18:22:08 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1602921    
Bug Blocks: 1581350, 1584264    

Description Douglas Fuller 2018-06-15 17:47:32 UTC
Larger RGW deployments often thrash RADOS objects and incur performance penalties during filestore split and merge operations. Disable filestore merging automatically for these deployments.

Set:

[osd]
filestore_merge_threshold = -1 #default 10

Comment 3 Douglas Fuller 2018-06-28 14:33:37 UTC
-10 is a better choice because the absolute value is used in some filestore calculations.

Comment 4 Douglas Fuller 2018-06-28 15:10:10 UTC
https://github.com/ceph/ceph/pull/22761

Comment 9 Tiffany Nguyen 2018-08-24 05:26:50 UTC
Verified with build 12.2.5-39.el7cp.  filestore_merge_threshold is set as -10 by default.

[root@c04-h05-6048r ~]# ceph -v 
ceph version 12.2.5-39.el7cp (f12d44e46a54948a86dd27b16c77d97475ba2d4e) luminous (stable) 
[root@c04-h05-6048r ~]# ceph daemon osd.0 config get filestore_merge_threshold  
{ 
    "filestore_merge_threshold": "-10" 
}

Comment 12 errata-xmlrpc 2018-09-26 18:22:08 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2018:2819