ci: shard test job + cheap argon2 in test mode #13

Merged

steve merged 1 commits from ci-faster-tests into main 2026-05-05 08:44:31 +01:00

Conversation 0 Commits 1 Files Changed 2

+48 -4

steve commented 2026-05-05 08:41:11 +01:00

Owner

Copy Link

Copy Source

Why

The CI test job runs ~3m, dominated by internal/server/http (~156s under -race). Two cheap changes here.

What

1. Matrix-shard the test job into server-http, store, and rest (computed via go list | grep -v). Each shard gets its own runner with all CPUs to itself, so wall time becomes max(shard) instead of CPU-starved on a single runner.
2. Cheap argon2 in test mode. HashPassword swaps to 8 KiB / 1 iter / 1 lane when testing.Testing() returns true. Production params untouched. VerifyPassword reads params from the encoded hash, so existing test call sites work unchanged.

Verification

• go vet ./... clean
• internal/auth tests pass
• internal/server/http -race locally: 18.3s (vs 20s baseline) — speedup is small locally because this box has plenty of CPU; on the weak CI runner where each hash costs many multiples more under -race, the impact should be larger

## Why The CI test job runs ~3m, dominated by `internal/server/http` (~156s under `-race`). Two cheap changes here. ## What 1. **Matrix-shard the test job** into `server-http`, `store`, and `rest` (computed via `go list | grep -v`). Each shard gets its own runner with all CPUs to itself, so wall time becomes `max(shard)` instead of CPU-starved on a single runner. 2. **Cheap argon2 in test mode.** `HashPassword` swaps to 8 KiB / 1 iter / 1 lane when `testing.Testing()` returns true. Production params untouched. `VerifyPassword` reads params from the encoded hash, so existing test call sites work unchanged. ## Verification - `go vet ./...` clean - `internal/auth` tests pass - `internal/server/http -race` locally: 18.3s (vs 20s baseline) — speedup is small locally because this box has plenty of CPU; on the weak CI runner where each hash costs many multiples more under `-race`, the impact should be larger

steve added 1 commit 2026-05-05 08:41:12 +01:00

ci: shard test job + cheap argon2 in test mode ... 03e5ec31f1

Test job was wall-clocked by `internal/server/http` (~156s on the
self-hosted runner under -race). Two changes here cut that:

1. Matrix-shard the test job by package group: server-http, store,
   and "rest" (everything else, computed via `go list | grep -v`).
   Each shard runs on its own runner so the heavy package isn't
   CPU-starved by siblings.

2. `auth.HashPassword` drops to cheap argon2id params (8 KiB / 1
   iter / 1 lane) when `testing.Testing()` returns true. Production
   params are unchanged. VerifyPassword reads params from the
   encoded hash so cheap-params hashes verify identically — no test
   call sites need to change.

steve merged commit 4eab42a9c3 into main 2026-05-05 08:44:31 +01:00

steve referenced this issue from a commit 2026-05-09 14:23:46 +01:00

Merge pull request 'ci: shard test job + cheap argon2 in test mode' (#13) from ci-faster-tests into main

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

steve

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: steve/restic-manager#13