[Bugs] [Bug 1155017] New: Tracker bug for afrv1 changelog support in afrv2

Tue Oct 21 09:28:50 UTC 2014

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

            Bug ID: 1155017
           Summary: Tracker bug for afrv1 changelog support in afrv2
           Product: GlusterFS
           Version: 3.6.0
         Component: replicate
          Assignee: gluster-bugs at redhat.com
          Reporter: pkarampu at redhat.com
                CC: bugs at gluster.org, gluster-bugs at redhat.com,
                    vbellur at redhat.com
        Depends On: 1128721, 1136829
            Blocks: 1117822 (glusterfs-3.6.0)

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

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

Description of problem:
This bug tracks the changes need to be done for afrv1 changelog support in
afrv2

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

How reproducible:

Steps to Reproduce:
1.
2.
3.

Actual results:

Expected results:

Additional info:

--- Additional comment from Anand Avati on 2014-08-11 08:19:08 EDT ---

REVIEW: http://review.gluster.org/8456 (cluster/afr: s/healed_sinks/sinks in
data self-heal) posted (#1) for review on master by Pranith Kumar Karampuri
(pkarampu at redhat.com)

--- Additional comment from Anand Avati on 2014-08-12 00:02:47 EDT ---

REVIEW: http://review.gluster.org/8456 (cluster/afr: Fix all locked_on bricks
are sinks check is self-heals) posted (#2) for review on master by Pranith
Kumar Karampuri (pkarampu at redhat.com)

--- Additional comment from Anand Avati on 2014-08-12 00:03:33 EDT ---

REVIEW: http://review.gluster.org/8456 (cluster/afr: Fix all locked_on bricks
are sinks check in self-heals) posted (#3) for review on master by Pranith
Kumar Karampuri (pkarampu at redhat.com)

--- Additional comment from Anand Avati on 2014-08-12 02:41:55 EDT ---

REVIEW: http://review.gluster.org/8456 (cluster/afr: Fix all locked_on bricks
are sinks check in self-heals) posted (#4) for review on master by Pranith
Kumar Karampuri (pkarampu at redhat.com)

--- Additional comment from Anand Avati on 2014-08-25 09:11:47 EDT ---

REVIEW: http://review.gluster.org/8456 (cluster/afr: Fix all locked_on bricks
are sinks check in self-heals) posted (#5) for review on master by Pranith
Kumar Karampuri (pkarampu at redhat.com)

--- Additional comment from Anand Avati on 2014-08-25 09:12:05 EDT ---

REVIEW: http://review.gluster.org/8536 (cluster/afr: Add afr-v1 xattr
compatibility) posted (#1) for review on master by Pranith Kumar Karampuri
(pkarampu at redhat.com)

--- Additional comment from Anand Avati on 2014-08-25 09:12:08 EDT ---

REVIEW: http://review.gluster.org/8537 (cluster/afr: Fix synchronization issue
with entry, name self-heals) posted (#1) for review on master by Pranith Kumar
Karampuri (pkarampu at redhat.com)

--- Additional comment from Anand Avati on 2014-08-25 09:12:11 EDT ---

REVIEW: http://review.gluster.org/8538 (tests: Data self-heal test cases)
posted (#1) for review on master by Pranith Kumar Karampuri
(pkarampu at redhat.com)

--- Additional comment from Anand Avati on 2014-08-25 09:12:32 EDT ---

REVIEW: http://review.gluster.org/8539 (tests: entry self-heal test cases)
posted (#1) for review on master by Pranith Kumar Karampuri
(pkarampu at redhat.com)

--- Additional comment from Anand Avati on 2014-08-26 01:52:59 EDT ---

REVIEW: http://review.gluster.org/8456 (cluster/afr: Fix all locked_on bricks
are sinks check in self-heals) posted (#6) for review on master by Pranith
Kumar Karampuri (pkarampu at redhat.com)

--- Additional comment from Anand Avati on 2014-08-26 04:17:28 EDT ---

COMMIT: http://review.gluster.org/8456 committed in master by Pranith Kumar
Karampuri (pkarampu at redhat.com) 
------
commit dc450ab5e3798b47e89e90b64c49cb8e1f20efce
Author: Pranith Kumar K <pkarampu at redhat.com>
Date:   Fri Aug 8 11:52:17 2014 +0530

    cluster/afr: Fix all locked_on bricks are sinks check in self-heals

    Problem:
    Counts may give wrong results when the number of bricks is > 2. If the
    locks are acquired on one source and sink, but the source accuses even the
    down sink then there will be 2 sinks and lock is acquired on 2 bricks so
    even when there is a clear source and sink **_finalize_source functions
think
    the file/directory is in split-brain.

    Fix:
    Check that all the bricks which are locked are sinks.

    Change-Id: Ia43790e8e1bfb5e72a3d0b56bcad94abd0dc58ab
    BUG: 1128721
    Signed-off-by: Pranith Kumar K <pkarampu at redhat.com>
    Reviewed-on: http://review.gluster.org/8456
    Reviewed-by: Ravishankar N <ravishankar at redhat.com>
    Reviewed-by: Krutika Dhananjay <kdhananj at redhat.com>
    Tested-by: Gluster Build System <jenkins at build.gluster.com>

--- Additional comment from Anand Avati on 2014-08-26 08:26:01 EDT ---

REVIEW: http://review.gluster.org/8537 (cluster/afr: Fix synchronization issue
with entry, name self-heals) posted (#2) for review on master by Pranith Kumar
Karampuri (pkarampu at redhat.com)

--- Additional comment from Anand Avati on 2014-08-26 08:26:04 EDT ---

REVIEW: http://review.gluster.org/8536 (cluster/afr: Add afr-v1 xattr
compatibility) posted (#2) for review on master by Pranith Kumar Karampuri
(pkarampu at redhat.com)

--- Additional comment from Anand Avati on 2014-08-26 08:26:07 EDT ---

REVIEW: http://review.gluster.org/8539 (tests: entry self-heal test cases)
posted (#2) for review on master by Pranith Kumar Karampuri
(pkarampu at redhat.com)

--- Additional comment from Anand Avati on 2014-08-26 08:26:10 EDT ---

REVIEW: http://review.gluster.org/8538 (tests: Data self-heal test cases)
posted (#2) for review on master by Pranith Kumar Karampuri
(pkarampu at redhat.com)

--- Additional comment from Anand Avati on 2014-09-03 08:03:37 EDT ---

REVIEW: http://review.gluster.org/8593 (cluster/afr: Fix all locked_on bricks
are sinks check in self-heals) posted (#1) for review on release-3.6 by Pranith
Kumar Karampuri (pkarampu at redhat.com)

--- Additional comment from Anand Avati on 2014-09-12 05:55:41 EDT ---

COMMIT: http://review.gluster.org/8593 committed in release-3.6 by Vijay Bellur
(vbellur at redhat.com) 
------
commit 43d49fab8cce7d06fc63f23757abc7f2c70a6d48
Author: Pranith Kumar K <pkarampu at redhat.com>
Date:   Fri Aug 8 11:52:17 2014 +0530

    cluster/afr: Fix all locked_on bricks are sinks check in self-heals

            Backport of http://review.gluster.org/8456

    Problem:
    Counts may give wrong results when the number of bricks is > 2. If the
    locks are acquired on one source and sink, but the source accuses even the
    down sink then there will be 2 sinks and lock is acquired on 2 bricks so
    even when there is a clear source and sink **_finalize_source functions
think
    the file/directory is in split-brain.

    Fix:
    Check that all the bricks which are locked are sinks.

    BUG: 1136829
    Change-Id: I56a8f9ff261bdeec8c441237c485036141b6f00d
    Signed-off-by: Pranith Kumar K <pkarampu at redhat.com>
    Reviewed-on: http://review.gluster.org/8593
    Reviewed-by: Krutika Dhananjay <kdhananj at redhat.com>
    Tested-by: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Ravishankar N <ravishankar at redhat.com>
    Reviewed-by: Vijay Bellur <vbellur at redhat.com>

--- Additional comment from Niels de Vos on 2014-09-22 08:45:29 EDT ---

A beta release for GlusterFS 3.6.0 has been released. Please verify if the
release solves this bug report for you. In case the glusterfs-3.6.0beta1
release does not have a resolution for this issue, leave a comment in this bug
and move the status to ASSIGNED. If this release fixes the problem for you,
leave a note and change the status to VERIFIED.

Packages for several distributions should become available in the near future.
Keep an eye on the Gluster Users mailinglist [2] and the update (possibly an
"updates-testing" repository) infrastructure for your distribution.

[1]
http://supercolony.gluster.org/pipermail/gluster-users/2014-September/018836.html
[2] http://supercolony.gluster.org/pipermail/gluster-users/

--- Additional comment from Anand Avati on 2014-09-23 10:48:25 EDT ---

REVIEW: http://review.gluster.org/8823 (cluster/afr: Do not reset pending
xattrs on gfid or type mismatch in entry-sh) posted (#1) for review on
release-3.6 by Krutika Dhananjay (kdhananj at redhat.com)

--- Additional comment from Anand Avati on 2014-09-23 12:50:26 EDT ---

COMMIT: http://review.gluster.org/8823 committed in release-3.6 by Vijay Bellur
(vbellur at redhat.com) 
------
commit 6fffc4eaf408cb9b0b2891404ee54f4ab1dd5e2e
Author: Krutika Dhananjay <kdhananj at redhat.com>
Date:   Thu Sep 18 18:26:08 2014 +0530

    cluster/afr: Do not reset pending xattrs on gfid or type mismatch in
entry-sh

            Backport of: http://review.gluster.org/8816

    Change-Id: I8463a579f542a2336b02edba8f5fbfea0edbbffe
    BUG: 1136829
    Signed-off-by: Krutika Dhananjay <kdhananj at redhat.com>
    Reviewed-on: http://review.gluster.org/8823
    Reviewed-by: Pranith Kumar Karampuri <pkarampu at redhat.com>
    Tested-by: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Vijay Bellur <vbellur at redhat.com>

--- Additional comment from Anand Avati on 2014-09-25 03:08:00 EDT ---

REVIEW: http://review.gluster.org/8845 (cluster/afr: Fix locking issues in
entry self-heal) posted (#1) for review on release-3.6 by Krutika Dhananjay
(kdhananj at redhat.com)

--- Additional comment from Niels de Vos on 2014-09-25 04:27:40 EDT ---

A beta release for GlusterFS 3.6.0 has been released. Please verify if the
release solves this bug report for you. In case the glusterfs-3.6.0beta2
release does not have a resolution for this issue, leave a comment in this bug
and move the status to ASSIGNED. If this release fixes the problem for you,
leave a note and change the status to VERIFIED.

Packages for several distributions should become available in the near future.
Keep an eye on the Gluster Users mailinglist [2] and the update (possibly an
"updates-testing" repository) infrastructure for your distribution.

[1]
http://supercolony.gluster.org/pipermail/gluster-users/2014-September/018883.html
[2] http://supercolony.gluster.org/pipermail/gluster-users/

--- Additional comment from Anand Avati on 2014-09-26 07:05:09 EDT ---

COMMIT: http://review.gluster.org/8845 committed in release-3.6 by Vijay Bellur
(vbellur at redhat.com) 
------
commit 537ad5b9ea1e95cf68c2ced2e56e10283f221e11
Author: Krutika Dhananjay <kdhananj at redhat.com>
Date:   Tue Sep 23 17:03:53 2014 +0530

    cluster/afr: Fix locking issues in entry self-heal

            Backport of: http://review.gluster.org/#/c/8837/

    Original reporter of the bug & designer of the solution:
            Pranith Kumar K <pkarampu at redhat.com>

    Change-Id: I6e36326e1d9398ede82166b358ab438367dd3011
    BUG: 1136829
    Signed-off-by: Krutika Dhananjay <kdhananj at redhat.com>
    Reviewed-on: http://review.gluster.org/8845
    Reviewed-by: Pranith Kumar Karampuri <pkarampu at redhat.com>
    Tested-by: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Vijay Bellur <vbellur at redhat.com>

Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1117822
[Bug 1117822] Tracker bug for GlusterFS 3.6.0
https://bugzilla.redhat.com/show_bug.cgi?id=1128721
[Bug 1128721] Tracker bug for afrv1 changelog support in afrv2
https://bugzilla.redhat.com/show_bug.cgi?id=1136829
[Bug 1136829] Tracker bug for afrv1 changelog support in afrv2
-- 
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=fnNqUzPfBy&a=cc_unsubscribe