Follow-up from PR #409 review (suggested by @arekay-nv in #409 (comment)).
Today warmup is a special-cased pre-phase: its knobs are scattered (settings.warmup block, settings.timeouts.warmup_drain_timeout_s) and phases are implicit. Proposal:
The Timeouts per-phase drain fields map 1:1 onto per-phase blocks when this lands.
Follow-up from PR #409 review (suggested by @arekay-nv in #409 (comment)).
Today warmup is a special-cased pre-phase: its knobs are scattered (
settings.warmupblock,settings.timeouts.warmup_drain_timeout_s) and phases are implicit. Proposal:settings.timeouts.run_timeout_swatchdog (landed in feat(config): consolidate all time knobs into settings.timeouts; make --timeout a real run watchdog #409) stays the only global deadline.The
Timeoutsper-phase drain fields map 1:1 onto per-phase blocks when this lands.