fix: use safer condition

Without the previous fix, Retry.Max == 0 would potentially loop
on retryable errors. This is bad so even though it shouldn't happen
now. Make the condition safer.

Signed-off-by: Mark Hindess <mark.hindess@gmail.com>
Signed-off-by: Ioan Zicu <ioan.zicu@nokia.com>

admin.go

@@ -210,7 +210,7 @@ func (ca *clusterAdmin) retryOnError(retryable func(error) bool, fn func() error
 	for attemptsRemaining := ca.conf.Admin.Retry.Max + 1; ; {
 		err := fn()
 		attemptsRemaining--
-		if err == nil || attemptsRemaining == 0 || !retryable(err) {
+		if err == nil || attemptsRemaining <= 0 || !retryable(err) {
 			return err
 		}
 		Logger.Printf(