Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

election-provider: easier recovery from Phase::Emergency #11472

Closed
kianenigma opened this issue May 19, 2022 · 3 comments
Closed

election-provider: easier recovery from Phase::Emergency #11472

kianenigma opened this issue May 19, 2022 · 3 comments
Labels
J0-enhancement An additional feature request. U2-some_time_soon Issue is worth doing soon. Z2-medium Can be fixed by a coder with good Rust knowledge but little knowledge of the codebase. Z6-mentor An easy task where a mentor is available. Please indicate in the issue who the mentor could be.

Comments

@kianenigma
Copy link
Contributor

currently, if we go to Phase::Emergency, the only way out is governance:

  1. governance_fallback
  2. submit_emergency_solution

Both of which need complicated governance process.

It might be reasonable to relax this. Basically, if the signed phase fails, it should simply just continue, and miners should still be allowed to submit signed submissions that meet all criteria. The difference however should be that in this case we process the signed submission on the fly. A correct submission should be fairly cheap, and a bad submission should be highly punished.
@kianenigma kianenigma added J0-enhancement An additional feature request. U2-some_time_soon Issue is worth doing soon. Z6-mentor An easy task where a mentor is available. Please indicate in the issue who the mentor could be. Z2-medium Can be fixed by a coder with good Rust knowledge but little knowledge of the codebase. labels May 19, 2022
@Szegoo
Copy link
Contributor

Szegoo commented May 29, 2022

@kianenigma I was reading the code of the election provider, and I am not sure I understand what the term 'target' means in the code. I am pretty new to this codebase, so I hope you can help me.

@Szegoo
Copy link
Contributor

Szegoo commented Jun 17, 2022

I will give this a try.

@Szegoo Szegoo mentioned this issue Jun 18, 2022
Closed
3 tasks
@kianenigma
Copy link
Contributor Author

Won't be solved as explained in the linked PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
J0-enhancement An additional feature request. U2-some_time_soon Issue is worth doing soon. Z2-medium Can be fixed by a coder with good Rust knowledge but little knowledge of the codebase. Z6-mentor An easy task where a mentor is available. Please indicate in the issue who the mentor could be.
Projects
(Nominated) Proof of Stake
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

submit emergency solution Szegoo/substrate
2 participants
@kianenigma @Szegoo