From ac5491a25ff0b6aedbd91ce800022a195f3401a0 Mon Sep 17 00:00:00 2001 From: Steve Yegge Date: Sat, 13 Dec 2025 07:29:24 -0800 Subject: [PATCH] revert(ci): remove RAM disk experiment from Windows CI RAM disk approach proved insufficient for Windows test speedup: - t.Chdir() panics on ImDisk drives - Temp-only RAM disk provides marginal improvement (~50%) - Still times out at 30 minutes Reverted to simple Windows tests with continue-on-error. Created bd-bmev to replace with smoke tests instead. Closed bd-5we as RAM disk approach didn't work. --- .github/workflows/ci.yml | 36 ++++-------------------------------- 1 file changed, 4 insertions(+), 32 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 4ea849cd..c1fd2d3c 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -51,23 +51,15 @@ jobs: file: ./coverage.out fail_ci_if_error: false + # TODO: Replace with smoke tests (see bd-bmev) + # Full test suite times out; RAM disk experiments didn't help enough test-windows-cmd: name: Test (Windows - cmd) runs-on: windows-latest - continue-on-error: true # Windows filesystem is slow; don't block PRs + continue-on-error: true # Windows tests timeout; don't block PRs steps: - uses: actions/checkout@v6 - # RAM disk for temp/cache only (not workspace - t.Chdir breaks on ImDisk) - - name: Setup RAM Disk - uses: chad-golden/setup-ramdisk@v1.0.1 - with: - size-in-mb: 4096 - drive-letter: 'R' - - - name: Prepare RAM disk directories - run: New-Item -ItemType Directory -Force -Path R:\tmp, R:\go-cache, R:\go-mod - - name: Set up Go uses: actions/setup-go@v5 with: @@ -82,30 +74,15 @@ jobs: run: go build -v ./cmd/bd - name: Test cmd package - env: - TEMP: R:\tmp - TMP: R:\tmp - GOCACHE: R:\go-cache - GOMODCACHE: R:\go-mod run: go test -v -short -timeout=30m -parallel=4 ./cmd/... test-windows-internal: name: Test (Windows - internal) runs-on: windows-latest - continue-on-error: true # Windows filesystem is slow; don't block PRs + continue-on-error: true # Windows tests timeout; don't block PRs steps: - uses: actions/checkout@v6 - # RAM disk for temp/cache only (not workspace - t.Chdir breaks on ImDisk) - - name: Setup RAM Disk - uses: chad-golden/setup-ramdisk@v1.0.1 - with: - size-in-mb: 4096 - drive-letter: 'R' - - - name: Prepare RAM disk directories - run: New-Item -ItemType Directory -Force -Path R:\tmp, R:\go-cache, R:\go-mod - - name: Set up Go uses: actions/setup-go@v5 with: @@ -117,11 +94,6 @@ jobs: git config --global user.email "ci@beads.test" - name: Test internal packages - env: - TEMP: R:\tmp - TMP: R:\tmp - GOCACHE: R:\go-cache - GOMODCACHE: R:\go-mod run: go test -v -short -timeout=30m -parallel=4 ./internal/... lint: