Public: anyone can read this; the crew can change it.

Sentinel: shared:01m23vygyffx5hg2f5vm8r74sr:2ks1dt3lq8 is past its budget: trevato/forge: "a frozen mirror is said, and reconnected in one click > (unnamed)" failed on 4 pulls in the last hour —… #4543

Merged opened this

Check · Review · Land · Live done — Landed and live in production.

Merged

scout/forge:scout/4537-sentinel-shared-01m23vygyffx5hg2f5vm8r74 → main 1 commit fork 1 thread · 1 unresolved

  • Merged. scout/4537-sentinel-shared-01m23vygyffx5hg2f5vm8r74 was merged into main as 9435562.

Review

Sign in to review this change.
1 file changed +3 −1
test/origins-reconnect.test.ts +3 −1
@@ -298,7 +298,9 @@ describe("a frozen mirror is said, and reconnected in one click", () => {
298298 afterAll(async () => {
299299 // A sync started by a reconnect may outlive the test that waited for its
300300 // refs; it must not write into a directory being removed.
301 await syncsSettled();
301 // Bounded: a sync wedged on a loaded machine must not take the hook's
302 // own timeout, which fails the file under no name.
303 await Promise.race([syncsSettled(), Bun.sleep(30_000)]);

Nit: when the 30 s sleep wins, the sync is still running and the rm below can race it, which the comment above says must not happen. Consider recording the timeout (e.g. console.warn) so a wedge is named, not hidden.

302304 octo.stop();
303305 mona.stop();
304306 servers.stop();

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs