[Gluster-devel] [PATCH BUG:3085] Make backupvolfile-server option actually work.
Jeff Darcy
jdarcy at redhat.com
Mon Jun 27 15:20:59 UTC 2011
On 06/27/2011 11:16 AM, Devon Miller wrote:
> I was thinking more of the case where no backup server is
> configured. If the primary server fails, and a backup is not defined,
> then found=0 and err=0. So, the script will eventually exit with a 0
> when it really should have exited with a 1.
You're right, the code doesn't fix that (pre-existing) hole. We should
really set err=1 before we check for the definition of a backup server.
Thanks!
More information about the Gluster-devel
mailing list