Skip to content

ci: run regular tests on GitHub-hosted runners (parallel mem acceptance on Github-Large-1)#434

Merged
syifan merged 1 commit into
mainfrom
ci/github-hosted-runners
Jun 25, 2026
Merged

ci: run regular tests on GitHub-hosted runners (parallel mem acceptance on Github-Large-1)#434
syifan merged 1 commit into
mainfrom
ci/github-hosted-runners

Conversation

@syifan

@syifan syifan commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Why

MGPUSim MI300 calibration jobs saturate the shared self-hosted runners, so Akita's regular PR CI sits queued. Move the functional jobs to GitHub-hosted runners.

Changes (.github/workflows/akita_test.yml)

Job Runner
daisen2 compile, monitoring2 compile, build+lint+test, DRAM validation, NoC acceptance ubuntu-latest
mem acceptance group: Github-Large-1

mem_acceptance_test runs the parallel-engine variants (mem/acceptance_test.py invokes the binaries with -parallel, which call WithParallelEngine()), so it needs real cores → Github-Large-1. noc_acceptance_test does not use the parallel engine, so it stays free.

Notes

  • The Python-install step already falls back to apt (dnf ... || apt-get ...), and ubuntu-latest ships gcc, so CGO builds (daisen2's go-sqlite3) still work.
  • For pull_request runs GitHub uses the workflow from the PR's own branch, so open PRs pick this up after merging/rebasing main (or by applying it to their branch).

🤖 Generated with Claude Code

syifan added a commit that referenced this pull request Jun 25, 2026
Mirror of #434 so this PR's CI runs on free/large runners while the shared
self-hosted fleet is busy with MGPUSim calibration. Drop on rebase once #434 lands.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
syifan added a commit that referenced this pull request Jun 25, 2026
Mirror of #434 so this PR's CI runs on free/large runners while the shared
self-hosted fleet is busy with MGPUSim calibration. Drop on rebase once #434 lands.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The self-hosted runner fleet is saturated by MGPUSim MI300 calibration jobs, so
regular PR CI sits queued. Move the functional jobs (daisen2/monitoring2
compile, build+lint+test, DRAM validation, NoC acceptance) to ubuntu-latest.

mem_acceptance_test runs the parallel-engine variants (mem/acceptance_test.py
invokes the binaries with -parallel, which call WithParallelEngine), so it goes
to the Github-Large-1 multi-core runner group instead; a 2-core free runner
barely exercises parallelism. noc_acceptance_test does not use the parallel
engine, so it stays on a free runner.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@syifan syifan force-pushed the ci/github-hosted-runners branch from 2c2d8d0 to 69c3301 Compare June 25, 2026 20:50
syifan added a commit that referenced this pull request Jun 25, 2026
Mirror of #434 so this PR's CI runs on free/large runners while the shared
self-hosted fleet is busy with MGPUSim calibration. Drop on rebase once #434 lands.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
syifan added a commit that referenced this pull request Jun 25, 2026
Mirror of #434 so this PR's CI runs on free/large runners while the shared
self-hosted fleet is busy with MGPUSim calibration. Drop on rebase once #434 lands.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@syifan syifan merged commit aa89b48 into main Jun 25, 2026
6 checks passed
@syifan syifan deleted the ci/github-hosted-runners branch June 25, 2026 22:01
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