[CI] Update Mooncake version in CI - #161
Conversation
Signed-off-by: 0oshowero0 <o0shower0o@outlook.com>
CLA Signature Pass0oshowero0, thanks for your pull request. All authors of the commits have signed the CLA. 👍 |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
Pull request overview
This PR updates the CI dependency installation for the Mooncake transfer engine by removing a previously applied version constraint in the main test workflow.
Changes:
- Removed the
<0.3.12pin formooncake-transfer-engine-non-cudain the test workflow. - Removed inline comments explaining why the pin existed.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| pip install -e ".[test,build,yuanrong]" | ||
| # Pin below 0.3.12: the 0.3.12 non-cuda wheel ships a mooncake_master binary | ||
| # linked against libcudart.so.12, which fails to start on runners without CUDA. | ||
| pip install "mooncake-transfer-engine-non-cuda<0.3.12" | ||
| pip install "mooncake-transfer-engine-non-cuda" |
CLA Signature Pass0oshowero0, thanks for your pull request. All authors of the commits have signed the CLA. 👍 |
As title