Skip to content

test: skip IPC peer access tests when IPC is unavailable#2092

Open
rluo8 wants to merge 1 commit into
NVIDIA:mainfrom
rluo8:fix/ipc-peer-access-skip
Open

test: skip IPC peer access tests when IPC is unavailable#2092
rluo8 wants to merge 1 commit into
NVIDIA:mainfrom
rluo8:fix/ipc-peer-access-skip

Conversation

@rluo8
Copy link
Copy Markdown
Contributor

@rluo8 rluo8 commented May 15, 2026

This is to fix bug 6176793

Problem:
FAILED memory_ipc\test_peer_access.py::TestPeerAccessNotPreservedOnImport::test_main - RuntimeError: IPC is not available on Windows
FAILED memory_ipc\test_peer_access.py::TestBufferPeerAccessAfterImport::test_main[True] - RuntimeError: IPC is not available on Windows
FAILED memory_ipc\test_peer_access.py::TestBufferPeerAccessAfterImport::test_main[False] - RuntimeError: IPC is not available on Windows

The three tests in cuda_core/tests/memory_ipc/test_peer_access.py depend on the mempool_device_x2 fixture, which only checks for two GPUs with mutual peer access — it does not check for IPC support.

The fix is to add an ipc_mempool_device_x2 fixture in cuda_core/tests/conftest.py that composes mempool_device_x2 with the same IPC-capability checks already used by ipc_device .

After the fix, the tests could be successfully skipped.
tests/memory_ipc/test_peer_access.py::TestPeerAccessNotPreservedOnImport::test_main SKIPPED (Device does not...)
tests/memory_ipc/test_peer_access.py::TestBufferPeerAccessAfterImport::test_main[True] SKIPPED (Device does ...)
tests/memory_ipc/test_peer_access.py::TestBufferPeerAccessAfterImport::test_main[False] SKIPPED (Device does...)

@github-actions github-actions Bot added the cuda.core Everything related to the cuda.core module label May 15, 2026
@rluo8
Copy link
Copy Markdown
Contributor Author

rluo8 commented May 15, 2026

Hi @mdboom @rwgk ,
Could you please help review it?
Thanks.

@github-actions
Copy link
Copy Markdown

@leofang leofang requested a review from Andy-Jost May 15, 2026 15:22
@leofang leofang added triage Needs the team's attention test Improvements or additions to tests labels May 15, 2026
Copy link
Copy Markdown
Member

@leofang leofang left a comment

Choose a reason for hiding this comment

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

I left a comment in the bug, I don't think this is correct.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cuda.core Everything related to the cuda.core module test Improvements or additions to tests triage Needs the team's attention

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants