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

Switch to Wasmtime for parachain execution #2265

Merged
merged 4 commits into from
Jan 15, 2021
Merged

Conversation

rphmeier
Copy link
Contributor

No description provided.

@rphmeier rphmeier added A0-please_review Pull request needs code review. B1-releasenotes C1-low PR touches the given topic and has a low impact on builders. labels Jan 14, 2021
@rphmeier rphmeier requested review from bkchr and pepyakin January 14, 2021 00:15
@bkchr
Copy link
Member

bkchr commented Jan 14, 2021

One change is still required. We need to add a wasmtime feature to polkadot-parachain and activate it here:

wasmtime = [ "sc-cli/wasmtime" ]

(And the code would also need to check for this feature)

@rphmeier
Copy link
Contributor Author

@bkchr I have made that change

@bkchr
Copy link
Member

bkchr commented Jan 15, 2021

bot merge

@ghost
Copy link

ghost commented Jan 15, 2021

Waiting for commit status.

@ghost
Copy link

ghost commented Jan 15, 2021

Merge failed: "2 of 3 required status checks are expected."

@bkchr bkchr merged commit fa3c93d into master Jan 15, 2021
@bkchr bkchr deleted the rh-parachain-wasmtime branch January 15, 2021 17:03
xlc pushed a commit to xlc/polkadot that referenced this pull request Jan 26, 2021
* Switch to Wasmtime for parachain execution

* Fix the signature for halt's validate_block

* wasmtime feature enabled from cli

* Adds missing feature

Co-authored-by: Sergei Shulepov <[email protected]>
Co-authored-by: Bastian Köcher <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants