You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This was found on a querier and put it into a tight crashloop, might be nice to catch this and just delete the problem file, or consider logging the error and sleeping for 5min to let someone exec into the pod?
The text was updated successfully, but these errors were encountered:
…rafana#2970)
* Make use of spanlogger when functions that generate spans result in an
error, this is so we consistently have the error=true tag set.
Signed-off-by: Callum Styan <[email protected]>
* Add spanlogger Error to errcheck exclude
Signed-off-by: Callum Styan <[email protected]>
* Review cleanup.
Signed-off-by: Callum Styan <[email protected]>
* Address more review feedback.
Signed-off-by: Callum Styan <[email protected]>
* We need nolint comments on these lines if we want to assign the ctx in
attempt to avoid use of the incorrect ctx in the future.
Signed-off-by: Callum Styan <[email protected]>
* Globally exclude SpanLogger.Error() errcheck
Signed-off-by: Marco Pracucci <[email protected]>
Co-authored-by: Marco Pracucci <[email protected]>
This was found on a querier and put it into a tight crashloop, might be nice to catch this and just delete the problem file, or consider logging the error and sleeping for 5min to let someone exec into the pod?
The text was updated successfully, but these errors were encountered: