Bug 1427099

Summary:	[RFE] [Samba] Implement Negative lookup cache feature to improve create performance
Product:	[Red Hat Storage] Red Hat Gluster Storage	Reporter:	Poornima G <pgurusid>
Component:	samba	Assignee:	Poornima G <pgurusid>
Status:	CLOSED ERRATA	QA Contact:	Karan Sandha <ksandha>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	rhgs-3.3	CC:	amukherj, asrivast, pgurusid, rcyriac, rhinduja, rhs-smb, rjoseph
Target Milestone:	---	Keywords:	FutureFeature
Target Release:	RHGS 3.3.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.8.4-23	Doc Type:	Enhancement
Doc Text:	A negative lookup cache has been implemented for SMB clients. This removes redundant lookup operations, improving speed for file and directory creation from SMB clients.	Story Points:	---
Clone Of:
Clones:	1436709 (view as bug list)		Environment:
Last Closed:	2017-09-21 04:33:25 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:
Bug Blocks:	1417138, 1436709

Description Poornima G 2017-02-27 11:13:42 UTC

Description of problem:
Create in Samba is very slow compared to other access protocol. One of the main reason is negative lookups. After implementing this feature also it will not be on par with FUSE and NFS but will improve current performance by multiple folds.
The numbers are yet to be tested.

The feature is WIP and is not posted upstream other than design document.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 4 Atin Mukherjee 2017-03-07 13:16:28 UTC

upstream patch : https://review.gluster.org/16864

Comment 6 Atin Mukherjee 2017-04-06 04:02:49 UTC

(In reply to Atin Mukherjee from comment #4)
> upstream patch : https://review.gluster.org/16864

The above is abandoned now, the new patch set is https://review.gluster.org/#/c/16952/

Comment 9 Vivek Das 2017-04-24 09:41:11 UTC

1 Day and 1 round of testing has been performed over negative lookup cache.
The bug below is encountered but it is yet not clear that nl.cache caused it to happen.
https://bugzilla.redhat.com/show_bug.cgi?id=1444509

Need more testing to confirm.

Comment 10 Vivek Das 2017-04-26 12:36:32 UTC

Did one round of sanity along with multiple client testing scenarios, basic things seems to be working.

Comment 13 Poornima G 2017-09-11 06:52:33 UTC

Ack

Comment 15 errata-xmlrpc 2017-09-21 04:33:25 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-2017:2774

Comment 16 errata-xmlrpc 2017-09-21 04:57:48 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-2017:2774