also looking at this
fix(supervisor): stop reporting a shutdown as a readiness timeout
#2273
2 files
+47
-1
| @@ -323,6 +323,14 @@ export class SiteSupervisor { | ||
| 323 | 323 | await delay(this.pollIntervalMs) |
| 324 | 324 | } |
| 325 | 325 | |
| 326 | // The loop also exits when `stopAll()` flips `stopped`, which is a | |
| 327 | // deliberate shutdown rather than a timeout. Falling through to `fail()` | |
| 328 | // there logged "did not become ready within 120s" for every site that was | |
| 329 | // still booting, on every shutdown — a failure that never happened, and a | |
| 330 | // second teardown on top of the one `stop()` is already running. | |
| 331 | if (this.stopped) | |
| 332 | return | |
| 333 | ||
| 326 | 334 | if (state.status === 'starting') |
| 327 | 335 | this.fail(state, `did not become ready within ${Math.round(this.startupTimeoutMs / 1000)}s`) |
| 328 | 336 | } |