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

[ci] skip the failing cpp tests on mac #45273

Merged
merged 1 commit into from
May 14, 2024
Merged

[ci] skip the failing cpp tests on mac #45273

merged 1 commit into from
May 14, 2024

Conversation

can-anyscale
Copy link
Collaborator

@can-anyscale can-anyscale commented May 11, 2024

Skip the current failing cpp tests on mac so that it can pass. It currently soft-fails, hinders other errors, etc.

Test:

@can-anyscale can-anyscale marked this pull request as ready for review May 13, 2024 17:31
Comment on lines +308 to +309
ray cpp --generate-bazel-project-template-to ray-template
pushd ray-template && bash run.sh
Copy link
Collaborator

Choose a reason for hiding this comment

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

shouldn't we fix this rather than skipping?

maybe have two tests one with this, one without or something? so that the failure is separated rather than just skipped?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

let's create a ticket to fix it yes, my proposal is to stop running it on CI until that is fixed

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

split into another job is also fine yes, but it burns money by keeping failing and retrying itself

Copy link
Collaborator

Choose a reason for hiding this comment

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

burns money by keeping failing and retrying itself

they just exit 1 here. I am okay with skipping running tests, but I prefer to have a red color if it should be release blocking.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

red color if it should be release blocking

This is the problem yeah; the whole jobs are currently soft-failed and completely ignored (I removed the failing part and also removed the soft fail in this PR).

@can-anyscale
Copy link
Collaborator Author

we probably need @jjyao to see if we can track this test separately and not through a soft fail job in CI

@can-anyscale
Copy link
Collaborator Author

can-anyscale commented May 14, 2024

discussed offline, Jiajun will help to cut tickets for ant to follow up; we'll skip it in CI until it's fixed

@can-anyscale can-anyscale merged commit 5b2b019 into master May 14, 2024
6 checks passed
@can-anyscale can-anyscale deleted the can-skipmac branch May 14, 2024 23:51
ryanaoleary pushed a commit to ryanaoleary/ray that referenced this pull request Jun 6, 2024
Skip the current failing cpp tests on mac so that it can pass. It
currently soft-fails, hinders other errors, etc.

Test:
-
https://buildkite.com/ray-project/premerge/builds/25983#018f69fc-44dc-4329-b5af-0dcfdd59d74b

Signed-off-by: can <[email protected]>
Signed-off-by: Ryan O'Leary <[email protected]>
ryanaoleary pushed a commit to ryanaoleary/ray that referenced this pull request Jun 6, 2024
Skip the current failing cpp tests on mac so that it can pass. It
currently soft-fails, hinders other errors, etc.

Test:
-
https://buildkite.com/ray-project/premerge/builds/25983#018f69fc-44dc-4329-b5af-0dcfdd59d74b

Signed-off-by: can <[email protected]>
Signed-off-by: Ryan O'Leary <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants