<p dir="ltr"></p>
<p dir="ltr">-Atin<br>
Sent from one plus one<br>
On Jul 17, 2015 5:32 PM, "Manikandan Selvaganesh" <<a href="mailto:mselvaga@redhat.com">mselvaga@redhat.com</a>> wrote:<br>
><br>
><br>
><br>
> --<br>
> Regards,<br>
> Manikandan Selvaganesh.<br>
><br>
> ----- Forwarded Message -----<br>
> From: Manikandan Selvaganesh <<a href="mailto:mselvaga@redhat.com">mselvaga@redhat.com</a>><br>
> To: <a href="mailto:gluster-devel@gluster.org">gluster-devel@gluster.org</a><br>
> Sent: Fri, 17 Jul 2015 05:58:25 -0400 (EDT)<br>
> Subject: "gluster peer probe" on localhost should fail with appropriate value<br>
><br>
> Hi,<br>
><br>
> Currently, the command "gluster peer probe $localhost" shows "peer probe: success. Probe on localhost not needed" with a ret value of 0.<br>
><br>
> 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.<br>
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.<br>
><br>
> 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)?<br>
><br>
> 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".<br>
><br>
> [1] <a href="http://review.gluster.org/#/c/11704/">http://review.gluster.org/#/c/11704/</a><br>
><br>
> Any suggestions would be helpful!!<br>
><br>
><br>
> --<br>
> Thanks & regards,<br>
> Manikandan Selvaganesh.<br>
><br>
><br>
> _______________________________________________<br>
> Gluster-devel mailing list<br>
> <a href="mailto:Gluster-devel@gluster.org">Gluster-devel@gluster.org</a><br>
> <a href="http://www.gluster.org/mailman/listinfo/gluster-devel">http://www.gluster.org/mailman/listinfo/gluster-devel</a><br>
</p>