Skip to content

Commit

Permalink
Suppress logging of /ready handler errors
Browse files Browse the repository at this point in the history
This is achieved by returning Status Unavailable (which is exempt from
logging in the logging middleware) instead of an Internal Server Error.

Fixes #810
  • Loading branch information
juliusv committed Jun 6, 2018
1 parent f661671 commit fb841c8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/ingester/ingester.go
Original file line number Diff line number Diff line change
Expand Up @@ -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)
}
}

Expand Down

0 comments on commit fb841c8

Please sign in to comment.