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

Host function not implemented when forking v1.11 chain. #754

Closed
metricaez opened this issue May 15, 2024 · 7 comments
Closed

Host function not implemented when forking v1.11 chain. #754

metricaez opened this issue May 15, 2024 · 7 comments

Comments

@metricaez
Copy link

metricaez commented May 15, 2024

We are currently facing an issue when forking a chain that is at v1.11 with the following log:

[12:09:46.874] INFO (block-builder): Failed to apply extrinsic {} Error: Host function not implemented: ext_storage_proof_size_storage_proof_size_version_1
    app: "chopsticks"

We have encountered a similar issue in the past in our own implementation when upgrading from v1.8 to v1.9 as a new node function is implemented from that version and forward as shown here.

@xlc
Copy link
Member

xlc commented May 16, 2024

it needs to be implemented on smoldot first

@xlc
Copy link
Member

xlc commented May 17, 2024

can you provide a wasm and config file so we can later check if the implementation works?

@qiweiii qiweiii mentioned this issue May 17, 2024
@metricaez
Copy link
Author

metricaez commented May 17, 2024

hey! unfortunately I cannot provide the current rpc we are trying with but I have built a lean version that has no business logic of the runtime that I can share with you and still reproduces the error. Please let me know what do exactly would you need me to share with you. thanks

@xlc
Copy link
Member

xlc commented May 20, 2024

you can upload a wasm and config file here that can repreoduce this issue
also give #755 a try to see if it fixes the issue

@metricaez
Copy link
Author

Hi, I have built and tested #755 and it does fix the issue, thanks!

On the other hand, I am curious and a little bit confused about the wasm request since the issue comes from the node impl and not the runtime itself. Could you please elaborate ?

@xlc
Copy link
Member

xlc commented May 20, 2024

issue is always coming from wasm because that’s the only thing chopsticks is testing
for this particular case, the wasm is using a new host function that wasn’t implemented by smoldot so I requested smoldot to implement it and have chopsticks updated to include the new host function

@xlc
Copy link
Member

xlc commented May 21, 2024

should be fixed by #755

@xlc xlc closed this as completed May 21, 2024
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

2 participants