[Gluster-devel] Need inputs for command deprecation output

[Ravishankar N]

On 05/16/2014 07:23 AM, Pranith Kumar Karampuri wrote:
>
> ----- Original Message -----
>> From: "Ravishankar N" <ravishankar at redhat.com>
>> To: "Pranith Kumar Karampuri" <pkarampu at redhat.com>, "Gluster Devel" <gluster-devel at gluster.org>
>> Sent: Friday, May 16, 2014 7:15:58 AM
>> Subject: Re: [Gluster-devel] Need inputs for command deprecation output
>>
>> On 05/16/2014 06:25 AM, Pranith Kumar Karampuri wrote:
>>> Hi,
>>>      As part of changing behaviour of 'volume heal' commands. I want the
>>>      commands to show the following output. Any feedback in making them
>>>      better would be awesome :-).
>>>
>>> root at pranithk-laptop - ~
>>> 06:20:10 :) ⚡ gluster volume heal r2 info healed
>>> This command has been deprecated
>>>
>>> root at pranithk-laptop - ~
>>> 06:20:13 :( ⚡ gluster volume heal r2 info heal-failed
>>> This command has been deprecated
>> When a command is deprecated, it still works the way it did but gives
>> out a warning about it not being maintained and possible alternatives to it.
>> If I understand http://review.gluster.org/#/c/7766/ correctly, we are
>> not supporting these commands any more, in which case the right message
>> would be "Command not supported"
> I am wondering if we should even let the command be sent to self-heal-daemons from glusterd.
>
> How about
> 06:20:10 :) ⚡ gluster volume heal r2 info healed
> Command not supported.
Makes sense; +1
> Instead of
> 06:20:10 :) ⚡ gluster volume heal r2 info healed
> brick: brick-1
> status: Command not supported
>
> brick: brick-2
> status: Command not supported
>
> Pranith
>> -Ravi
>>> Pranith.
>>> _______________________________________________
>>> Gluster-devel mailing list
>>> Gluster-devel at gluster.org
>>> http://supercolony.gluster.org/mailman/listinfo/gluster-devel
>>