[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Gluster-devel] Default behavior of remove brick op
From: |
Justin Clift |
Subject: |
Re: [Gluster-devel] Default behavior of remove brick op |
Date: |
Tue, 18 Mar 2014 10:42:01 +0000 |
On 18/03/2014, at 10:40 AM, Kaushal M wrote:
> IMO, its best if we just remove the default action instead of changing
> its meaning. It is best if force the user to provide an operation for
> the remove-brick command. This way, users using scripts will know that
> something has changed when the script breaks. It is a simple fix if
> the users want to original behavior back, just add commit force as the
> operation.
>
> If we change the default to start, scripts would not break and users
> wouldn't be notified. They'll continue running the script believing
> that the bricks have been forcefully removed, where as it wouldn't be
> the case. This could lead to further problems.
>
> Regarding the deprecation, I suggest we add the deprecation message to
> 3.5 before it ships. We will not be breaking any of the assumed
> functionality for this release, and can safely do the actual change in
> 3.6.
>
> tl;dr, Require an explicit operation for the remove-brick command and
> add the deprecation message to 3.5.
+1
Sounds good to me. :)
+ Justin
--
Open Source and Standards @ Red Hat
twitter.com/realjustinclift