On 05/16/2014 07:23 AM, Pranith Kumar Karampuri wrote:

----- Original Message -----
From: "Ravishankar N" <ravishan...@redhat.com>
To: "Pranith Kumar Karampuri" <pkara...@redhat.com>, "Gluster Devel" 
<gluster-devel@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@pranithk-laptop - ~
06:20:10 :) ⚡ gluster volume heal r2 info healed
This command has been deprecated

root@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@gluster.org
http://supercolony.gluster.org/mailman/listinfo/gluster-devel


_______________________________________________
Gluster-devel mailing list
Gluster-devel@gluster.org
http://supercolony.gluster.org/mailman/listinfo/gluster-devel

Reply via email to