Skip to content

Commit aa296af

Browse files
committed
Implement retry logic in CI workflows.
1 parent 2f4e1cc commit aa296af

File tree

1 file changed

+17
-1
lines changed

1 file changed

+17
-1
lines changed

.notes/justin/worklogs/2025-09-20-ci-improvements.md

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,22 @@ Created `scripts/retry.sh` with 3 retry attempts and 5-second delays between ret
2828
- Updated E2E test command to use: `../scripts/retry.sh pnpm test:e2e`
2929

3030
### Changes made
31-
- `scripts/retry.sh`: New retry wrapper script
31+
- `scripts/retry.sh`: New retry wrapper script with 3 retry attempts and 5-second delays
3232
- `.github/workflows/smoke-test-starters.yml`: Added macOS, wrapped command with retry
3333
- `.github/workflows/playground-e2e-tests.yml`: Added macOS, wrapped command with retry
34+
35+
### Testing
36+
- Tested retry script locally - works correctly with both successful and failing commands
37+
- Script shows proper attempt counting and delay behavior
38+
- Fixed initial issue where script showed 5 retries instead of 3
39+
40+
### Results
41+
- Both workflows now include macOS in addition to Ubuntu (2 OS × 4 package managers = 8 jobs each)
42+
- Playground E2E tests already had fail-fast: false (no change needed)
43+
- Commands are wrapped with retry logic to handle transient failures
44+
- Workflows will only trigger on pushes to main or pull requests (not feature branch pushes)
45+
46+
### Matrix expansion
47+
- **Before**: 4 jobs per workflow (1 OS × 4 package managers)
48+
- **After**: 8 jobs per workflow (2 OS × 4 package managers)
49+
- Total CI jobs increased from 8 to 16 across both workflows

0 commit comments

Comments
 (0)