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

Sentinel: shared:01m23vygyffx5hg2f5vm8r74sr:30jm3u45lg was degraded 61 minutes today: trevato/forge: "the merge queue > ten green pulls: a candidate carries eight, the next the other two, and the log says … #5362

Merged opened this

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

Merged

scout/forge:scout/5343-sentinel-shared-01m23vygyffx5hg2f5vm8r74 → main 1 commit fork

  • Merged. scout/5343-sentinel-shared-01m23vygyffx5hg2f5vm8r74 was merged into main as c4bfe60.

Review

Sign in to review this change.
1 file changed +7 −2
test/merge-queue.test.ts +7 −2
@@ -209,10 +209,15 @@ describe("the merge queue", () => {
209209 }
210210 expect(MAX_CANDIDATE).toBe(8);
211211 expect(gatesOf(repo)).toHaveLength(2);
212 expect(git(repo, "rev-list", "--count", `${base}..${gatesOf(repo)[0]!.commit}`)).toBe("8");
212 // Which gate answers first is the machine's call: the verdict is the
213 // shape, eight on main and the other two on top of them.
214 const carried = gatesOf(repo)
215 .map((g) => Number(git(repo, "rev-list", "--count", `${base}..${g.commit}`)))
216 .sort((a, b) => a - b);
217 expect(carried).toEqual([8, 10]);
213218 for (const pull of pulls) expect(merged(repo, pull)).toBe(true);
214219 const queue = said.filter((l) => l.includes("[queue] "));
215 expect(queue.some((l) => /\[queue\] gating 8 pulls since \d\d:\d\d .*waiting=2/.test(l))).toBe(true);
220 expect(queue.some((l) => /\[queue\] gating 8 pulls since \d\d:\d\d .*waiting=\d/.test(l))).toBe(true);
216221 expect(queue.some((l) => l.includes("[queue] gated 8 pulls: green"))).toBe(true);
217222 expect(queue.some((l) => l.includes("[queue] gated 2 pulls: green"))).toBe(true);
218223 }, SLOW);

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs