Stream: git-wasmtime

Topic: wasmtime / PR #7847 Fix timeouts in wasmtime serve, by us...


view this post on Zulip Wasmtime GitHub notifications bot (Jan 30 2024 at 00:51):

elliottt requested wasmtime-core-reviewers for a review on PR #7847.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 30 2024 at 00:51):

elliottt requested fitzgen for a review on PR #7847.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 30 2024 at 00:51):

elliottt opened PR #7847 from elliottt:trevor/fix-serve-epoch to bytecodealliance:main:

Using an epoch of 1 and setting its duration to the full timeout configured does not work for wasmtime serve, as the epoch is counted globally per engine. This commit switches to dividing the timeout by 10, and expiring an instance after 11 epochs have been observed, giving a maximum overshoot of 10%.

Co-authored-by: Jamey Sharp <jsharp@fastly.com>

<!--
Please make sure you include the following information:

Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.html

Please ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->

view this post on Zulip Wasmtime GitHub notifications bot (Jan 30 2024 at 00:52):

elliottt requested alexcrichton for a review on PR #7847.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 30 2024 at 03:30):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 30 2024 at 04:06):

alexcrichton merged PR #7847.


Last updated: Nov 22 2024 at 17:03 UTC