[Bugs] [Bug 1511310] Test bug-1483058-replace-brick-quorum-validation.t fails inconsistently

[bugzilla at redhat.com]

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

Atin Mukherjee <amukherj at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |amukherj at redhat.com
          Component|glusterd                    |tests
           Assignee|bugs at gluster.org            |amukherj at redhat.com



--- Comment #3 from Atin Mukherjee <amukherj at redhat.com> ---
This is indeed a bad test. The issue is the attributes to check if a peer is up
and the quorum is regained are different. 

peer_count checks for peerinfo->status which will be set to connected the
moment glusterd receives a RPC_CLNT_CONNECT event from its peer where as the
quorum check is calculated based on if peerinfo->quorum_contrib is set to
QUORUM_UP which is done at glusterd_friend_sm () and that might happen post
RPC_CLNT_CONNECT. In between these two events, if the replace brick commit
force is issued, then the same will fail with quorum rejection. I'll see how to
handle this scenario in the test and will send the patch soon.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.