gluster-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Gluster-devel] Fwd: Snapshot feature design review


From: James
Subject: Re: [Gluster-devel] Fwd: Snapshot feature design review
Date: Wed, 18 Dec 2013 23:01:36 -0500

Just saw this now. Haven't had to review in depth. One question:

Will all the commands have --xml options like the normal Gluster commands?
It is very useful to have these stable interfaces.

Cheers,
James


On Wed, Dec 18, 2013 at 10:58 PM, Vijay Bellur <address@hidden> wrote:
> On 10/17/2013 09:52 PM, Nagaprasad Sathyanarayana wrote:
>>
>>
>>>>
>>>> Hi All,
>>>>
>>>> The design document has been updated, and we have tried to address
>>>> all the review comments and design issues to the best of our ability.
>>>>
>>>> Please review the design and the document when possible.
>>>>
>>>> The design document can be found @
>>>> https://forge.gluster.org/snapshot/pages/Home
>>>>
>>>> Please feel free to critique/comment.
>
>
>
> Belated feedback on the CLI. CLI does seem to have a lot of hyphen/dash
> options. This is something that we have avoided in gluster so far.
> Conforming to the same design principle would be nice to have.
>
> The only reason why a hyphenated option seems necessary is due to the
> consistency groups. It would be better to treat consistency-group as a first
> class entity and have a separate mechanism to define that. For e.g.:
>
> #gluster consistency-group create <name>
>
> #gluster consistency-group add <vol1,vol2,...volN>
>
> #gluster consistency-group del <vol1...volN>
>
>
> Once this is accomplished, we can avoid using the hyphenated options and
> distinguish between single volume snapshots and consistency-group snapshots
> by using a keyword for consistency groups.
>
> For e.g., creation of a snapshot can be structured as:
>
> #gluster snapshot create [consistency-group] <name> [<description>]
>
> Other snapshot commands can be structured on similar lines.
>
> Thoughts?
>
> Thanks,
> Vijay
>
> _______________________________________________
> Gluster-devel mailing list
> address@hidden
> https://lists.nongnu.org/mailman/listinfo/gluster-devel



reply via email to

[Prev in Thread] Current Thread [Next in Thread]