Skip to content

CONTRIB/CI: Add cuda_ipc to PID namespace test#11347

Open
shasson5 wants to merge 4 commits intoopenucx:masterfrom
shasson5:cuda2
Open

CONTRIB/CI: Add cuda_ipc to PID namespace test#11347
shasson5 wants to merge 4 commits intoopenucx:masterfrom
shasson5:cuda2

Conversation

@shasson5
Copy link
Copy Markdown
Contributor

@shasson5 shasson5 commented Apr 15, 2026

What?

CONTRIB/CI: Add cuda_ipc to PID namespace test

Why?

Support testing cuda_ipc between namespaces

@shasson5 shasson5 added the WIP-DNM Work in progress / Do not review label Apr 26, 2026
Comment thread buildlib/pr/main.yml
options: $(DOCKER_OPT_ARGS) $(DOCKER_OPT_VOLUMES)

stages:
- stage: Codestyle
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What?!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I marked this PR as WIP-DNM, in order to perform some tests in pipline

Comment thread contrib/test_namespace.sh Outdated

echo "==== Running perftest different PID namespace test for $tl ====" >&2

cmd="$base_perftest -t $test_type -m $mem_type -p $server_port"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

local + for unshare_cmd

Comment thread contrib/test_namespace.sh Outdated
test_namespace_pid cma host ucp_get
if [ "X$have_cuda" != "Xno" ]
then
output=$(test_namespace_pid cuda_ipc,cuda_copy cuda ucp_put_bw)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is that reliable, safe and robust?

Comment thread contrib/test_namespace.sh Outdated

echo "==== Running perftest different PID namespace test for $tl ====" >&2

cmd="$base_perftest -t $test_type -m $mem_type -p $server_port"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

base_perftest is a global dependency?

@gleon99
Copy link
Copy Markdown
Contributor

gleon99 commented Apr 27, 2026

@shasson5 please update when it's ready for review

@shasson5
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 4 pipeline(s), but failed to run 1 pipeline(s).

@shasson5
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 4 pipeline(s), but failed to run 1 pipeline(s).

@shasson5 shasson5 removed the WIP-DNM Work in progress / Do not review label May 3, 2026
@shasson5
Copy link
Copy Markdown
Contributor Author

shasson5 commented May 3, 2026

@shasson5 please update when it's ready for review

ready for review @gleon99

@shasson5
Copy link
Copy Markdown
Contributor Author

shasson5 commented May 4, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 4 pipeline(s), but failed to run 1 pipeline(s).

@openucx openucx deleted a comment from svc-nixl May 4, 2026
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.

2 participants