-
Notifications
You must be signed in to change notification settings - Fork 810
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
compactor went into tailspin #3914
Comments
Isn't it a temporarily error from which will eventually recover once consul gets back online? |
Well it stayed like this ~12 hours and recovered when I restarted compactor. |
I edited the problem description to show the end of the logfile. |
My bad. I thought the error occurred while checking if a tenant is owned by the compactor replica. It actually happened here: cortex/pkg/compactor/compactor.go Lines 391 to 394 in a0c89bd
I agree, we should have a timeout. |
While I'm here, that message "value is nil" doesn't provide much "info". |
This issue has been automatically marked as stale because it has not had any activity in the past 60 days. It will be closed in 15 days if no further activity occurs. Thank you for your contributions. |
Still valid and we actually have a PR open. |
Describe the bug
Consul was down at the time compactor started, and it never recovered:
We have compactor sharding turned on:
Expected behavior
I think it should exit with error in this situation; crashlooping would make the fault more obvious to the operator, and after a few restarts it would have managed to talk to Consul in my case.
The text was updated successfully, but these errors were encountered: