Re: replace-brick command modification

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

 



Gaurav,

I think that it is appropriate to keep the commit force options for replace-brick, just to prevent less experienced admins from self-inflicted data loss scenarios.

The add-brick/remove-brick pair of operations is not an intuitive choice for admins who are trying to solve a problem with a specific brick. In this situation, admins are generally thinking 'how can I move the data from this brick to another one', and an admin that is casually surfing documentation might infer that the replace-brick operation is the correct one, rather than a sequence of commands that are somehow magically related.

I believe that keeping the mandatory commit force options for replace-brick will help give these admins reason to pause and re-consider if this is the right command for them to do, and prevent cases where new gluster admins start shouting 'gluster lost my data'.

Regards,
  Kelvin


On 04/02/2015 07:26 AM, Gaurav Garg wrote:
Hi all,

Since GlusterFs version 3.6.0  "gluster volume replace-brick <VOLNAME> <SOURCE-BRICK> <NEW-BRICK> {start [force]|pause|abort|status|commit }" command have deprecated. Only "gluster volume replace-brick <VOLNAME> <SOURCE-BRICK> <NEW-BRICK> commit force" command supported.

for bug https://bugzilla.redhat.com/show_bug.cgi?id=1094119 , Patch http://review.gluster.org/#/c/10101/   is removing cli/glusterd code for  "gluster volume replace-brick <VOLNAME> <BRICK> <NEW-BRICK> {start [force]|pause|abort|status|commit }" command. so only we have commit force option supported for replace-brick command.

Should we have new command "gluster volume replace-brick <VOLNAME> <SOURCE-BRICK> <NEW-BRICK>" instead of having "gluster volume replace-brick <VOLNAME> <SOURCE-BRICK> <NEW-BRICK> commit force" command.


Thanks & Regards
Gaurav Garg
_______________________________________________
Gluster-users mailing list
Gluster-users@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-users


Attachment: smime.p7s
Description: S/MIME Cryptographic Signature

_______________________________________________
Gluster-users mailing list
Gluster-users@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-users

[Index of Archives]     [Gluster Development]     [Linux Filesytems Development]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux