Skip to content

Conversation

@israbbani
Copy link
Contributor

@israbbani israbbani commented Nov 18, 2025

The test was flaky because the pid 123 could actually exist.

@israbbani israbbani added the go add ONLY when ready to merge, run all tests label Nov 18, 2025
@israbbani israbbani marked this pull request as ready for review November 18, 2025 18:51
@israbbani israbbani requested a review from a team as a code owner November 18, 2025 18:51
@ray-gardener ray-gardener bot added the core Issues that should be addressed in Ray Core label Nov 18, 2025
Copy link
Collaborator

@edoakes edoakes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lol!

@edoakes edoakes merged commit d1cce8c into master Nov 18, 2025
5 of 6 checks passed
@edoakes edoakes deleted the irabbani/cgroups-flaky-test branch November 18, 2025 20:05
Aydin-ab pushed a commit to Aydin-ab/ray-aydin that referenced this pull request Nov 19, 2025
…on. (ray-project#58748)

The test was flaky because the pid `123` could actually exist.

Signed-off-by: irabbani <[email protected]>
Signed-off-by: Aydin Abiar <[email protected]>
400Ping pushed a commit to 400Ping/ray that referenced this pull request Nov 21, 2025
…on. (ray-project#58748)

The test was flaky because the pid `123` could actually exist.

Signed-off-by: irabbani <[email protected]>
ykdojo pushed a commit to ykdojo/ray that referenced this pull request Nov 27, 2025
…on. (ray-project#58748)

The test was flaky because the pid `123` could actually exist.

Signed-off-by: irabbani <[email protected]>
Signed-off-by: YK <[email protected]>
SheldonTsen pushed a commit to SheldonTsen/ray that referenced this pull request Dec 1, 2025
…on. (ray-project#58748)

The test was flaky because the pid `123` could actually exist.

Signed-off-by: irabbani <[email protected]>
Future-Outlier pushed a commit to Future-Outlier/ray that referenced this pull request Dec 7, 2025
…on. (ray-project#58748)

The test was flaky because the pid `123` could actually exist.

Signed-off-by: irabbani <[email protected]>
Signed-off-by: Future-Outlier <[email protected]>
Aydin-ab pushed a commit to kunling-anyscale/ray that referenced this pull request Dec 9, 2025
…on. (ray-project#58748)

The test was flaky because the pid `123` could actually exist.

Signed-off-by: irabbani <[email protected]>
Signed-off-by: Aydin Abiar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Issues that should be addressed in Ray Core go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants