Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

Force authoring on first slot after runtime upgrade #4

Closed
JoshOrndorff opened this issue Nov 1, 2021 · 0 comments
Closed

Force authoring on first slot after runtime upgrade #4

JoshOrndorff opened this issue Nov 1, 2021 · 0 comments

Comments

@JoshOrndorff
Copy link
Contributor

Currently the consensus worker predicts eligibility on a slot-by-slot basis. There is also an option to skip the prediction and force authoring. Typically nodes will not want to force authoring because it makes their logs noisy and wastes CPU time, however it can be a nice tool to have when diagnosing slot prediction issues.

Because of paritytech/polkadot-sdk#64 any storage migrations related to the process of mapping consensus ids to runtime ids can cause prediction problems. While it is hoped that the upstream issue will be fixed, in the meantime, it is useful to have the client-side force authoring in the first slot after a runtime upgrade.

This was first developed when nimbus still shared a repo with cumulus. See https://github.com/purestake/cumulus/commits/elois-force-authoring-when-runtime-change. However this change was not yet brought back into the main nimbus branch.

If it becomes necessary to use this workaround again, I prefer to pass an enum to the client-side worker:

pub enum SkipPrediction {
  Always,
  Sometimes,
  AfterRuntimeUpgade,
}
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant