From 51a5a9f1d180e252bd50df976f4622735876b2aa Mon Sep 17 00:00:00 2001 From: clee2000 <44682903+clee2000@users.noreply.github.com> Date: Thu, 2 Nov 2023 14:24:57 -0700 Subject: [PATCH] [bot] Remove flaky detection for flaky across jobs (#4693) Currently buggy due to tests jumping from shard to shard. --- torchci/pages/api/flaky-tests/disable.ts | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/torchci/pages/api/flaky-tests/disable.ts b/torchci/pages/api/flaky-tests/disable.ts index 81e874feb9..8427c2964d 100644 --- a/torchci/pages/api/flaky-tests/disable.ts +++ b/torchci/pages/api/flaky-tests/disable.ts @@ -36,21 +36,17 @@ async function disableFlakyTestsAndReenableNonFlakyTests() { octokit, flakyTests, flakyTestsAcrossFileReruns, - flakyTestsAcrossJobs, issues, disabledNonFlakyTests, ] = await Promise.all([ getOctokit(owner, repo), fetchFlakyTests(`${NUM_HOURS}`), fetchFlakyTestsAcrossFileReruns(`${NUM_HOURS}`), - fetchFlakyTestsAcrossJobs(`${NUM_HOURS_ACROSS_JOBS}`), // use a larger time window so we can get more data fetchIssuesByLabel("skipped"), fetchDisabledNonFlakyTests(), ]); - const allFlakyTests = flakyTests - .concat(flakyTestsAcrossJobs) - .concat(flakyTestsAcrossFileReruns); + const allFlakyTests = flakyTests.concat(flakyTestsAcrossFileReruns); // If the test is flaky only on PRs, we should not disable it yet. const flakyTestsOnTrunk = filterThreshold( filterOutPRFlakyTests(allFlakyTests)