[Bugs] [Bug 1540908] New: Do lock conflict check correctly for wait-list

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1540908

            Bug ID: 1540908
           Summary: Do lock conflict check correctly for wait-list
           Product: Red Hat Gluster Storage
           Version: 3.4
         Component: disperse
          Assignee: aspandey at redhat.com
          Reporter: pkarampu at redhat.com
        QA Contact: nchilaka at redhat.com
                CC: bugs at gluster.org, rhs-bugs at redhat.com,
                    storage-qa-internal at redhat.com
        Depends On: 1540669, 1540882, 1540896



+++ This bug was initially created as a clone of Bug #1540896 +++

+++ This bug was initially created as a clone of Bug #1540882 +++

+++ This bug was initially created as a clone of Bug #1540669 +++

Description of problem:
    Problem:
    ec_link_has_lock_conflict() is traversing over only owner_list
    but the function is also getting called with wait_list.

    Fix:
    Modify ec_link_has_lock_conflict() to traverse lists correctly.
    Updated the callers to reflect the changes.


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


How reproducible:
Found while code-reading

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

--- Additional comment from Worker Ant on 2018-01-31 12:11:00 EST ---

REVIEW: https://review.gluster.org/19415 (cluster/ec: Do lock conflict check
correctly for wait-list) posted (#1) for review on master by Pranith Kumar
Karampuri

--- Additional comment from Worker Ant on 2018-02-01 04:07:51 EST ---

COMMIT: https://review.gluster.org/19415 committed in master by "Xavier
Hernandez" <jahernan at redhat.com> with a commit message- cluster/ec: Do lock
conflict check correctly for wait-list

Problem:
ec_link_has_lock_conflict() is traversing over only owner_list
but the function is also getting called with wait_list.

Fix:
Modify ec_link_has_lock_conflict() to traverse lists correctly.
Updated the callers to reflect the changes.

BUG: 1540669
Change-Id: Ibd7ea10f4498e7c2761f9a6faac6d5cb7d750c91
Signed-off-by: Pranith Kumar K <pkarampu at redhat.com>

--- Additional comment from Worker Ant on 2018-02-01 04:32:38 EST ---

REVIEW: https://review.gluster.org/19426 (cluster/ec: Do lock conflict check
correctly for wait-list) posted (#1) for review on release-4.0 by Pranith Kumar
Karampuri

--- Additional comment from Worker Ant on 2018-02-01 04:53:35 EST ---

REVIEW: https://review.gluster.org/19428 (cluster/ec: Do lock conflict check
correctly for wait-list) posted (#1) for review on release-3.13 by Pranith
Kumar Karampuri


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1540669
[Bug 1540669] Do lock conflict check correctly for wait-list
https://bugzilla.redhat.com/show_bug.cgi?id=1540882
[Bug 1540882] Do lock conflict check correctly for wait-list
https://bugzilla.redhat.com/show_bug.cgi?id=1540896
[Bug 1540896] Do lock conflict check correctly for wait-list
-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=LLZOmeC2Rn&a=cc_unsubscribe