Skip to content

feat: Enable configurable VM memory limits for Wasmtime#551

Merged
mpwarres merged 3 commits into
proxy-wasm:mainfrom
dallen68:WasmConfig
Jun 17, 2026
Merged

feat: Enable configurable VM memory limits for Wasmtime#551
mpwarres merged 3 commits into
proxy-wasm:mainfrom
dallen68:WasmConfig

Conversation

@dallen68

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: dallen68 <dantebot015@gmail.com>

@mpwarres mpwarres left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, looks like clang-format needed though. Thanks!

Signed-off-by: dallen68 <dantebot015@gmail.com>

@leonm1 leonm1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be good to go with these changes, thanks.

This only applies to wasmtime for right now. We can potentially standardize the config object over time when we add additional config options to other engines.

Comment thread include/proxy-wasm/wasmtime.h Outdated
Comment thread src/wasmtime/wasmtime.cc Outdated
Signed-off-by: dallen68 <dantebot015@gmail.com>
@mpwarres mpwarres merged commit 83de5ea into proxy-wasm:main Jun 17, 2026
54 of 56 checks passed
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.

3 participants