-
Notifications
You must be signed in to change notification settings - Fork 4.6k
v1.18: Adds more info to panic message in AccountsHashVerifier (backport of #35353) #35359
Conversation
(cherry picked from commit 6aaaf85)
Justification for backporting: So far this issue has been difficult to debug, as it often depends on the state of the machine (and filesystem) when the panic occurs. Without this backport, then we are effectively saying that we are OK with v1.18 not aiding in debugging. And then likely we would not backport a fix either (because if we aren't willing to backport debug info, why would we suddenly be OK with a more invasive fix). |
Review requests:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm good with the BP; the changes have a very narrow scope and will provide additional insight to debug a panic that we're seeing intermittently
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved for beta
@sakridge Can I get your review on this backport? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
This is an automatic backport of pull request #35353 done by Mergify.
Mergify commands and options
More conditions and actions can be found in the documentation.
You can also trigger Mergify actions by commenting on this pull request:
@Mergifyio refresh
will re-evaluate the rules@Mergifyio rebase
will rebase this PR on its base branch@Mergifyio update
will merge the base branch into this PR@Mergifyio backport <destination>
will backport this PR on<destination>
branchAdditionally, on Mergify dashboard you can:
Finally, you can contact us on https://mergify.com