-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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
net/http: unexplained timeout on wasip1/wasm
#64321
Comments
Found new dashboard test flakes for:
2023-11-21 21:28 wasip1-wasm-wazero go@d1dcffdb net/http (log)
|
This may be a bug in https://go.dev/cl/526117. I will investigate. (CC @golang/wasm) |
Note that the actual interestingly-running test here is:
(Filed #64404 for the unclear logging.) |
wasip1/wasm
Given that there was a test that had only been running for 2s, it seems like the test must have hung somewhere else for a long time. Unassigning to collect more data and look for a pattern. (CC @neild) |
Found new dashboard test flakes for:
2023-12-07 19:52 gotip-wasip1-wasm_wazero go@e1c0349a net/http.TestIssue4191_InfiniteGetTimeout (log)
2024-01-08 17:02 gotip-wasip1-wasm_wazero go@10a66d68 net/http.TestIssue4191_InfiniteGetTimeout (log)
|
The above seems like a different failure mode; filed as #65177. |
Change https://go.dev/cl/557037 mentions this issue: |
This may or may not be fixed by https://go.dev/cl/557037. |
Fixes #65177. Updates #65178. Updates #64321. Change-Id: I698fd3b688c7dfbde692eb7c29cbdafc89e7ca32 Cq-Include-Trybots: luci.golang.try:gotip-js-wasm,gotip-wasip1-wasm_wasmtime,gotip-wasip1-wasm_wazero Reviewed-on: https://go-review.googlesource.com/c/go/+/557037 Auto-Submit: Bryan Mills <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]> Reviewed-by: Damien Neil <[email protected]>
Found new dashboard test flakes for:
2024-01-29 22:55 js-wasm-node18 go@0c64be46 net/http (log)
2024-01-29 22:55 wasip1-wasm-wazero go@0c64be46 net/http (log)
2024-01-30 00:04 js-wasm-node18 go@f96d9a64 net/http (log)
2024-01-30 00:04 wasip1-wasm-wazero go@f96d9a64 net/http (log)
|
Found new dashboard test flakes for:
2024-01-29 22:55 gotip-wasip1-wasm_wasmtime go@0c64be46 net/http.TestServerWriteTimeout [ABORT] (log)
2024-01-29 22:55 gotip-wasip1-wasm_wazero go@0c64be46 net/http.TestServerWriteTimeout [ABORT] (log)
2024-01-30 00:04 gotip-wasip1-wasm_wasmtime go@f96d9a64 net/http.TestServerWriteTimeout [ABORT] (log)
2024-01-30 00:04 gotip-wasip1-wasm_wazero go@f96d9a64 net/http.TestServerWriteTimeout [ABORT] (log)
|
Found new dashboard test flakes for:
2024-01-04 20:40 go1.22-wasip1-wasm_wazero release-branch.go1.22@1e07c144 net/http.TestChunkedNoContent [ABORT] (log)
|
Fixes golang#65177. Updates golang#65178. Updates golang#64321. Change-Id: I698fd3b688c7dfbde692eb7c29cbdafc89e7ca32 Cq-Include-Trybots: luci.golang.try:gotip-js-wasm,gotip-wasip1-wasm_wasmtime,gotip-wasip1-wasm_wazero Reviewed-on: https://go-review.googlesource.com/c/go/+/557037 Auto-Submit: Bryan Mills <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]> Reviewed-by: Damien Neil <[email protected]>
Change https://go.dev/cl/566175 mentions this issue: |
… on js and wasip1 For #65883. Updates #65177. Updates #65178. Updates #64321. Change-Id: I698fd3b688c7dfbde692eb7c29cbdafc89e7ca32 Cq-Include-Trybots: luci.golang.try:go1.22-js-wasm,go1.22-wasip1-wasm_wasmtime,go1.22-wasip1-wasm_wazero Reviewed-on: https://go-review.googlesource.com/c/go/+/557037 Auto-Submit: Bryan Mills <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]> Reviewed-by: Damien Neil <[email protected]> (cherry picked from commit f19f31f) Reviewed-on: https://go-review.googlesource.com/c/go/+/566175 Reviewed-by: Dmitri Shuralyov <[email protected]>
… on js and wasip1 For golang#65883. Updates golang#65177. Updates golang#65178. Updates golang#64321. Change-Id: I698fd3b688c7dfbde692eb7c29cbdafc89e7ca32 Cq-Include-Trybots: luci.golang.try:go1.22-js-wasm,go1.22-wasip1-wasm_wasmtime,go1.22-wasip1-wasm_wazero Reviewed-on: https://go-review.googlesource.com/c/go/+/557037 Auto-Submit: Bryan Mills <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]> Reviewed-by: Damien Neil <[email protected]> (cherry picked from commit f19f31f) Reviewed-on: https://go-review.googlesource.com/c/go/+/566175 Reviewed-by: Dmitri Shuralyov <[email protected]>
… on js and wasip1 For golang#65883. Updates golang#65177. Updates golang#65178. Updates golang#64321. Change-Id: I698fd3b688c7dfbde692eb7c29cbdafc89e7ca32 Cq-Include-Trybots: luci.golang.try:go1.22-js-wasm,go1.22-wasip1-wasm_wasmtime,go1.22-wasip1-wasm_wazero Reviewed-on: https://go-review.googlesource.com/c/go/+/557037 Auto-Submit: Bryan Mills <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]> Reviewed-by: Damien Neil <[email protected]> (cherry picked from commit f19f31f) Reviewed-on: https://go-review.googlesource.com/c/go/+/566175 Reviewed-by: Dmitri Shuralyov <[email protected]>
Issue created automatically to collect these failures.
Example (log):
— watchflakes
The text was updated successfully, but these errors were encountered: