[Bugs] [Bug 1322552] Self-heal and manual heal not healing some file

bugzilla at redhat.com bugzilla at redhat.com
Thu Mar 31 15:56:18 UTC 2016


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



--- Comment #3 from Vijay Bellur <vbellur at redhat.com> ---
COMMIT: http://review.gluster.org/13861 committed in release-3.7 by Venky
Shankar (vshankar at redhat.com) 
------
commit 5560961e5188c6b8f4b09d57245513a8f9612ff4
Author: Kotresh HR <khiremat at redhat.com>
Date:   Tue Mar 29 19:06:46 2016 +0530

    features/changelog: Don't modify 'pargfid' in 'resolve_pargfid_to_path'

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

    If 'changelog' is enabled and 'changelog.capture-del-path' option is on
    it calls 'resolve_pargfid_to_path' which modifies 'pargfid' sent by
    caller.  'changelog_unlink' calls this routine directly with
    'loc->pargfid' resulting it being modified and point to root instead of
    actual pargfid. This is a nasty bug and could cause the deletion of
    entry on root directory instead on actual parent when 'loc->path' is
    not present.  Hence this fix to make 'pargfid' a const pointer and
    'resolve_pargfid' to work on copy of pargfid.

    Glusterfind session creation enables these options by default to
    capture deleted entry path in changelog.

    Thanks Pranith for root causing this.

    BUG: 1322552
    Change-Id: I9f2bc44b5604b224462594c12b7d79e68198d693
    Signed-off-by: Kotresh HR <khiremat at redhat.com>
    Reviewed-on: http://review.gluster.org/13861
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Venky Shankar <vshankar at redhat.com>

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are on the CC list for the bug.


More information about the Bugs mailing list