Skip to content

[NVBUG-6448152][test] TEST ONLY; DO NOT REVIEW: trace disaggregated transfer lifecycle - #16567

Closed
chienchunhung wants to merge 5 commits into
NVIDIA:mainfrom
chienchunhung:codex/nvbug-6448152-post15139-lifecycle-trace
Closed

[NVBUG-6448152][test] TEST ONLY; DO NOT REVIEW: trace disaggregated transfer lifecycle#16567
chienchunhung wants to merge 5 commits into
NVIDIA:mainfrom
chienchunhung:codex/nvbug-6448152-post15139-lifecycle-trace

Conversation

@chienchunhung

Copy link
Copy Markdown
Collaborator

Diagnostic only

This is a CI-only experiment and is not intended to merge. It is a direct child of the exact PR #16566 treatment head, with no rebase or current-main code mixed into the experiment tree.

Why

The matched #15139 / #15737-controlled A/B produced:

The common #15737 sender wakeup fix therefore does not remove the #15139-specific lifecycle stall. The treatment repeatedly reports CTX KV-transfer future wait timeouts while GEN continues decoding.

This run distinguishes among:

  1. local NIXL futures remaining pending;
  2. TP/PP consensus entry skew or a slow PP rendezvous;
  3. a completed consensus decision that is delayed before Python resource/KV release.

What is instrumented

TRTLLM_NVBUG_6448152_TRACE=1 is enabled only in the exact GB300 perf-sanity YAML. The trace records transition-only NVBUG6448152_DIAG markers for:

  • CTX/GEN transfer enqueue and local future terminal state;
  • the first CTX future wait timeout per request;
  • TP and PP consensus enter/exit, local duration, rank, communicator rank, and call sequence;
  • slow consensus calls and bounded 512-call checkpoints while transfers are active;
  • global commit, Python call source/sequence, termination dispatch, and resource release.

The diagnostic path adds no collective, request-state decision, cancellation, timeout, or workload change. The default-off path retains the original two-communicator reduction call. Absolute steady-clock timestamps are not compared across nodes.

Experiment identity

Local validation

  • clang-format, YAPF, YAML parsing, Python byte-compilation, git diff --check, codespell, and relevant lint hooks pass.
  • A local C++ build is unavailable, so CI compilation is the first compile validation.
  • Two unrelated historical-tree test-list hooks crash under Python 3.9 on str | None; they were skipped for the commit after all patch-relevant hooks passed.

chienchunhung and others added 5 commits July 17, 2026 15:10
Signed-off-by: Chien-Chun Hung <2679986+chienchunhung@users.noreply.github.com>
Signed-off-by: Yanchao Lu <yanchaol@nvidia.com>
(cherry picked from commit 833ddd2)
Signed-off-by: Chien-Chun Hung <2679986+chienchunhung@users.noreply.github.com>
Apply only the CacheSender readiness synchronization portion of NVIDIA#15737 to the exact NVIDIA#15139 boundary experiment. This keeps the historical control and treatment matched while removing the known lost-wakeup confound.

Signed-off-by: Chien-Chun Hung <2679986+chienchunhung@users.noreply.github.com>
Keep the historical experiment tree unchanged while making the diagnostic PR mergeable for the Blossom launch gate.

Signed-off-by: Chien-Chun Hung <2679986+chienchunhung@users.noreply.github.com>
Signed-off-by: Chien-Chun Hung <2679986+chienchunhung@users.noreply.github.com>
@chienchunhung

Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast --stage-list "GB300-12_GPUs-3_Nodes-PyTorch-Disagg-PerfSanity-CTX1-NODE1-GPU4-GEN1-NODE2-GPU8-Post-Merge-1"

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #60126 [ run ] triggered by Bot. Commit: ad79abd Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #60126 [ run ] completed with state FAILURE. Commit: ad79abd
/LLM/main/L0_MergeRequest_PR pipeline #48509 (Partly Tested) completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@chienchunhung chienchunhung changed the title [NVBUG-6448152][test] trace disagg transfer lifecycle [NVBUG-6448152][test] TEST ONLY; DO NOT REVIEW: trace disaggregated transfer lifecycle Jul 21, 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.

3 participants