Skip to content
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

Add Parachain status check to pre-flight #319

Open
gianfra-t opened this issue Dec 10, 2024 · 7 comments
Open

Add Parachain status check to pre-flight #319

gianfra-t opened this issue Dec 10, 2024 · 7 comments

Comments

@gianfra-t
Copy link
Contributor

Description and potential addition

Recently an off-ramp failed at the RequestRedeem stage due to the Oracle being offline, and the redeem extrinsic failing with security.ParachainNotRunning. Furthermore, since the extrinsic was executed but failed, there is no way to recover these funds by re-executing it again.

This can actually be checked on initialization of the app (or in the backend and append to the status), and avoid starting off-ramps that will fail for certain.

@prayagd
Copy link
Collaborator

prayagd commented Dec 11, 2024

Hey team! Please add your planning poker estimate with Zenhub @ebma @gianfra-t @Sharqiewicz

@prayagd
Copy link
Collaborator

prayagd commented Dec 16, 2024

@vadaynujra should we priortize this, i consider this important as failing check would fail the offramp?

@ebma
Copy link
Member

ebma commented Dec 16, 2024

Please do, I also think it's important.

@vadaynujra
Copy link

@prayagd makes sense to prioritise this one given the low effort. Could you also confirm (preferably with a screenshot), what the user would see if the checks fail, today?

@ebma
Copy link
Member

ebma commented Dec 16, 2024

@vadaynujra we would just add this to the set of current checks and thus, it would also show the 'Application initialization failed...' error message, without giving any specific hint.

@vadaynujra vadaynujra changed the title Add Parachain status check to pre-flight. Add Parachain status check to pre-flight Dec 16, 2024
@vadaynujra
Copy link

Understood, thanks @ebma

@prayagd
Copy link
Collaborator

prayagd commented Jan 16, 2025

This ticket is depriortized for the current sprint

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants