-
Notifications
You must be signed in to change notification settings - Fork 119
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
Suspend MantaPay when InternalLedgerError #977
Suspend MantaPay when InternalLedgerError #977
Conversation
Signed-off-by: Georgi Zlatarev <georgi.zlatarev@manta.network>
Signed-off-by: Georgi Zlatarev <georgi.zlatarev@manta.network>
Signed-off-by: Georgi Zlatarev <georgi.zlatarev@manta.network>
Signed-off-by: Georgi Zlatarev <georgi.zlatarev@manta.network>
…xtra unit case and comments. Signed-off-by: Georgi Zlatarev <georgi.zlatarev@manta.network>
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.
Nice, LGTM
I forget or don't have the context. why do we suspend mantapay if this error happen? |
If these errors happen, the ledger is unrecoverable broken, so we should just shut it down. |
Signed-off-by: Georgi Zlatarev <georgi.zlatarev@manta.network>
Signed-off-by: Georgi Zlatarev <georgi.zlatarev@manta.network>
7699700
Signed-off-by: Adam Reif <Garandor@manta.network>
7f368d1
to
775be42
Compare
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.
A few more tests that make sure errors that shouldn't suspend mantapay indeed do not maybe useful too. Code LGTM though
Each invocation of |
Signed-off-by: Georgi Zlatarev georgi.zlatarev@manta.network
Description
The following errors mean that the ledger is uncrecovarble broken:
ReceiverLedgerError::ChecksumError
ReceiverLedgerError::MerkleTreeCapacityError
TransferLedgerError::ChecksumError
TransferLedgerError::VerifiyingContextDecodeError
When they occur we just trigger tx-pause
pause_pallets
for MantaPay, in order to shut down any further operations, until we figure out a way forward.Before we can approve this PR for merge, please make sure that all the following items have been checked off:
L-
group to this PRA-
andC-
groups to this PRA-calamari
,A-dolphin
and/orA-manta
if your changes are meant for/impact either of these (CI depends on it)Files changed
in the Github PR explorer.Situational Notes:
BaseFilter
. Ensure every extrinsic works from front-end. If there's corresponding tool, ensure both work for each other.try-runtime
. This includes migrations inherited from upstream changes, and you can search the diffs for modifications of#[pallet::storage]
items to check for any.