commit dc18cb665eb452d71602e1a980d7669a67265dfc
author Hugo Arès <hugo.ares@ericsson.com> Fri Apr 28 09:42:49 2017 +0200
committer Hugo Arès <hugo.ares@ericsson.com> Fri Apr 28 09:42:49 2017 +0200
tree 1fe9dbc18cfc45f2fa1d5dd7290cb163a9b2a63f
parent 47c2691a109ffedf8737f3697a80eef82dd5a4a9

Fix replication retries when maxRetries is set to 0

maxRetries defaults to 0 which means retry indefinitely but it was not
retrying.

Change-Id: I6c8d8d6b1511d67b647f25193dcfe860687eea76

src/main/java/com/googlesource/gerrit/plugins/replication/PushOne.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/replication/PushOne.java b/src/main/java/com/googlesource/gerrit/plugins/replication/PushOne.java
index 473ebc5..525c990 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/replication/PushOne.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/replication/PushOne.java
@@ -201,7 +201,7 @@
   boolean setToRetry() {
     retrying = true;
     retryCount++;
-    return retryCount <= maxRetries;
+    return maxRetries == 0 || retryCount <= maxRetries;
   }
 
   void canceledByReplication() {