Merge "Log when post is unsuccessful and max retries exceed"
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()) {