[Bugs] [Bug 1475282] [Remove-brick] Few files are getting migrated eventhough the bricks crossed cluster.min-free-disk value

[bugzilla at redhat.com]

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



--- Comment #7 from Worker Ant <bugzilla-bot at gluster.org> ---
COMMIT: https://review.gluster.org/17876 committed in master by Raghavendra G
(rgowdapp at redhat.com) 
------
commit c7cc5049fc8965836926a6f1b3a3270ad1aa566d
Author: Susant Palai <spalai at redhat.com>
Date:   Wed Jul 26 17:12:03 2017 +0530

    cluster/dht: rebalance min-free-disk fix

    To calculate available space on a subvolume we used to do
    the following in __dht_check_free_space.

    post_availspace = (dst_statfs.f_bavail * dst_statfs.f_frsize) -
stbuf->ia_size

    Now to subtracting the file size from available space is tricky here.
    Sometime available space will be lesser than the file size and since all
the
    participating members in calculation are unsigned int, the result is a
large
    number (integer overflow).

    Solution: We do not need to subtract the file size from the space
available,
    since fallocate would have reserved file size space already.

    Change-Id: I4f724358c44b9911933742ff3ff8d55b3dfda1cb
    BUG: 1475282
    Signed-off-by: Susant Palai <spalai at redhat.com>
    Reviewed-on: https://review.gluster.org/17876
    Smoke: Gluster Build System <jenkins at build.gluster.org>
    Reviewed-by: Raghavendra G <rgowdapp at redhat.com>
    Reviewed-by: N Balachandran <nbalacha at redhat.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.org>

-- 
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=7baqeBEuUr&a=cc_unsubscribe