Skip to content

ci: isolate PR230 SDMA initialization failures (do not merge) - #232

Draft
vegetabledoww wants to merge 3 commits into
hw-native-sys:mainfrom
vegetabledoww:diag/pr230-prefix-cache-logs
Draft

ci: isolate PR230 SDMA initialization failures (do not merge)#232
vegetabledoww wants to merge 3 commits into
hw-native-sys:mainfrom
vegetabledoww:diag/pr230-prefix-cache-logs

Conversation

@vegetabledoww

@vegetabledoww vegetabledoww commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Diagnostic run for #230. Do not merge.

The full-log prefix-cache reproducer failed before serving became healthy: SDMA warmup returned 507034, with CANN reporting vector core task timeouts on multiple devices. The original prefill TENSOR_WAIT_TIMEOUT remains unresolved.

This revision runs model-free ChipWorker initialization probes on the eight task-submit assigned devices. Each probe uses a fresh process. All SDMA-disabled controls run first, followed by SDMA-enabled probes, sequentially per device. This tests whether initialization fails without model weights, inference kernels, or simultaneous multi-device startup. Each process has a 240-second timeout.

Artifacts preserve per-device stdout and CANN logs, JSON exit codes and durations, the warmup binary SHA256, and pinned source revisions. Pipeline failure propagation is explicit; a failed or timed-out probe fails CI.

Runtime and toolchain pins remain the same as the failing run. No model code, runtime implementation, or original skill PR files are changed. The normal model matrix is replaced only on this temporary diagnostic branch.

Validation: Python syntax, workflow YAML parsing, shell syntax, and git diff checks passed locally. Hardware probes run only on upstream CI.

@coderabbitai

coderabbitai Bot commented Sep 11, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@vegetabledoww vegetabledoww changed the title ci: diagnose PR230 prefix-cache timeout (do not merge) ci: isolate PR230 SDMA initialization failures (do not merge) Sep 11, 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.

1 participant