Skip to content

Update the start arguments of a latest Polkadot node and upgrade PJS#157

Merged
rzadp merged 8 commits intomasterfrom
rzadp/key-generation
Jul 1, 2024
Merged

Update the start arguments of a latest Polkadot node and upgrade PJS#157
rzadp merged 8 commits intomasterfrom
rzadp/key-generation

Conversation

@rzadp
Copy link
Copy Markdown
Contributor

@rzadp rzadp commented Jul 1, 2024

No description provided.

@rzadp rzadp requested review from a team as code owners July 1, 2024 07:30
@rzadp
Copy link
Copy Markdown
Contributor Author

rzadp commented Jul 1, 2024

With latest 1.13.0, our E2E tests are failing with:

Execution aborted due to trap: wasm trap: wasm `unreachable` instruction executed
    WASM backtrace:
    error while executing at wasm backtrace:
        0: 0x6948bb - <unknown>!rust_begin_unwind
        1: 0x150e0 - <unknown>!core::panicking::panic_fmt::he07f4fcfc0e8e78f
        2: 0x42c1e9 - <unknown>!TaggedTransactionQueue_validate_transaction: RuntimeApi, Execution failed: Execution aborted due to trap: wasm trap: wasm `unreachable` instruction executed\nWASM backtrace:\nerror while executing at wasm backtrace:\n    0: 0x6948bb - <unknown>!rust_begin_unwind\n    1: 0x150e0 - <unknown>!core::p…

      at apply (node_modules/@polkadot/util/cjs/logger.js:60:24)
      at Object.error (node_modules/@polkadot/util/cjs/logger.js:119:31)
      at logErrorMessage (node_modules/@polkadot/rpc-core/cjs/bundle.js:26:7)
      at node_modules/@polkadot/rpc-core/cjs/bundle.js:179:21

I don't know yet if we are messing something up (we are patching the binary after all), or if we discovered some breaking change in Polkadot.

@rzadp
Copy link
Copy Markdown
Contributor Author

rzadp commented Jul 1, 2024

Looks like the issue is with funding the treasury:

.transferKeepAlive(treasuryAccount, new BN("10000000000000"))

We fund the treasury in case it is empty, to allow it to send out funds after the tip referendum is approved.

@rzadp
Copy link
Copy Markdown
Contributor Author

rzadp commented Jul 1, 2024

There's been (announced) breaking changes in Polkadot - like that CheckMetadataHash feature.

I had to upgrade the PJS version and then it works.

I also had to wait for extrinsics to be included in blocks, because I had issues with nonces otherwise.

@mutantcornholio Could you review again please?

@rzadp rzadp changed the title Update the start arguments of a latest Polkadot node Update the start arguments of a latest Polkadot node and upgrade PJS Jul 1, 2024
@rzadp rzadp merged commit d410155 into master Jul 1, 2024
@rzadp rzadp deleted the rzadp/key-generation branch July 1, 2024 11:39
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.

2 participants