diff options
author | Atin Mukherjee <amukherj@redhat.com> | 2018-01-22 09:49:34 +0530 |
---|---|---|
committer | Atin Mukherjee <amukherj@redhat.com> | 2018-01-25 08:01:22 +0000 |
commit | 9d708a3739c8201d23f996c413d6b08f8b13dd90 (patch) | |
tree | 548bee3ac7b786d5d131e346dad875717fbde7a0 /api | |
parent | 7631b22a5960363518ad13da8661c76a1ffec253 (diff) | |
download | glusterfs-9d708a3739c8201d23f996c413d6b08f8b13dd90.tar.gz glusterfs-9d708a3739c8201d23f996c413d6b08f8b13dd90.tar.xz glusterfs-9d708a3739c8201d23f996c413d6b08f8b13dd90.zip |
glusterd: process pmap sign in only when port is marked as free
Because of some crazy race in volume start code path because of friend
handshaking with volumes with quorum enabled we might end up into a situation
where glusterd would start a brick and get a disconnect and then immediately try
to start the same brick instance based on another friend update request. And
then if for the very first brick even if the process doesn't come up at the end
sign in event gets sent and we end up having two duplicate portmap entries for
the same brick. Since in brick start we mark the previous port as free, its
better to consider a sign in request as no op if the corresponding port type is
marked as free.
Change-Id: I995c348c7b6988956d24b06bf3f09ab64280fc32
BUG: 1537362
Signed-off-by: Atin Mukherjee <amukherj@redhat.com>
Diffstat (limited to 'api')
0 files changed, 0 insertions, 0 deletions