[None][fix] Use verified UCX environment on oci-aga - #17465
Conversation
e75f450 to
d5117ca
Compare
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (2)
WalkthroughThe OCI AGA UCX rule now uses CUDA and TCP transports with IPv4 priority. The corresponding cluster test expectation matches the new export. ChangesOCI AGA UCX configuration
Estimated code review effort: 2 (Simple) | ~10 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
/bot run |
Signed-off-by: Chuang Zhu <111838961+chuangz0@users.noreply.github.com>
d5117ca to
8976864
Compare
|
/bot run |
|
PR_Github #65221 [ run ] triggered by Bot. Commit: |
|
PR_Github #65221 [ run ] completed with state
|
Summary
oci-agaUCX environment with the validated transport set and IPv4 TCP priority.Motivation
The previous environment variables could cause failures on the
oci-agacluster. The new environment variables have been validated onoci-agawithout observed issues.Testing
pytest -q tests/unittest/scripts/test_cluster_env.py(17 passed)Dev Engineer Review
oci-agaUCX configuration to usecuda_ipc,cuda_copy,sm,self,tcp.UCX_TCP_AF_PRIO=inetfor IPv4 TCP priority.QA Engineer Review
tests/unittest/scripts/test_cluster_env.py.oci-aga-cs-001UCX export.tests/unittest/scripts/test_cluster_env.py.