ci: paginate Docker completion status reads

This commit is contained in:
Dallin Romney
2026-08-09 15:03:01 +08:00
parent fdeb02208c
commit 2770570676
2 changed files with 9 additions and 5 deletions
@@ -2758,7 +2758,7 @@ jobs:
sha256sum "${notes_file}" | awk '{print $1}'
)"
docker_status_file="${RUNNER_TEMP}/extended-stable-docker-status.json"
gh api "repos/${GITHUB_REPOSITORY}/commits/${TARGET_SHA}/status" > "${docker_status_file}"
gh api "repos/${GITHUB_REPOSITORY}/commits/${TARGET_SHA}/status?per_page=100" > "${docker_status_file}"
docker_status_run_id="$(node .release-harness/scripts/docker-channel-promote.mjs \
--find-status-file "${docker_status_file}" \
--version "${release_version}" \
@@ -2879,7 +2879,7 @@ jobs:
status_file="${RUNNER_TEMP}/extended-stable-docker-status.json"
docker_status_run_id=""
for attempt in $(seq 1 12); do
gh api "repos/${GITHUB_REPOSITORY}/commits/${TARGET_SHA}/status" > "${status_file}"
gh api "repos/${GITHUB_REPOSITORY}/commits/${TARGET_SHA}/status?per_page=100" > "${status_file}"
docker_status_run_id="$(node scripts/docker-channel-promote.mjs \
--find-status-file "${status_file}" \
--version "${release_version}" \
@@ -1602,6 +1602,7 @@ describe("release validation no-push transport", () => {
expect(createDraft.run).toContain('verify_release_resource "${release_id}" false true');
expect(createDraft.run).toContain("wait_until_not_latest");
expect(createDraft.run).toContain("Docker completion will be checked independently");
expect(createDraft.run).toContain("commits/${TARGET_SHA}/status?per_page=100");
expect(createDraft.run).toContain("--find-status-file");
expect(createDraft.run).toContain("docker_already_published=true");
expect(createDraft.run).toContain("public without Docker completion");
@@ -1617,9 +1618,12 @@ describe("release validation no-push transport", () => {
expect(verifyDockerCompletion.if).toContain(
"needs.prepare_extended_stable_release.outputs.docker_already_published == 'true'",
);
expect(step(verifyDockerCompletion, "Verify durable Docker completion status").run).toContain(
"--find-status-file",
);
const verifyDockerCompletionRun = step(
verifyDockerCompletion,
"Verify durable Docker completion status",
).run;
expect(verifyDockerCompletionRun).toContain("commits/${TARGET_SHA}/status?per_page=100");
expect(verifyDockerCompletionRun).toContain("--find-status-file");
expect(finalizeRelease.needs).toEqual([
"resolve_release_target",
"prepare_extended_stable_release",