Alan Protasio
Alan Protasio
I think i did this on the refactor when removing chunks storage.
Humm.. Should we consider having the healthcheck on another port without mtls? I think i agree with @bboreham , i dont like the idea of shipping curl inside the image.
The old or order samples should be a 4xx. You dont have any other error on the distributor logs?
If the error causes a 5xx for the end user (distributor could not write the quorum) cortex prints the error .. if is just one ingester having problem it indeed...
Reopening
I believe this was just not implemented yet? PRs are welcome! :D
Thanks @pstibrany ... But maybe is not the same problem as it still happens on the actual main branch on prometheus. (my repro case is based on the latest changes)
Potential fix for this should be something like this: https://github.com/alanprot/prometheus/commit/fbc420637e52c83cd074bf5d77c4b643f681bf83
Seems that both files (template and md) needs to be included in the PR in order to make the lint happy :D
I think we can close this for now! This code changed quite a bit and we dont see this problem anymore! Thanks!!!