On Sat, May 17, 2014 at 03:45:51AM -0400, Pranith Kumar Karampuri wrote: > > > ----- Original Message ----- > > From: "Niels de Vos" <ndevos@xxxxxxxxxx> > > To: "Vijay Bellur" <vbellur@xxxxxxxxxx> > > Cc: "Pranith Kumar Karampuri" <pkarampu@xxxxxxxxxx>, "Ravishankar N" <ravishankar@xxxxxxxxxx>, "Gluster Devel" > > <gluster-devel@xxxxxxxxxxx> > > Sent: Friday, May 16, 2014 7:11:45 PM > > Subject: Re: Need inputs for command deprecation output > > > > On Fri, May 16, 2014 at 05:35:06PM +0530, Vijay Bellur wrote: > > > 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. > > > > > > > > > > Since we no longer intend supporting this command, it might be > > > better to withdraw the command from CLI and have documentation > > > reflect the possible alternatives for this command. > > > > I'd like to see an error message that the command has been replaced. The > > message should point to 'gluster volume help $whatever' or the man-page > > in case it has been updated for the new command. Accessing online > > documentation on the internet is not always possible, so try to restrict > > to resources available on the local system. > > How about giving the following message: > 06:20:10 :) ⚡ gluster volume heal r2 info healed > Command not supported. Please use "gluster volume heal <volname> info" and logs to find the information. Yes, something like that works should be fine, users are not completely left wondering what to do next :) > > In addition, a non-zero exit code should be returned. This will help > > authors/users of scripts (in case they exist) to detect unexpected > > behavior. > > Yes this is already taken care. I am just concerned about the message > user sees. Okay, great! _______________________________________________ Gluster-devel mailing list Gluster-devel@xxxxxxxxxxx http://supercolony.gluster.org/mailman/listinfo/gluster-devel