Bug 1338593 - clean posix locks based on client-id as part of server_connection_cleanup
Summary: clean posix locks based on client-id as part of server_connection_cleanup
Keywords:
Status: CLOSED DEFERRED
Alias: None
Product: GlusterFS
Classification: Community
Component: locks
Version: mainline
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Susant Kumar Palai
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 1384388
TreeView+ depends on / blocked
 
Reported: 2016-05-23 05:37 UTC by Susant Kumar Palai
Modified: 2019-05-11 02:10 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
: 1384388 (view as bug list)
Environment:
Last Closed: 2019-05-11 02:10:27 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:

Attachments (Terms of Use)

Description Susant Kumar Palai 2016-05-23 05:37:38 UTC 
Description of problem:
  Currently posix locks are removed based on the fd number in case of a server disconnect event. Post lock migration, the locks will not have any fd association on destination. Hence, posix locks need to be removed based on client-id.
Comment 1 Vijay Bellur 2016-05-23 06:20:15 UTC 
REVIEW: http://review.gluster.org/14492 (posix/locks: associate posix locks with client-id for cleanup) posted (#1) for review on master by Susant Palai (spalai)
Comment 2 Vijay Bellur 2016-06-13 10:40:30 UTC 
REVIEW: http://review.gluster.org/14492 (posix/locks: associate posix locks with client-uuid) posted (#2) for review on master by Susant Palai (spalai)
Comment 3 Vijay Bellur 2016-08-01 05:47:32 UTC 
REVIEW: http://review.gluster.org/14492 (posix/locks: associate posix locks with client-uuid) posted (#3) for review on master by Susant Palai (spalai)
Comment 4 Worker Ant 2016-09-14 07:40:58 UTC 
REVIEW: http://review.gluster.org/14492 (posix/locks: associate posix locks with client-uuid) posted (#4) for review on master by Susant Palai (spalai)
Comment 5 Worker Ant 2016-09-20 08:39:28 UTC 
REVIEW: http://review.gluster.org/14492 (posix/locks: associate posix locks with client-uuid) posted (#5) for review on master by Susant Palai (spalai)
Comment 6 Worker Ant 2016-09-20 12:56:34 UTC 
REVIEW: http://review.gluster.org/14492 (posix/locks: associate posix locks with client-uuid) posted (#6) for review on master by Susant Palai (spalai)
Comment 7 Worker Ant 2016-09-27 13:44:36 UTC 
REVIEW: http://review.gluster.org/14492 (posix/locks: associate posix locks with client-uuid) posted (#7) for review on master by Susant Palai (spalai)
Comment 8 Worker Ant 2016-09-27 13:49:40 UTC 
REVIEW: http://review.gluster.org/14492 (posix/locks: associate posix locks with client-uuid) posted (#8) for review on master by Susant Palai (spalai)
Comment 9 Worker Ant 2016-09-28 03:07:50 UTC 
REVIEW: http://review.gluster.org/14492 (posix/locks: associate posix locks with client-uuid) posted (#9) for review on master by Susant Palai (spalai)
Comment 10 Worker Ant 2016-10-03 10:27:08 UTC 
REVIEW: http://review.gluster.org/14492 (posix/locks: associate posix locks with client-uuid) posted (#10) for review on master by Susant Palai (spalai)
Comment 11 Worker Ant 2016-10-03 10:53:55 UTC 
REVIEW: http://review.gluster.org/14492 (posix/locks: associate posix locks with client-uuid) posted (#11) for review on master by Susant Palai (spalai)
Comment 12 Worker Ant 2016-10-03 11:03:51 UTC 
REVIEW: http://review.gluster.org/14492 (posix/locks: associate posix locks with client-uuid) posted (#12) for review on master by Susant Palai (spalai)
Comment 13 Amar Tumballi 2019-05-11 02:10:27 UTC 
Any progress done?

Considering we had not issues due to this in long time, marking it as DEFERRED. Will revisit when we get time.
Note You need to log in before you can comment on or make changes to this bug.