ci: move Security cron to weekly Saturday Sydney night#34
Conversation
Was daily at 03:00 UTC = 13:00 AEST = middle of Sydney workday. Combined with the 8h-per-target deep fuzz (PR #33), that's ~128 runner-hours/day saturating the ARC pool during work hours. New schedule: Saturday 11:07 UTC. - AEST (winter): Sat 21:07 Sydney - AEDT (summer): Sat 22:07 Sydney Year-round Saturday night, regardless of DST. PR-time still has strong coverage: - Fast Security Checks (cargo audit, deny, clippy, tests) - Quick Fuzz (120s corpus smoke) - Cargo Vet (since #32 — runs on pull_request) - CodeQL (per push) Off-minute (:07) avoids cron pile-up at :00. `cancel-in-progress` concurrency still applies; with weekly cadence and ~16h runs, no overlap.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe security workflow scheduling was adjusted from daily execution at 03:00 UTC to weekly execution at 11:07 UTC on Saturdays. Updated comments explain the new cadence and off-minute timing choice to distribute runner-hour consumption and avoid cron pile-up. ChangesSecurity Workflow Schedule
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Summary
The Security workflow was firing daily at
0 3 * * *UTC = 13:00 Sydney = middle of the maintainer's workday. With PR #33's 8h-per-target deep fuzz, that means ~128 runner-hours/day saturating ARC during work hours.Moves to weekly Saturday 11:07 UTC:
Year-round Saturday night, regardless of DST flip.
Why weekly is enough
For a stable crypto crate, deep fuzz is for finding bugs, not gating merges. PR-time coverage is already strong:
Daily 8h fuzz buys diminishing returns — most regressions land via PRs and get caught by the PR-time gates. Weekly deep fuzz preserves the value (finding deep state-explosion bugs over long runs) while cutting runner cost ~7×.
Off-minute
:07rather than:00— GitHub Actions' shared cron scheduler delays jobs that land on:00due to API pile-ups. Off-minute is a free reliability win.Test plan
Cargo Vet (Supply Chain)Deep Fuzz and Kani (bothschedule-only — by design)Summary by CodeRabbit