[Bugs] [Bug 1335728] [geo-rep]: Multiple geo-rep session to the same slave is allowed for different users

[bugzilla at redhat.com]

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



--- Comment #2 from Vijay Bellur <vbellur at redhat.com> ---
COMMIT: http://review.gluster.org/14322 committed in release-3.7 by Aravinda VK
(avishwan at redhat.com) 
------
commit 70192bfe5f7f956843d094ec9cb484b23ce45556
Author: Saravanakumar Arumugam <sarumuga at redhat.com>
Date:   Tue Dec 29 19:22:36 2015 +0530

    glusterd/geo-rep: slave volume uuid to identify a geo-rep session

    Problem:
    Currently, it is possible to create multiple geo-rep session from
    the Master host to Slave host(s), where Slave host(s) belonging
    to the same volume.

    For example:
    Consider Master Host M1 having volume tv1 and Slave volume tv2,
    which spans across two Slave hosts S1 and S2.
    Currently, it is possible to create geo-rep session from
    M1(tv1) to S1(tv2) as well as from M1(tv1) to S2(tv2).

    When the Slave Host is alone modified, it is identified as a new geo-rep
    session (as slave host and slave volume together are identifying
    Slave side).

    Also, it is possible to create both root and non-root geo-rep session
between
    same Master volume and Slave volume. This should also be avoided.

    Solution:
    This multiple geo-rep session creation must be avoided and
    in order to avoid, use Slave volume uuid to identify a Slave.
    This way, we can identify whether a session is already created for
    the same Slave volume and avoid creating again (using different host).

    When the session creation is forced in the above scenario, rename
    the existing geo-rep session directory with new Slave Host mentioned.

    Change-Id: I9239759cbc0d15dad63c48b8cf62950bb687c7c8
    BUG: 1335728
    Signed-off-by: Saravanakumar Arumugam <sarumuga at redhat.com>
    Signed-off-by: Aravinda VK <avishwan at redhat.com>
    Reviewed-on: http://review.gluster.org/13111
    Reviewed-by: Kotresh HR <khiremat at redhat.com>
    Tested-by: Kotresh HR <khiremat at redhat.com>
    Reviewed-by: Atin Mukherjee <amukherj at redhat.com>
     (cherry picked from commit a9128cda34b1f696b717ba09fa0ac5a929be8969)
    Reviewed-on: http://review.gluster.org/14322
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=2kCnmlHraC&a=cc_unsubscribe