Log when post is unsuccessful and max retries exceed Change-Id: Ic977db13a73cf89d8f086fa32466d046e23f2b13
diff --git a/src/main/java/com/googlesource/gerrit/plugins/webhooks/PostTask.java b/src/main/java/com/googlesource/gerrit/plugins/webhooks/PostTask.java index c069e3a..844f87a 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/webhooks/PostTask.java +++ b/src/main/java/com/googlesource/gerrit/plugins/webhooks/PostTask.java
@@ -79,9 +79,15 @@ execCnt++; HttpResult result = session.get().post(remote, content.get()); - if (!result.successful && execCnt < remote.getMaxTries()) { - logRetry(result.message); - reschedule(); + if (!result.successful) { + if (execCnt < remote.getMaxTries()) { + logRetry(result.message); + reschedule(); + } else { + log.atSevere().log( + "Failed to post: %s. Exceeded max retries(%d). Reason: %s", + this, remote.getMaxTries(), result.message); + } } } catch (Throwable e) { if (isRecoverable(e) && execCnt < remote.getMaxTries()) {