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

Runtime upgrade/extrinsic scheduling #1

Closed
drewstone opened this issue Mar 31, 2020 · 4 comments
Closed

Runtime upgrade/extrinsic scheduling #1

drewstone opened this issue Mar 31, 2020 · 4 comments

Comments

@drewstone
Copy link

It seems useful for future networks that developers are able to schedule the process of upgrades that occur. At the moment it isn't clear what ordering runtime upgrade are processed (maybe they follow the ordering of their position in the construct_runtime! call and maybe not).

Having a way to schedule which modules will run their on_runtime_upgrade hook sounds nice to have especially given it would allow chains, like Edgeware, to catch up to substrate master without jumping through so many hoops.

@drewstone drewstone changed the title Upgrade/extrinsic scheduling Runtime upgrade/extrinsic scheduling Mar 31, 2020
@bkchr bkchr transferred this issue from paritytech/substrate Mar 31, 2020
@joepetrowski
Copy link
Collaborator

We do have a scheduler incoming, will this help?
paritytech/substrate#5412

@drewstone
Copy link
Author

At first glance I do not think so @joepetrowski. I'm talking specifically about ordering on_runtime_upgrade calls once a new upgrade is pushed. Say if I want the on_runtime_upgrade in the balances module to explicitly go before the one in the system module or any others.

@joepetrowski
Copy link
Collaborator

Ah, I see what you mean. Yeah, on_runtime_upgrade executes in the order of pallets declared in construct_runtime. If you wanted to execute something out of order you may have to do two separate upgrades.

@nuke-web3
Copy link
Contributor

@drewstone is this still an issue? If so, please reopen this 🙏🏼

As far as I can tell, there is no plans for such an integration, but if you wanted to bring this up on https://github.com/paritytech/substrate , the core dev team may be able to comment on alternatives and may even try to make this possible.

al3mart added a commit that referenced this issue Aug 25, 2023
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

3 participants