Skip to content

Commit 9de9553

Browse files
committed
ci(build-tests): add daily 08:00 UTC schedule alongside pull_request
The matrix conversion dropped wall time to ~10 min (slowest cell), which is now tractable as blocking per-PR CI. Per-PR catches the "PR-introduced regression" class; schedule catches the "upstream image bump on master" class that no PR triggers. Both paths are cheap on a low-volume repo, and they cover different failure modes. 08:00 UTC is 1h after derive-ip-map's daily reconcile (07:00) so the two scheduled flows don't race on the same window. Drops the pre-merge "TODO swap pull_request for schedule" note that was premised on a 25-min sequential wall time -- that constraint is gone, so per-PR can stay. Refs: #146 (issue), #148 (this PR) Assisted-by: Claude Code
1 parent fa9e75b commit 9de9553

1 file changed

Lines changed: 13 additions & 8 deletions

File tree

.github/workflows/build-tests-live.yml

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -11,21 +11,26 @@ name: build-tests (live integration)
1111
# another. Scenario list MUST stay in sync with tools/build-tests/
1212
# integration/run-all.sh (the local-execution wrapper).
1313
#
14-
# Trigger:
15-
# - pull_request: ON during Phase 2 PR development (visible feedback
16-
# on every push). This will be dropped before merge -- per-PR is too
17-
# slow for the value once we have nightly coverage.
18-
# - workflow_dispatch: manual reruns (kept post-merge).
19-
# - schedule: TODO -- add `cron: '0 8 * * *'` (daily 08:00 UTC, 1h
20-
# after the auto-derive reconcile job) before merging this PR, in
21-
# the same commit that drops `pull_request`.
14+
# Trigger (three paths, complementary):
15+
# - pull_request: per-push feedback on PRs that touch the live-test
16+
# surface. Made feasible by the matrix below -- slowest cell is
17+
# ~10 min, so it's tractable as blocking CI for a low-volume repo.
18+
# - schedule: 08:00 UTC daily, 1h after the auto-derive reconcile
19+
# job. Catches the class of regression no PR could -- upstream
20+
# image bumps (Alpine, mariadb 10.6 → 10.7, openemr/flex) that
21+
# break demo_build.sh's assumptions without anyone editing it.
22+
# - workflow_dispatch: manual reruns (debug a flaky scheduled run
23+
# without waiting for tomorrow's tick).
2224

2325
on:
2426
pull_request:
2527
paths:
2628
- demo_build.sh
2729
- tools/build-tests/**
2830
- .github/workflows/build-tests-live.yml
31+
schedule:
32+
# Daily 08:00 UTC, 1h after the derive-ip-map reconcile job (07:00).
33+
- cron: '0 8 * * *'
2934
workflow_dispatch:
3035

3136
concurrency:

0 commit comments

Comments
 (0)