[Gluster-devel] Question on merging zfs snapshot support into the mainline glusterfs

[Rajesh Joseph]

On Tue, Jun 21, 2016 at 4:24 PM, Pranith Kumar Karampuri <
pkarampu at redhat.com> wrote:

> hi,
>       Is there a plan to come up with an interface for snapshot
> functionality? For example, in handling different types of sockets in
> gluster all we need to do is to specify which interface we want to use and
> ib,network-socket,unix-domain sockets all implement the interface. The code
> doesn't have to assume anything about underlying socket type. Do you guys
> think it is a worthwhile effort to separate out the logic of interface and
> the code which uses snapshots? I see quite a few of if (strcmp ("zfs",
> fstype)) code which can all be removed if we do this. Giving btrfs
> snapshots in future will be a breeze as well, this way? All we need to do
> is implementing snapshot interface using btrfs snapshot commands. I am not
> talking about this patch per se. Just wanted to seek your inputs about
> future plans for ease of maintaining the feature.
>

As I said in my previous mail this is in plan and we will be doing it. But
due to other priorities this was not taken in yet.


>
> On Tue, Jun 21, 2016 at 11:46 AM, Atin Mukherjee <amukherj at redhat.com>
> wrote:
>
>>
>>
>> On 06/21/2016 11:41 AM, Rajesh Joseph wrote:
>> > What kind of locking issues you see? If you can provide some more
>> > information I can be able to help you.
>>
>> That's related to stale lock issues on GlusterD which are there in 3.6.1
>> since the fixes landed in the branch post 3.6.1. I have already provided
>> the workaround/way to fix them [1]
>>
>> [1]
>> http://www.gluster.org/pipermail/gluster-users/2016-June/thread.html#26995
>>
>> ~Atin
>> _______________________________________________
>> Gluster-devel mailing list
>> Gluster-devel at gluster.org
>> http://www.gluster.org/mailman/listinfo/gluster-devel
>>
>
>
>
> --
> Pranith
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.gluster.org/pipermail/gluster-devel/attachments/20160622/63008a5d/attachment.html>