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

WasmExecutor flag to ignore onchain heappages value #14508

Conversation

fgamundi
Copy link
Contributor

@fgamundi fgamundi commented Jul 4, 2023

Add ignore_onchain_heap_pages flag to WasmExecutorBuilder to ignore the on-chain value of heap pages and use the default one.

More info in paritytech/polkadot-sdk#71

@fgamundi fgamundi requested a review from koute as a code owner July 4, 2023 12:01
@fgamundi fgamundi marked this pull request as draft July 4, 2023 12:01
@fgamundi fgamundi marked this pull request as ready for review July 4, 2023 12:03
@bkchr bkchr added A0-please_review Pull request needs code review. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit B1-note_worthy Changes should be noted in the release notes T0-node This PR/Issue is related to the topic “node”. labels Jul 7, 2023
@fgamundi
Copy link
Contributor Author

Not sure why gitlab-check-dependent-cumulus is failing

@bkchr
Copy link
Member

bkchr commented Jul 11, 2023

Not sure why gitlab-check-dependent-cumulus is failing

Not related to your changes.

@crystalin
Copy link
Contributor

Anything preventing to merge it ?

@bkchr
Copy link
Member

bkchr commented Jul 13, 2023

Yes, a second review :P @koute ?:)

@ggwpez ggwpez added the C1-low PR touches the given topic and has a low impact on builders. label Jul 13, 2023
@koute
Copy link
Contributor

koute commented Jul 14, 2023

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 130c9ad into paritytech:master Jul 14, 2023
@fgamundi fgamundi deleted the fg-ignore-onchain-heap branch July 14, 2023 13:12
Agusrodri pushed a commit to moonbeam-foundation/substrate that referenced this pull request Jul 17, 2023
* WasmExecutor flag to ignore onchain heappages value

* fmt
nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
* WasmExecutor flag to ignore onchain heappages value

* fmt
Ank4n pushed a commit that referenced this pull request Jul 22, 2023
* WasmExecutor flag to ignore onchain heappages value

* fmt
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. B1-note_worthy Changes should be noted in the release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit T0-node This PR/Issue is related to the topic “node”.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants