mirror of
https://github.com/QwenLM/qwen-code.git
synced 2026-07-24 16:35:01 +00:00
ac1e473777
* ci: check out PR head SHA to avoid missing merge-ref checkout failures * ci: skip coverage comment on cancelled runs to free concurrency slot post_coverage_comment used if: always(), so on a cancelled run it still ran on the hosted pool. During rapid pushes the cancelled run's coverage job sat queued for a hosted runner, keeping the run non-terminal and holding the branch concurrency slot — so the next push's required Test (ubuntu-latest) check waited behind it (observed ~9 min, with thrash). Switch to !cancelled(): coverage still posts on test failure, but a cancelled run releases the slot immediately. Its coverage would be stale anyway. * ci(triage): route /triage comment runs to the ECS pool `/triage` comments trigger the authorize+triage jobs on hosted runners, which queue behind CI during peak hours (~2 min vs 4 s off-peak observed). Route the comment-triggered triage job to the ECS self-hosted pool, which is not bound by the GitHub-hosted concurrency limit. Safe scope: only `issue_comment` events; the job is read-only and checks out the base ref (never PR-head code). Gated on the same MAINTAINER_ECS_RUNNER_DISABLED kill-switch as `authorize`, so a disabled ECS pool falls back to hosted. PR/issues/dispatch triggers stay hosted. * ci: fetch PR head by ref name to fix 'not our ref' checkout failure * ci: align Test checkout ref with Lint (add branch_ref fallback) On workflow_dispatch the Test job runs (its if: !cancelled() overrides the implicit success() gate when classify_pr is skipped), so its checkout must honor the branch_ref input like Lint does. Without it, dispatch runs validated github.ref in Test while Lint validated the user-supplied branch_ref. * ci: retry merge-ref checkout instead of pinning PR head; drop /triage ECS routing Per review feedback: checking out refs/pull/N/head dropped merge-result validation, and merge queue isn't enabled on main to backstop it. Keep merge-ref semantics (github.ref) for the required Lint/Test checkouts and retry once to absorb the transient merge-ref lag the PR-head change was working around. Also revert /triage routing to ECS: that job runs an agent (sandbox:false, shell/write tools, repo tokens) over untrusted PR/issue text, so it stays on ephemeral hosted runners, not the persistent self-hosted pool. * ci: back off before the merge-ref checkout retry The merge ref (refs/pull/N/merge) is built asynchronously, so a back-to-back retry can hit the still-unbuilt ref again. Sleep 10s before retrying so the retry absorbs the multi-second build lag, not just instantaneous blips. |
||
|---|---|---|
| .. | ||
| actions/post-coverage-comment | ||
| ISSUE_TEMPLATE | ||
| scripts | ||
| workflows | ||
| actionlint.yaml | ||
| dependabot.yml | ||
| pull_request_template.md | ||
| release.yml | ||