Skip to content
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

Windows connect test flaking with all pipe instances are busy #153

Closed
jsturtevant opened this issue Jul 14, 2023 · 3 comments · Fixed by #206
Closed

Windows connect test flaking with all pipe instances are busy #153

jsturtevant opened this issue Jul 14, 2023 · 3 comments · Fixed by #206
Labels
T-CI Changes in project's CI

Comments

@jsturtevant
Copy link
Contributor

          > @jsturtevant any thoughts on
---- synchronous::publisher::tests::test_connect stdout ----
thread 'synchronous::publisher::tests::test_connect' panicked at 'called `Result::unwrap()` on an `Err` value: Ttrpc(Windows(231))', crates\shim\src\synchronous\publisher.rs:159:50
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

231 is all pipe instances are busy might be a flaky test. Has is shown up in other PR's? I will create an issue and see if I can reproduce.

Originally posted by @jsturtevant in #152 (comment)

@jsturtevant
Copy link
Contributor Author

this looks to be a flake as the nightly job is passing: https://github.com/containerd/rust-extensions/actions/runs/5570559244

@mxpv mxpv added the T-CI Changes in project's CI label Aug 9, 2023
@jsturtevant
Copy link
Contributor Author

@jsturtevant
Copy link
Contributor Author

saw this today: https://github.com/containerd/rust-extensions/actions/runs/6318762369/job/17158455962?pr=181

I've tried reproducing locally with the following but not having any luck.

for ($i=0; $i -le 100; $i++) { cargo test -p containerd-shim }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-CI Changes in project's CI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants