mirror of
https://github.com/Matysh/houseplan-card
synced 2026-10-04 13:48:57 +00:00
Диспатч уходит после того, как ссылка ветки доехала
`workflow_dispatch` в API принимает только ref: SHA туда передать нельзя, имя ветки резолвится на стороне GitHub в момент запуска. Конвейер перед этим сам переписывает ветку ребейзом — и 12.09 на #536 диспатч, отправленный через три секунды после force-push, встал на ДОпушевый SHA. Гейт искал прогон строго на SHA материала, не нашёл и вернул задачу автору со словами «материал сменился». Чинить было нечего: дерево задачи не менялось ни на байт, материал сдвинул сам конвейер. Две меры, у каждой своя роль. Шаг ребейза не заканчивается, пока REST не отдаст новую вершину — именно REST, потому что через него же идёт диспатч. Минута ожидания, после чего отказ, а не молчание: диспатч на устаревший SHA стоит трёх минут гейта и потерянного захода. Гейт, не дождавшись прогона на материале и увидев на ветке диспатч на другом SHA, сначала пробует запустить ещё раз. Своя гонка этим закрывается, чужой коммит переживает и вторую попытку, а формулировка отказа больше не называет сменой материала то, что ею не является. Issue: #539 User-Visible: no
This commit is contained in:
@@ -7710,6 +7710,31 @@ const MUTANT_DEFINITIONS = [
|
||||
replace: '',
|
||||
}],
|
||||
},
|
||||
{
|
||||
id: 'gate-blames-the-author-for-its-own-stale-dispatch',
|
||||
guard: 'node --test test/validate-gate.test.mjs',
|
||||
because: '#539: `workflow_dispatch` принимает только ref, и конвейер сам переписывает '
|
||||
+ 'ветку ребейзом за секунды до запуска. Без повторной попытки собственная гонка '
|
||||
+ 'выглядит как «материал сменился», и задача уезжает автору, которому чинить нечего — '
|
||||
+ 'ровно это случилось с #536 12.09',
|
||||
patches: [{
|
||||
file: 'scripts/validate-gate.mjs',
|
||||
find: ' if (elsewhere && attempts < DISPATCH_ATTEMPTS) {',
|
||||
replace: ' if (false && elsewhere && attempts < DISPATCH_ATTEMPTS) {',
|
||||
}],
|
||||
},
|
||||
{
|
||||
id: 'rebase-dispatches-before-the-ref-settles',
|
||||
guard: 'node --test test/review-doc-guard.test.mjs',
|
||||
because: '#539: ссылка на стороне GitHub доезжает не мгновенно, а диспатч идёт по имени '
|
||||
+ 'ветки. Без ожидания шаг возвращается сразу после push, и запуск встаёт на вершину, '
|
||||
+ 'которой на ветке уже нет',
|
||||
patches: [{
|
||||
file: '.github/workflows/process.yml',
|
||||
find: ' if [ "$seen" = "$after" ]; then settled=true; break; fi',
|
||||
replace: ' settled=true; break',
|
||||
}],
|
||||
},
|
||||
{
|
||||
id: 'render-invalidation-unknown-key-ignored',
|
||||
guard: 'node --test test/render-invalidation.test.mjs',
|
||||
|
||||
@@ -20,6 +20,17 @@ import { resolve } from 'node:path';
|
||||
import { VALIDATE_APPEAR_MS, VALIDATE_TOTAL_MS } from './merge-candidate.mjs';
|
||||
|
||||
export const POLL_MS = 20_000;
|
||||
/**
|
||||
* Сколько раз гейт пробует запустить свой dispatch (#539).
|
||||
*
|
||||
* `workflow_dispatch` в API принимает только ref, а не SHA: имя ветки
|
||||
* резолвится на стороне GitHub в момент запуска. Конвейер перед этим сам
|
||||
* переписывает ветку ребейзом, и 12.09 на #536 диспатч, отправленный через
|
||||
* три секунды после force-push, встал на ДОпушевый SHA — гейт не нашёл прогона
|
||||
* на материале и вернул задачу автору, которому чинить было нечего. Вторая
|
||||
* попытка закрывает это окно: ссылка к тому времени доезжает гарантированно.
|
||||
*/
|
||||
export const DISPATCH_ATTEMPTS = 2;
|
||||
|
||||
/** Кандидат в доказательства: dispatch — только там мутанты могут быть запрошены. */
|
||||
export function isMutantRun(run) {
|
||||
@@ -51,6 +62,7 @@ export async function validateGate({ ref, sha, ops, appearMs = VALIDATE_APPEAR_M
|
||||
const ignored = new Set(); // завершённые dispatch, которые ничего не доказывают: отменённые и зелёные без мутантов
|
||||
let tracked = null;
|
||||
let dispatchedAt = null;
|
||||
let attempts = 0;
|
||||
while (ops.now() - started < totalMs) {
|
||||
const runs = (await ops.listRuns(sha)).filter((x) => isMutantRun(x) && !ignored.has(x.databaseId));
|
||||
const run = runs.find((x) => tracked && x.databaseId === tracked) || runs[0];
|
||||
@@ -75,13 +87,27 @@ export async function validateGate({ ref, sha, ops, appearMs = VALIDATE_APPEAR_M
|
||||
} else if (dispatchedAt === null) {
|
||||
await ops.dispatch(ref);
|
||||
dispatchedAt = ops.now();
|
||||
attempts = 1;
|
||||
} else if (ops.now() - dispatchedAt > appearMs) {
|
||||
// Прогон должен был появиться. Если на ветке появился dispatch на другом
|
||||
// SHA — материал сменился под ногами; иначе запуск просто не прошёл.
|
||||
// Прогон должен был появиться. Если на ветке стоит dispatch на другом
|
||||
// SHA — либо ссылка ветки в момент запуска ещё отдавала прежнюю вершину
|
||||
// (#539: конвейер сам переписал её ребейзом за секунды до этого), либо
|
||||
// материал действительно сменился. Различать гадательно нечем, поэтому
|
||||
// гейт сначала пробует ещё раз: собственная гонка этим закрывается, а
|
||||
// чужой коммит переживёт и вторую попытку.
|
||||
const elsewhere = (await ops.listRunsOnRef(ref)).filter(isMutantRun).find((x) => x.headSha && x.headSha !== sha);
|
||||
if (elsewhere && attempts < DISPATCH_ATTEMPTS) {
|
||||
await ops.dispatch(ref);
|
||||
dispatchedAt = ops.now();
|
||||
attempts += 1;
|
||||
await ops.sleep(pollMs);
|
||||
continue;
|
||||
}
|
||||
return {
|
||||
result: 'missing', url: elsewhere?.url || null,
|
||||
note: elsewhere ? `материал сменился: dispatch-прогон стоит на ${String(elsewhere.headSha).slice(0, 8)}` : 'dispatch-прогон не появился за 3 минуты',
|
||||
note: elsewhere
|
||||
? `ссылка ветки не указывает на материал: ${attempts} попыт(ки) диспатча встали на ${String(elsewhere.headSha).slice(0, 8)}`
|
||||
: 'dispatch-прогон не появился за 3 минуты',
|
||||
};
|
||||
}
|
||||
await ops.sleep(pollMs);
|
||||
|
||||
Reference in New Issue
Block a user