-
Notifications
You must be signed in to change notification settings - Fork 21
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
[FINAL] feat: WASM heap memory limit for canisters #278
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks a lot @mraszyk
This adds `wasm_memory_limit` to `CanisterSettings` and `DefiniteCanisterSettings`. The corresponding spec change: - dfinity/interface-spec#278
@mraszyk: could you please merge this PR? The replica supports the feature now. Rust CDK and agent changes are blocked by this spec PR being merged according to the SDK team. |
* feat: Add `wasm_memory_limit` to the management canister This adds `wasm_memory_limit` to `CanisterSettings` and `DefiniteCanisterSettings`. The corresponding spec change: - dfinity/interface-spec#278 * Add the PR number to the change log * unify doc format * better test coverage --------- Co-authored-by: Linwei Shang <[email protected]>
@ulan There seem to be some mismatches between the spec and the implementation - could you please take a look at my above comments? |
This PR introduces a configurable WASM heap memory limit for canisters.