Skip to content

Commit 574dae3

Browse files
committed
chore: Extend time limit for waiting for staging checks during integration test
Signed-off-by: Yoriyasu Yano <[email protected]>
1 parent 362e887 commit 574dae3

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

deployments/release/integration_test.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -537,8 +537,9 @@ Deno.test("passed required rule and failed automerge requires review", async (t)
537537
});
538538

539539
/**
540-
* Wait for the Fensak Staging check to show up on the head commit of the given branch. This will timeout after 1
541-
* minute.
540+
* Wait for the Fensak Staging check to show up on the head commit of the given branch. This will timeout after
541+
* 3 minutes. 3 minutes feels very long, but unfortunately sometimes GitHub has significant delays in the webhooks so it
542+
* makes sense to wait that long.
542543
*/
543544
async function waitForFensakStagingCheck(
544545
octokit: Octokit,
@@ -548,7 +549,7 @@ async function waitForFensakStagingCheck(
548549
previousCheckRuns?: number[],
549550
): Promise<GitHubCheckRun> {
550551
const maxRetries = 60;
551-
const sleepBetweenRetries = 1000;
552+
const sleepBetweenRetries = 3000;
552553

553554
const headSHA = await getHeadSHA(octokit, owner, repoName, branchName);
554555
for (let i = 0; i < maxRetries; i++) {

0 commit comments

Comments
 (0)