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

Add externalities for parachain execution #99

Closed
2 of 4 tasks
rphmeier opened this issue Jan 22, 2019 · 8 comments
Closed
2 of 4 tasks

Add externalities for parachain execution #99

rphmeier opened this issue Jan 22, 2019 · 8 comments
Labels
J0-enhancement An additional feature request.

Comments

@rphmeier
Copy link
Contributor

rphmeier commented Jan 22, 2019

All-in-all, this should be pretty similar to the set required for substrate.

Note that there is nothing about reading or interacting with storage. This is because parachains that require storage proofs will import all needed storage as a part of the BlockData which proves validity of the parachain candidate.

@rphmeier rphmeier added the J0-enhancement An additional feature request. label Jan 22, 2019
@rphmeier rphmeier added this to the PoC-4 (Interchain Communication) milestone Jan 22, 2019
@bkchr
Copy link
Member

bkchr commented Jun 21, 2019

In Cumulus we will require to have the ext_post_message externality + the Substrate externalities.
I would propose that we support "merging" different externalities declarations in Substrate. So, we can have ext_post_message + the Substrate stuff in Cumulus.

For Polkadot validator, I would use the same strategy, but make sure that storage access and anything else we don't want, traps.

@rphmeier
Copy link
Contributor Author

@bkchr Note that parachains are not executed within the substrate_executor but rather within a separate executor. This issue is about doing the native side of the externalities for that executor.

@bkchr
Copy link
Member

bkchr commented Jun 25, 2019

Isn't the substrate executor a subset of the parachain executor? I thought that we specified the substrate executor as the one that needs to be supported for running a parachain? (Especially the crypto functions.)

@rphmeier
Copy link
Contributor Author

We want to emulate most of the substrate externalities in a sandbox.

@gavofyork gavofyork removed this from the PoC-4 (Interchain Communication) milestone May 21, 2020
tomusdrw pushed a commit that referenced this issue Mar 26, 2021
@apopiak
Copy link
Contributor

apopiak commented Jul 26, 2021

@rphmeier
What's the state of sandbox support (and is there an issue tracking it and its dependencies)?

(Edit: is paritytech/substrate#9354 a blocker?)

@rphmeier
Copy link
Contributor Author

Yep, that's a blocker in practice. @pepyakin is better equipped to comment on future support

@pepyakin
Copy link
Contributor

Yeah, I think that issue is the way to go for tracking.

@rphmeier
Copy link
Contributor Author

outdated

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
J0-enhancement An additional feature request.
Projects
None yet
Development

No branches or pull requests

5 participants