Re: Need inputs for command deprecation output

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

 



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

----- Original Message -----
From: "Ravishankar N" <ravishankar@xxxxxxxxxx>
To: "Pranith Kumar Karampuri" <pkarampu@xxxxxxxxxx>, "Gluster Devel" <gluster-devel@xxxxxxxxxxx>
Sent: Friday, May 16, 2014 7:15:58 AM
Subject: Re:  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@xxxxxxxxxxx
http://supercolony.gluster.org/mailman/listinfo/gluster-devel


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





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

  Powered by Linux