Bug 1428080 - Fixes halo multi-region fail-over regression
Summary: Fixes halo multi-region fail-over regression
Keywords:
Status: CLOSED DEFERRED
Alias: None
Product: GlusterFS
Classification: Community
Component: replicate
Version: mainline
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-03-01 19:06 UTC by Vijay Bellur
Modified: 2019-03-08 04:40 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-03-08 04:40:48 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Vijay Bellur 2017-03-01 19:06:16 UTC
Fixes halo multi-region fail-over regression

Summary:
- After the afr-common.c refactor the quorum accounting broke, this diff
  fixes things so when an alternate brick is swapped in to provide the
  min-replicas quorum accounting works correctly and the FS doesn't go
  RO.  In short, the magic which makes halo clusters seamlessly fail-over
  to a remote region for writes broke :).

Test Plan:
prove -v tests/basic/halo-failover.t ->
https://phabricator.fb.com/P12179467

Reviewers: dph, jackl, cjh

Reviewed By: cjh

Subscribers: meyering

Differential Revision: https://phabricator.fb.com/D1390670

Tasks: 4117827

Change-Id: I2d7fb8ca1e80cd1b21cc12b11b0a3db812321080
Signed-off-by: Kevin Vigor <kvigor>
Reviewed-on: http://review.gluster.org/16203
NetBSD-regression: NetBSD Build System <jenkins.org>
Smoke: Gluster Build System <jenkins.org>
CentOS-regression: Gluster Build System <jenkins.org>
Reviewed-by: Shreyas Siravara <sshreyas>

Comment 1 Ravishankar N 2019-03-08 04:40:48 UTC
Basic halo replication functionality is present in master.There are no immediate plans for forward porting any pending halo related patches from the release-3.8-fb branch into master, hence closing the bug. Please feel free to re-open if there is a need for re-assessment.


Note You need to log in before you can comment on or make changes to this bug.