From fb841c8c0f5cf31477c3429d6f49766f2921172b Mon Sep 17 00:00:00 2001 From: Julius Volz Date: Wed, 6 Jun 2018 12:41:57 +0200 Subject: [PATCH] Suppress logging of /ready handler errors This is achieved by returning Status Unavailable (which is exempt from logging in the logging middleware) instead of an Internal Server Error. Fixes https://github.com/weaveworks/cortex/issues/810 --- pkg/ingester/ingester.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/ingester/ingester.go b/pkg/ingester/ingester.go index 86a7eaa87f..f7e4d85929 100644 --- a/pkg/ingester/ingester.go +++ b/pkg/ingester/ingester.go @@ -493,7 +493,7 @@ func (i *Ingester) ReadinessHandler(w http.ResponseWriter, r *http.Request) { if i.lifecycler.IsReady() { w.WriteHeader(http.StatusNoContent) } else { - w.WriteHeader(http.StatusInternalServerError) + w.WriteHeader(http.StatusServiceUnavailable) } }