[Bugs] [Bug 1473132] dht/cluster: rebalance/ remove-brick should honor min-free-disk

[bugzilla at redhat.com]

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



--- Comment #4 from Worker Ant <bugzilla-bot at gluster.org> ---
COMMIT: https://review.gluster.org/17831 committed in release-3.10 by
Shyamsundar Ranganathan (srangana at redhat.com) 
------
commit c90bffe08466baba369a23964a68b9ac59eb0394
Author: Susant Palai <spalai at redhat.com>
Date:   Tue Apr 11 17:27:17 2017 +0530

    cluster/dht: Make rebalance honor min-free-disk

    test:  Manual

    created files of size 1K on 2 brick(of size 1GB) setup .
    added a brick of size 16GB.
    set min-free-disk to 12GB(so that first two bricks won't receive any
files).
    removed one of the 1st brick of size 1GB.

    Logs from test:
    [2017-04-12 08:52:08.196484] W [MSGID: 0]
[dht-rebalance.c:895:__dht_check_free_space]
     0-test1-dht: Write will cross min-free-disk for file - /tile32 on subvol -
test1-client-1.
    Looking for new subvol.

    [2017-04-12 08:52:08.196904] I [MSGID: 0]
[dht-rebalance.c:925:__dht_check_free_space]
    0-test1-dht: new target found - test1-client-2 for file - /tile32

     - Post migration we have two files. The new destination (/brick/1) has the
data file
    [root at vm1 ~]# ll /brick/1/tile32
    -rw-r--r--. 2 root root 0 Apr 12 14:22 /brick/1/tile32

     - On the old target the linkto file is there with linkto xattr pointing to
/brick/1
    [root at vm1 ~]# ll /tmp/2/tile32
    ---------T. 2 root root 1000 Apr 12 14:22 /tmp/2/tile32
    [root at vm1 ~]# getfattr -m . -de text /tmp/2/tile32
    getfattr: Removing leading '/' from absolute path names
    security.selinux="unconfined_u:object_r:user_tmp_t:s0"
    trusted.gfid="����:Aс�#�/'b2"
    trusted.glusterfs.dht.linkto="test1-client-2"

    Marking ./tests/features/worm_sh.t as bad test.
    Reason being, this patch failed on master branch as well and it has nothing
    to do with rebalance/remove-brick.

    > BUG: 1441508
    > Change-Id: I90bae251cda3d957a49cdceda90cd08311a392fb
    > Signed-off-by: Susant Palai <spalai at redhat.com>
    > Reviewed-on: https://review.gluster.org/17034
    > Smoke: Gluster Build System <jenkins at build.gluster.org>
    > NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    > Reviewed-by: Amar Tumballi <amarts at redhat.com>
    > Reviewed-by: Raghavendra G <rgowdapp at redhat.com>
    > CentOS-regression: Gluster Build System <jenkins at build.gluster.org>
    > Signed-off-by: Susant Palai <spalai at redhat.com>

    Change-Id: I90bae251cda3d957a49cdceda90cd08311a392fb
    BUG: 1473132
    Signed-off-by: Susant Palai <spalai at redhat.com>
    Reviewed-on: https://review.gluster.org/17831
    CentOS-regression: Gluster Build System <jenkins at build.gluster.org>
    Smoke: Gluster Build System <jenkins at build.gluster.org>
    Reviewed-by: Shyamsundar Ranganathan <srangana at redhat.com>

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