Skip to content

ci: one heavy test suite per runner VM #2044

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
cpunion wants to merge 2 commits into from
Closed

ci: one heavy test suite per runner VM #2044

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
fea6887
ci: one heavy test suite per runner VM
cpunion Jul 8, 2026
2bd9e7b
ci: asymmetric suite split — 4 VMs on ubuntu, 2 balanced groups on sc…
cpunion Jul 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 31 additions & 7 deletions .github/workflows/go.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,14 +18,25 @@ concurrency:

jobs:
test:
name: test (${{matrix.os}}, ${{matrix.llvm}}, ${{matrix.suite}})
continue-on-error: true
timeout-minutes: 60
strategy:
fail-fast: false
# Cap how many heavy, llgo-building test packages share one VM:
# their concurrent clang/lld bursts were killing 16GB ubuntu
# runners and starving mac runners into go-list WaitDelay
# expirations. ubuntu (plentiful runners) gets one suite per VM;
# macos (scarce runners) gets two balanced groups of at most two
# heavy packages each — the pre-backbone peak, which mac handled.
matrix:
os:
- macos-latest
- ubuntu-latest
llvm: [19]
include:
- { os: ubuntu-latest, llvm: 19, suite: cl }
- { os: ubuntu-latest, llvm: 19, suite: ssa }
- { os: ubuntu-latest, llvm: 19, suite: testgo }
- { os: ubuntu-latest, llvm: 19, suite: rest }
- { os: macos-latest, llvm: 19, suite: cl+cabi }
- { os: macos-latest, llvm: 19, suite: ssa+testgo+rest }
runs-on: ${{matrix.os}}
steps:
- uses: actions/checkout@v7
Expand Down Expand Up @@ -69,20 +80,33 @@ jobs:
- name: Build
run: go build -v ./...

- name: Select suite packages
id: suite
run: |
case "${{matrix.suite}}" in
cl) pkgs="./cl" ;;
ssa) pkgs="./ssa ./internal/cabi" ;;
testgo) pkgs="./test/..." ;;
rest) pkgs=$(go list ./... | grep -v -E '^github.com/goplus/llgo/(cl$|ssa$|internal/cabi$|test/)' | tr '\n' ' ') ;;
cl+cabi) pkgs="./cl ./internal/cabi" ;;
ssa+testgo+rest) pkgs="./ssa ./test/... $(go list ./... | grep -v -E '^github.com/goplus/llgo/(cl$|ssa$|internal/cabi$|test/)' | tr '\n' ' ')" ;;
esac
echo "pkgs=$pkgs" >> "$GITHUB_OUTPUT"

# Both platforms upload coverage: OS-specific paths (ELF vs Mach-O
# emission, per-OS runtime shims) are otherwise invisible to
# codecov/patch and fail it on lines only the other OS executes.
- name: Test with coverage
# 45m: the caller-info acceptance suite (test/go) legitimately grew
# the covered run past the old 30m budget on macOS runners.
run: go test -timeout 45m -coverprofile="coverage.txt" -covermode=atomic ./...
run: go test -timeout 45m -coverprofile="coverage.txt" -covermode=atomic ${{steps.suite.outputs.pkgs}}

- name: Test with embedded emulator env
if: contains(matrix.suite, 'cl')
env:
LLGO_EMBED_TESTS: "1"
run: go test -v -timeout 60m ./cl -run '^TestRunEmbedEmulator$'

- name: Check std symbol coverage
if: contains(matrix.suite, 'rest')
run: bash doc/_readme/scripts/check_std_cover.sh

- name: Upload coverage reports to Codecov
Expand Down
Loading