Skip to content

[core][DON'T MERGE] Triage nccl issue in test code#62282

Open
Sparks0219 wants to merge 1 commit intoray-project:masterfrom
Sparks0219:triage/doc-code-nccl-issue
Open

[core][DON'T MERGE] Triage nccl issue in test code#62282
Sparks0219 wants to merge 1 commit intoray-project:masterfrom
Sparks0219:triage/doc-code-nccl-issue

Conversation

@Sparks0219
Copy link
Copy Markdown
Contributor

Thank you for contributing to Ray! 🚀
Please review the Ray Contribution Guide before opening a pull request.

⚠️ Remove these instructions before submitting your PR.

💡 Tip: Mark as draft if you want early feedback, or ready for review when it's complete.

Description

Briefly describe what this PR accomplishes and why it's needed.

Related issues

Link related issues: "Fixes #1234", "Closes #1234", or "Related to #1234".

Additional information

Optional: Add implementation details, API changes, usage examples, screenshots, etc.

Signed-off-by: Joshua Lee <joshlee@anyscale.com>
@Sparks0219 Sparks0219 added the go add ONLY when ready to merge, run all tests label Apr 1, 2026
@Sparks0219 Sparks0219 changed the title [core] Triage nccl issue in test code [core][DON'T MERGE] Triage nccl issue in test code Apr 1, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the doc_code_direct_transport_nccl test target in doc/BUILD.bazel by adding the NCCL_DEBUG environment variable set to INFO. I have no feedback to provide.

@Sparks0219 Sparks0219 marked this pull request as ready for review April 1, 2026 21:41
@Sparks0219 Sparks0219 requested a review from a team as a code owner April 1, 2026 21:41
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

name = "doc_code_direct_transport_nccl",
size = "small",
srcs = ["source/ray-core/doc_code/direct_transport_nccl.py"],
env = {"NCCL_DEBUG": "INFO"},
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Debug environment variable left in test configuration

Low Severity

The PR title explicitly says "DON'T MERGE" and is for triaging an NCCL issue. The env = {"NCCL_DEBUG": "INFO"} addition is a temporary debugging measure that produces verbose NCCL logging output. This is intentionally temporary diagnostic code that risks being accidentally merged into the main branch.

Fix in Cursor Fix in Web

@ray-gardener ray-gardener bot added the core Issues that should be addressed in Ray Core label Apr 2, 2026
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.

Ray fails to serialize self-reference objects

1 participant