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

[Hotfix] Fix the failure of C++ tests #27249

Merged
merged 3 commits into from
Jul 29, 2022

Conversation

SongGuyang
Copy link
Contributor

Signed-off-by: 久龙 [email protected]

Fix the issue #27237

@SongGuyang SongGuyang changed the title [Hot fix] Fix the failure of C++ tests [Hotfix] Fix the failure of C++ tests Jul 29, 2022
@scv119
Copy link
Contributor

scv119 commented Jul 29, 2022

@architkulkarni can you take a look at the runtime env related code?

@scv119
Copy link
Contributor

scv119 commented Jul 29, 2022

@SongGuyang is it ready to merge? also can you help me understand the impact of release rc0 without this fix?

@SongGuyang
Copy link
Contributor Author

@scv119 The test has passed in my mac after this PR. But I can't varify it in CI now. I think we can merge it and release rc0 first. But If we don't pick this PR, the basic cpp tests will fail after the users install the rc wheel.

Copy link
Contributor

@architkulkarni architkulkarni left a comment

Choose a reason for hiding this comment

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

Runtime env code LGTM

@SongGuyang SongGuyang merged commit 0b60d90 into ray-project:master Jul 29, 2022
SongGuyang added a commit to alipay/ant-ray that referenced this pull request Jul 29, 2022
scv119 pushed a commit that referenced this pull request Jul 29, 2022
Stefan-1313 pushed a commit to Stefan-1313/ray_mod that referenced this pull request Aug 18, 2022
Signed-off-by: 久龙 <[email protected]>
Signed-off-by: Stefan van der Kleij <[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.

[CI][runtime env] osx:https://cpp:test_python_call_cpp is failing on master/release branch
4 participants