Bug 1599423

Summary: Add performance options to generated gdeploy config file
Product: [Red Hat Storage] Red Hat Gluster Storage Reporter: Manoj Pillai <mpillai>
Component: rhhiAssignee: Sahina Bose <sabose>
Status: CLOSED CURRENTRELEASE QA Contact: SATHEESARAN <sasundar>
Severity: high Docs Contact:
Priority: medium    
Version: rhgs-3.3CC: godas, mkarg, mpillai, nichawla, rhs-bugs, sabose, sankarshan
Target Milestone: ---Keywords: Performance
Target Release: RHHI-V 1.5   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1619627 (view as bug list) Environment:
Last Closed: 2019-05-20 04:54:42 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: 1619627    
Bug Blocks: 1520836    

Description Manoj Pillai 2018-07-09 18:22:19 UTC 
Description of problem:
The option performance.client-io-threads, which has been unavailable as a tuning option for replicated volumes in recent releases, is available in RHGS 3.3.1-async and RHGS 3.4. RHHI can benefit by adding it to the group virt profile, along with a few others. The following is the list of suggested additions to group virt:

performance.client-io-threads on
client.event-threads 4
server.event-threads 4

In addition, "cluster.choose-local false" is beneficial, and is already covered in bz #1568297.

In most random I/O tests on flash, the fuse thread is currently seen to be the bottleneck. client-io-threads, along with the other options, helps push IOPS far beyond what RHHI can currently provide.

Additional info:
Might need to revisit these settings if and when two planned enhancements arrive:
1. libgfapi support
2. multi-threadeded fuse reader
Comment 3 Yaniv Kaul 2018-08-20 12:38:20 UTC 
Sahina, any reason not to consider this for 4.2.6 / 4.2.7?
Comment 4 Sahina Bose 2018-08-20 13:03:23 UTC 
(In reply to Yaniv Kaul from comment #3)
> Sahina, any reason not to consider this for 4.2.6 / 4.2.7?

The group virt profile change needs to go into RHGS release. We can change the cockpit deployment to add these options in 4.2.6 - logged 1619256
Comment 12 Yaniv Kaul 2018-10-09 17:23:05 UTC 
Is this a tracker bug? Why doesn't it have all the ACKs? Isn't it going to 2.0?
Comment 13 Sahina Bose 2018-10-10 07:36:13 UTC 
(In reply to Yaniv Kaul from comment #12)
> Is this a tracker bug? Why doesn't it have all the ACKs? Isn't it going to
> 2.0?

This is a tracker, and going into 2.0 (added to in-flight tracker).dev acked now
Comment 14 Sahina Bose 2018-10-10 08:19:19 UTC 
The changes have been made to group virt profile as per Bug 1619627
Comment 18 SATHEESARAN 2018-10-10 08:38:12 UTC 
Verified with glusterfs-3.12.2-21.el7rhgs.

The required options are set on the volume and are available in the virt profile