[Bugs] [Bug 1231789] Not able to create snapshots for geo-replicated volumes when session is created with root user

[bugzilla at redhat.com]

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



--- Comment #3 from Anand Avati <aavati at redhat.com> ---
COMMIT: http://review.gluster.org/11233 committed in master by Krishnan
Parthasarathi (kparthas at redhat.com) 
------
commit daf22a3b59f42c897f345dab74c3852ddf9bddef
Author: Kotresh HR <khiremat at redhat.com>
Date:   Mon Jun 15 17:20:46 2015 +0530

    glusterd: Fix snapshot of a volume with geo-rep

    Problem:
        Snapshot fails for a volume configured with geo-rep
        if geo-rep is created with root user with the following
        syntax.
        gluster vol geo-rep <master_vol> root@<slave_host>::<slave_vol>

        It works fine if created with following syntax.
        gluster vol geo-rep <master_vol> <slave_host>::<slave_vol>

    Cause:
        Geo-rep maintains persistent dictionary of slave associated
        with master volume. The dictionary saves the slave info
        along with 'root@' as sent from cli.
        Snapshot while constructing the working dir path to copy
        configuration files, constructs using this dictionary.
        But the actual working dir is created with out considering
        'root@'. Hence the issue.

    Fix:
        Fix is done at two layers.
        1. Parse and negelect 'root@' in cli itself.
        2. For existing geo-rep sessions and upgrade scenarios,
           parse and neglect 'root@' in snapshot code as well.

    Change-Id: If4e04f7f776ef71df4dd1e7e053ef75db98762b2
    BUG: 1231789
    Signed-off-by: Kotresh HR <khiremat at redhat.com>
    Reviewed-on: http://review.gluster.org/11233
    Reviewed-by: Avra Sengupta <asengupt at redhat.com>
    Tested-by: Gluster Build System <jenkins at build.gluster.com>
    Tested-by: NetBSD Build System <jenkins at build.gluster.org>
    Reviewed-by: Krishnan Parthasarathi <kparthas at redhat.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=f5pZBTRh9l&a=cc_unsubscribe