Re: Fwd: "gluster peer probe" on localhost should fail with appropriate value

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

 



-Atin
Sent from one plus one
On Jul 17, 2015 5:32 PM, "Manikandan Selvaganesh" <mselvaga@xxxxxxxxxx> wrote:
>
>
>
> --
> Regards,
> Manikandan Selvaganesh.
>
> ----- Forwarded Message -----
> From: Manikandan Selvaganesh <mselvaga@xxxxxxxxxx>
> To: gluster-devel@xxxxxxxxxxx
> Sent: Fri, 17 Jul 2015 05:58:25 -0400 (EDT)
> Subject: "gluster peer probe" on localhost should fail with appropriate value
>
> Hi,
>
> Currently, the command "gluster peer probe $localhost" shows "peer probe: success. Probe on localhost not needed" with a ret value of 0.
>
> As per the discussion with Satheesaran(QE), since, we are showing a warning message, "Probe on localhost not needed", so, would it be better if we show "peer probe: failure: Probe on localhost not needed" with a ret value of -1.
My only concern on the patch is are we breaking any automation or not. I think it makes sense to forward this query to gluster-user mailing list and get the feedback from the users.
>
> In my opinion, it would be better to change if no script is dependent on this ret value. Are there any automation scripts or other functions which uses this ret value or is there any reason to not return an error(ret = -1)?
>
> I have sent a patch[1] changing the ret value to -1 if its a local host and "gluster peer probe $localhost" would show "peer probe: failed: Probe on localhost not needed".
>
> [1] http://review.gluster.org/#/c/11704/
>
> Any suggestions would be helpful!!
>
>
> --
> Thanks & regards,
> Manikandan Selvaganesh.
>
>
> _______________________________________________
> Gluster-devel mailing list
> Gluster-devel@xxxxxxxxxxx
> http://www.gluster.org/mailman/listinfo/gluster-devel

_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://www.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