diff options
| author | Richard Wareing <rwareing@fb.com> | 2016-04-01 20:42:50 -0700 |
|---|---|---|
| committer | Kevin Vigor <kvigor@fb.com> | 2016-12-29 11:15:39 -0800 |
| commit | 02f8b7300bc635dea9ae1fee6ef14c0d4725591a (patch) | |
| tree | 3d1b55756f09cbcd6763d87ad1c26ec3b7a5ddf9 /libglusterfs/src/common-utils.c | |
| parent | 141e879d83da3e6fa2f3891d8aa1c7895017c401 (diff) | |
cluster/afr: Hybrid mounts must honor _marked_ up/down states
Summary:
- Qsorted latencies must also take into account that a host can be
marked down yet still have a (slightly) better latency. This can
happen if max-replicas is reached, and there are multiple bricks which
meet the halo threshold. In such a case we must prefer the brick
which is marked up albeit with a perhaps slightly higher latency.
It's not worth doing any sort of swap in this case as it's going to be
jarring to the cluster and introduce flapping behavior.
Test Plan: - Halo prove tests @ https://phabricator.fb.com/P56251020
Reviewers: sshreyas, kvigor
Reviewed By: kvigor
Blame Revision: Change-Id: I858bbf44638a4978093dc56d4a98c96be4f8b45e
Change-Id: Ie3b79700ec63398bc30e7bcf75fb05931dd5d0d0
FB-commit-id: deb2f35db6f8b979857b4e3779b2a41c59a2e416
Signed-off-by: Kevin Vigor <kvigor@fb.com>
Reviewed-on: http://review.gluster.org/16307
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Shreyas Siravara <sshreyas@fb.com>
Smoke: Gluster Build System <jenkins@build.gluster.org>
Diffstat (limited to 'libglusterfs/src/common-utils.c')
0 files changed, 0 insertions, 0 deletions