| commit | 29c98b38f5b1f82806773fa9437974095b836e65 | [log] [tgz] |
|---|---|---|
| author | Luca Milanesio <luca.milanesio@gmail.com> | Mon Mar 18 14:40:55 2019 +0000 |
| committer | Luca Milanesio <luca.milanesio@gmail.com> | Mon Mar 18 16:03:55 2019 +0000 |
| tree | 01fbe6a99c3be910629f315d0c09d5fc4edc7f97 | |
| parent | 075144dc105cfe9cb5fc298efd8f8d1f5eed3670 [diff] |
Revert "HC returns 500 only with TIMEOUT or FAILED" This reverts commit 075144dc105cfe9cb5fc298efd8f8d1f5eed3670. Reason for revert: Problem was fixed already on stable-2.15. Merging that branch instead. Change-Id: I17fbe4ccb1c8ec1bfd8ad64463ce6a6425be9dd5
diff --git a/src/main/java/com/googlesource/gerrit/plugins/healthcheck/api/HealthCheckStatusEndpoint.java b/src/main/java/com/googlesource/gerrit/plugins/healthcheck/api/HealthCheckStatusEndpoint.java index c508222..26a2042 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/healthcheck/api/HealthCheckStatusEndpoint.java +++ b/src/main/java/com/googlesource/gerrit/plugins/healthcheck/api/HealthCheckStatusEndpoint.java
@@ -51,7 +51,7 @@ .filter( res -> res instanceof HealthCheck.StatusSummary - && ((HealthCheck.StatusSummary) res).isFailure()) + && ((HealthCheck.StatusSummary) res).result != HealthCheck.Result.PASSED) .findFirst() .isPresent()) { return HttpServletResponse.SC_INTERNAL_SERVER_ERROR;