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

add execute block call in follow-chain #513

Closed
wants to merge 7 commits into from
Closed

Conversation

qiweiii
Copy link
Member

@qiweiii qiweiii commented Nov 6, 2023

closes: #70

@qiweiii
Copy link
Member Author

qiweiii commented Nov 6, 2023

An error I get is

 [11:05:18.948] ERROR (follow-chain/49419): Error when processing new head
    err: {
      "type": "Error",
      "message": "wasm `unreachable` instruction executed",
      "stack":
          Error: wasm `unreachable` instruction executed
              at Object.callback (/Users/qiweiyang/GitHub/chopsticks/packages/chopsticks/src/plugins/follow-chain/index.ts:134:23)
    }

Any clue how I should debug this? I can confirm the wasm I used is the one with try-runtime feature

@xlc
Copy link
Member

xlc commented Nov 6, 2023

it should log more information if you have LOG_LEVEL=trace.

@qiweiii
Copy link
Member Author

qiweiii commented Nov 6, 2023

seems the log is the same

[17:27:12.719] TRACE (executor/55366): taskResponse
    response: {
      "Error": "wasm `unreachable` instruction executed"
    }

@ermalkaleci
Copy link
Collaborator

what wasm are you using? looks like something wrong with wasm file

@qiweiii
Copy link
Member Author

qiweiii commented Nov 6, 2023

what wasm are you using? looks like something wrong with wasm file

I am using wasm build from this

https://github.com/AcalaNetwork/Acala/blob/7198bb16bc6078cd574a2545aa0306161e2d196a/Makefile#L138

@ermalkaleci
Copy link
Collaborator

Remove true from setupContext

@ermalkaleci
Copy link
Collaborator

there's another problem as well. setupContext will start to latest and then we set head to finalized. So we need to set the correct block number in order to make wasm-override correct

@ermalkaleci
Copy link
Collaborator

this can't be implemented on follow-mode

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

Successfully merging this pull request may close these issues.

try-runtime integration
3 participants