diff options
| author | Ravishankar N <ravishankar@redhat.com> | 2015-08-07 10:10:54 +0530 |
|---|---|---|
| committer | Niels de Vos <ndevos@redhat.com> | 2015-08-07 04:49:57 -0700 |
| commit | 2a3f0fb035cf677d058a920d2fc1931a18e565ec (patch) | |
| tree | 2f5740cacf7ac2aaacda5bcb57a087e03cc0d8e5 /xlators/cluster/afr/src/afr-self-heal-data.c | |
| parent | 7255febab2c38cc89b71f2519a20d10f53586000 (diff) | |
afr: Do not wind statfs to arbiter brick
Problem: AFR serves statfs from the brick having the least free space
available. Since the size to be allocated to the arbiter brick in a 3
way replica is supposed to be considerably lesser than the other 2
bricks, statfs will be served from this brick which is incorrect.
Fix: Don't serve statfs from the arbiter brick.
Change-Id: I5af098b9c50626f52cf3d7dbb060bf754c797f05
BUG: 1251346
Reported-by: Fredrik Brandt <fredrikb@denlillaplaneten.se>
Signed-off-by: Ravishankar N <ravishankar@redhat.com>
Reviewed-on: http://review.gluster.org/11857
Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
Diffstat (limited to 'xlators/cluster/afr/src/afr-self-heal-data.c')
| -rw-r--r-- | xlators/cluster/afr/src/afr-self-heal-data.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/xlators/cluster/afr/src/afr-self-heal-data.c b/xlators/cluster/afr/src/afr-self-heal-data.c index 37e268b1586..32fcd1a2f2b 100644 --- a/xlators/cluster/afr/src/afr-self-heal-data.c +++ b/xlators/cluster/afr/src/afr-self-heal-data.c @@ -680,7 +680,7 @@ __afr_selfheal_data (call_frame_t *frame, xlator_t *this, fd_t *fd, source = ret; - if (priv->arbiter_count == 1 && source == ARBITER_BRICK_INDEX && + if (AFR_IS_ARBITER_BRICK(priv, source) && AFR_COUNT (sources, priv->child_count) == 1) { did_sh = _gf_false; goto unlock; |