Bug 1714172 - ec ignores lock contention notifications for partially acquired locks
Summary: ec ignores lock contention notifications for partially acquired locks
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: disperse
Version: 6
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Xavi Hernandez
QA Contact:
URL:
Whiteboard:
Depends On: 1708156 1717282
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-05-27 10:02 UTC by Xavi Hernandez
Modified: 2019-07-11 09:11 UTC (History)
3 users (show)

Fixed In Version:
Clone Of: 1708156
Environment:
Last Closed: 2019-06-03 04:08:53 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Gluster.org Gerrit 22774 0 None Merged cluster/ec: honor contention notifications for partially acquired locks 2019-06-03 04:08:52 UTC

Description Xavi Hernandez 2019-05-27 10:02:40 UTC
+++ This bug was initially created as a clone of Bug #1708156 +++

Description of problem:

When an inodelk is being acquired, it could happen that some bricks have already granted the lock while others don't. From the point of view of ec, the lock is not yet acquired.

If at this point one of the bricks that has already granted the lock receives another inodelk request, it will send a contention notification to ec.

Currently ec ignores those notifications until the lock is fully acquired. This means than once ec acquires the lock on all bricks, it won't be released immediately when eager-lock is used.

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


How reproducible:

Very frequently when there are multiple concurrent operations on same directory

Steps to Reproduce:
1. Create a disperse volume
2. Mount it from several clients
3. Create few files on a directory
4. Do 'ls' of that directory at the same time from all clients

Actual results:

Some 'ls' take several seconds to complete

Expected results:

All 'ls' should complete in less than a second

Comment 1 Worker Ant 2019-05-27 10:13:29 UTC
REVIEW: https://review.gluster.org/22774 (cluster/ec: honor contention notifications for partially acquired locks) posted (#1) for review on release-6 by Xavi Hernandez

Comment 2 Worker Ant 2019-06-03 04:08:53 UTC
REVIEW: https://review.gluster.org/22774 (cluster/ec: honor contention notifications for partially acquired locks) merged (#2) on release-6 by Amar Tumballi

Comment 3 hari gowtham 2019-07-11 09:11:53 UTC
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-6.3, please open a new bug report.

glusterfs-6.3 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] https://lists.gluster.org/pipermail/gluster-users/2019-July/036790.html
[2] https://www.gluster.org/pipermail/gluster-users/


Note You need to log in before you can comment on or make changes to this bug.