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

Error When Testing Async Backing Enabling Runtime Upgrade #806

Closed
Mihajlo-Pavlovic opened this issue Aug 22, 2024 · 4 comments · Fixed by #809
Closed

Error When Testing Async Backing Enabling Runtime Upgrade #806

Mihajlo-Pavlovic opened this issue Aug 22, 2024 · 4 comments · Fixed by #809

Comments

@Mihajlo-Pavlovic
Copy link

We are experiencing an error when trying to test the runtime upgrade which enables syncing on NeuroWeb Testnet.
We get this log on transaction after executing enactAuthorizedUpgrade:

[18:25:56.542] INFO (block-builder): NeuroWeb Testnet building #4,302,785
    app: "chopsticks"
    number: 4302785
    extrinsics: [
      "0x410284da2004a17a90220dba4b1e3c13…77cd4b77b846f2a5f07c070010a5d4e8"
    ]
    umpCount: 0
                     runtime  ERROR: panicked at /root/.cargo/git/checkouts/polkadot-sdk-cff69157b985ed76/8c8edac/substrate/frame/aura/src/lib.rs:398:9:
assertion `left == right` failed: Timestamp slot must match `CurrentSlot`
  left: Slot(143688101)
 right: Slot(287376202)
[18:25:58.828] WARN (block-builder): Failed to apply inherents {} Error: wasm `unreachable` instruction executed
    app: "chopsticks"
/Users/mihajlopavlovic/.npm/_npx/81ad9c881cb83600/node_modules/@acala-network/chopsticks-core/dist/cjs/blockchain/block-builder.js:214
            throw new Error('Failed to apply inherents');
                  ^

Error: Failed to apply inherents
    at initNewBlock (/Users/mihajlopavlovic/.npm/_npx/81ad9c881cb83600/node_modules/@acala-network/chopsticks-core/dist/cjs/blockchain/block-builder.js:214:19)
    at async buildBlock (/Users/mihajlopavlovic/.npm/_npx/81ad9c881cb83600/node_modules/@acala-network/chopsticks-core/dist/cjs/blockchain/block-builder.js:321:44)
    at async TxPool.buildBlock (/Users/mihajlopavlovic/.npm/_npx/81ad9c881cb83600/node_modules/@acala-network/chopsticks-core/dist/cjs/blockchain/txpool.js:340:43)
    at async TxPool.buildBlockIfNeeded (/Users/mihajlopavlovic/.npm/_npx/81ad9c881cb83600/node_modules/@acala-network/chopsticks-core/dist/cjs/blockchain/txpool.js:324:9)

We are using polkadot-v1.11.0 and starting chopsticks with latest tag: npx @acala-network/chopsticks@latest -c neuroweb-testnet

This is the same error as issue #730 which is now closed.

@Mihajlo-Pavlovic
Copy link
Author

It looks connected to this issue paritytech/polkadot-sdk#3211, do chopsticks have this fix?

@ermalkaleci
Copy link
Collaborator

If you're testing the transition to async backing then it's not supported

@ermalkaleci
Copy link
Collaborator

you should try zombienet to test this transition because it depends purely on the client and you need to test your client can do it

@ermalkaleci ermalkaleci linked a pull request Aug 24, 2024 that will close this issue
@ermalkaleci
Copy link
Collaborator

ermalkaleci commented Aug 24, 2024

@Mihajlo-Pavlovic this PR will add support but you still need to test the client. Chopsticks only tests runtime

@xlc xlc closed this as completed in #809 Aug 26, 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

Successfully merging a pull request may close this issue.

2 participants