-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
[ci] jail //python/ray/serve:test_websockets #39375
Conversation
Signed-off-by: can <can@anyscale.com>
bc8e99e
to
9617618
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thx, will fix it after
do you have a pr/link/something that we can track the progress of the fixing? |
it has become very flaky recently. Signed-off-by: can <can@anyscale.com>
Hi @aslonnie and @can-anyscale looks like the test is not being run. here is my pr triggering the test: https://buildkite.com/ray-project/premerge/builds/5248#018a71c9-6550-4f3e-bd16-12bbe7132bfc |
the test did run, but in 3 parallel dockers, and only one has tests. |
instance_type: medium | ||
soft_fail: true | ||
commands: | ||
- bazel run //ci/ray_ci:test_in_docker -- //... serve --run-flaky-tests --parallelism-per-worker 3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sihanwang41 , your PR #39413 seems to have fixed the test, so you can revert this PR's change in your PR.
- maybe remove the
--parallelism-per-worker 3
? - and/or fix the issue of running 0 tests in a parallel container.
oops my bad, I'll fix the 0 test issues |
it has become very flaky recently. Signed-off-by: can <can@anyscale.com> Signed-off-by: Jim Thompson <jimthompson5802@gmail.com>
it has become very flaky recently. Signed-off-by: can <can@anyscale.com> Signed-off-by: Victor <vctr.y.m@example.com>
//python/ray/serve:test_websockets is becoming very flaky, jail it
Test: