[Bugs] [Bug 1660736] New: dht_revalidate may not heal attrs on the brick root

[bugzilla at redhat.com]

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

            Bug ID: 1660736
           Summary: dht_revalidate may not heal attrs on the brick root
           Product: GlusterFS
           Version: 5
            Status: NEW
         Component: distribute
          Keywords: ZStream
          Severity: high
          Priority: high
          Assignee: bugs at gluster.org
          Reporter: nbalacha at redhat.com
                CC: bugs at gluster.org, rhs-bugs at redhat.com,
                    sankarshan at redhat.com, storage-qa-internal at redhat.com,
                    tdesala at redhat.com
        Depends On: 1648298
            Blocks: 1648296
  Target Milestone: ---
    Classification: Community



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

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

Description of problem:


In dht_revalidate_cbk, the stbuf returned by the brick is merged into
local->stbuf only if the dir contains a layout. local->stbuf is also used to
compare the stbuf returned in responses to check if an attr heal is needed.

If the newly added brick (which does not contain a layout) is the first one to
respond, its stbuf is not merged into local->stbuf. Subsequent responses from
existing bricks never see a mismatch in the stbuf and self heal is not
triggered.


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 Red Hat Bugzilla Rules Engine on 2018-11-09
05:58:05 EST ---

This bug is automatically being proposed for a Z-stream release of Red Hat
Gluster Storage 3 under active development and open for bug fixes, by setting
the release flag 'rhgs‑3.4.z' to '?'. 

If this bug should be proposed for a different release, please manually change
the proposed release flag.

--- Additional comment from Worker Ant on 2018-11-09 11:59:55 UTC ---

REVIEW: https://review.gluster.org/21611 (cluster/dht: sync brick root perms on
add brick) posted (#1) for review on master by N Balachandran

--- Additional comment from Worker Ant on 2018-11-19 05:45:13 UTC ---

REVIEW: https://review.gluster.org/21611 (cluster/dht: sync brick root perms on
add brick) posted (#5) for review on master by N Balachandran


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1648296
[Bug 1648296] dht_revalidate may not heal attrs on the brick root
https://bugzilla.redhat.com/show_bug.cgi?id=1648298
[Bug 1648298] dht_revalidate may not heal attrs on the brick root
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.