-
Notifications
You must be signed in to change notification settings - Fork 619
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
fix: throw correct error instead of InternalError #24680
fix: throw correct error instead of InternalError #24680
Conversation
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.
Thanks! I'd suggest to backport it too, just to be safe.
Created backport PR for
Please cherry-pick the changes locally and resolve any conflicts. git fetch origin backport-24680-to-stable/8.3
git worktree add --checkout .worktree/backport-24680-to-stable/8.3 backport-24680-to-stable/8.3
cd .worktree/backport-24680-to-stable/8.3
git reset --hard HEAD^
git cherry-pick -x b5d6ffdca4107fc3bd273dbe0ac067b42c547aa7
git push --force-with-lease |
Created backport PR for
Please cherry-pick the changes locally and resolve any conflicts. git fetch origin backport-24680-to-stable/8.4
git worktree add --checkout .worktree/backport-24680-to-stable/8.4 backport-24680-to-stable/8.4
cd .worktree/backport-24680-to-stable/8.4
git reset --hard HEAD^
git cherry-pick -x b5d6ffdca4107fc3bd273dbe0ac067b42c547aa7
git push --force-with-lease |
Successfully created backport PR for |
Successfully created backport PR for |
…or (#24689) # Description Backport of #24680 to `stable/8.6`. relates to original author: @entangled90
…or (#24688) # Description Backport of #24680 to `stable/8.5`. relates to original author: @entangled90
Description
InternalError
was thrown by mistake (which extendsVirtualMachineError
and could case the VM to terminate.Correct error is now thrown