Stream: git-wasmtime

Topic: wasmtime / PR #6878 feat(wasmtime-cli): restore support f...

view this post on Zulip Wasmtime GitHub notifications bot (Aug 25 2023 at 22:49):

eduardomourar edited PR #6878:

Within the wasmtime CLI, the current default behavior is to only inject the synchronous functions to linkers. This will add a flag called --async that will inject the asynchronous one instead.

This is required in order to enable back the wasi-http module because it has been implemented only with async functions. Additionally, it will help the async support code path to be tested as async should be enabled by default in the future.

This will enable back the wasi-http module within the wasmtime CLI. We will make the bindgen synchronous through the tokio executor available in wasmtime.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 25 2023 at 22:52):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 26 2023 at 00:21):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 27 2023 at 21:00):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 27 2023 at 21:47):

eduardomourar has marked PR #6878 as ready for review.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 28 2023 at 03:23):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 28 2023 at 08:16):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 28 2023 at 22:25):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 29 2023 at 00:46):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 29 2023 at 13:22):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 29 2023 at 14:48):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 29 2023 at 16:25):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 30 2023 at 09:15):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 30 2023 at 09:46):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 30 2023 at 14:28):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 31 2023 at 22:20):

pchickey submitted PR review:

With a few changes we can accept this PR, thanks!

Just as a heads up, I know you put a lot of labor into keeping the core-wasm wasmtime::Linker version of this code working, including the sync side of the bindings. We'll keep those working for the wasmtime 13 release, but after that we are going to swap the implementation over to use wasmtime-wasi::preview2 streams and pollables, as well as component resource types. When we make that switch, we will deprecate the core-wasm interfaces, since we don't want to maintain both.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 31 2023 at 22:20):

pchickey submitted PR review:

With a few changes we can accept this PR, thanks!

Just as a heads up, I know you put a lot of labor into keeping the core-wasm wasmtime::Linker version of this code working, including the sync side of the bindings. We'll keep those working for the wasmtime 13 release, but after that we are going to swap the implementation over to use wasmtime-wasi::preview2 streams and pollables, as well as component resource types. When we make that switch, we will deprecate the core-wasm interfaces, since we don't want to maintain both.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 31 2023 at 22:20):

pchickey created PR review comment:

this is impressive! nice job

view this post on Zulip Wasmtime GitHub notifications bot (Aug 31 2023 at 22:20):

pchickey created PR review comment:

Can you back out this formatting change, and some of the others in this file?

view this post on Zulip Wasmtime GitHub notifications bot (Aug 31 2023 at 22:20):

pchickey created PR review comment:

Using wasi-http with the component::Linker will fail because it conflicts with the stream methods in the wasmtime-wasi::preview2 implementation. So, we should instead bail in this case.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 31 2023 at 22:20):

pchickey created PR review comment:

can you please swap all println! in this file to tracing::debug!? I hadn't realized those were still in here.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 01 2023 at 09:15):

eduardomourar submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 01 2023 at 09:15):

eduardomourar created PR review comment:

that is actually not case because we are taking the implementation from the wasmtime-wasi crate when building the bindings: https://github.com/bytecodealliance/wasmtime/blob/75f3da7f029695787146cb89da5c8b3abc761738/crates/wasi-http/src/proxy.rs#L53-L54

we should be able to migrate the cli test to a component when the module code is deprecated.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 01 2023 at 17:08):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 01 2023 at 17:08):

pchickey created PR review comment:

right, but without actually using HostInputStream and HostOutputStream, it will still crash at runtime.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 01 2023 at 18:42):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 01 2023 at 18:42):

eduardomourar submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 01 2023 at 18:42):

eduardomourar created PR review comment:

done

view this post on Zulip Wasmtime GitHub notifications bot (Sep 01 2023 at 18:50):

pchickey has enabled auto merge for PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 05 2023 at 15:19):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 05 2023 at 18:52):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 05 2023 at 19:01):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 05 2023 at 20:00):

eduardomourar updated PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 05 2023 at 22:24):

pchickey has enabled auto merge for PR #6878.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 05 2023 at 23:09):

pchickey merged PR #6878.

Last updated: Dec 23 2024 at 13:07 UTC