From 899f65097b289d0bb033c2aa584cc54c7741c2b0 Mon Sep 17 00:00:00 2001 From: Dallin Romney Date: Thu, 25 Jun 2026 21:20:44 -0700 Subject: [PATCH] ci: park timing summary collection (#96930) --- .github/workflows/ci.yml | 3 ++- test/scripts/ci-workflow-guards.test.ts | 7 ++++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index b006e5ff80e1..66afce1889e4 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -2419,7 +2419,8 @@ jobs: - macos-swift - ios-build - android - if: ${{ !cancelled() && always() && github.event_name != 'push' && (github.event_name != 'pull_request' || !github.event.pull_request.draft) }} + # Re-enable this job when we want to collect CI timing data for timing optimization. + if: ${{ false && !cancelled() && always() && github.event_name != 'push' && (github.event_name != 'pull_request' || !github.event.pull_request.draft) }} runs-on: ubuntu-24.04 timeout-minutes: 5 steps: diff --git a/test/scripts/ci-workflow-guards.test.ts b/test/scripts/ci-workflow-guards.test.ts index 96261e3fee60..eaa09d2e872e 100644 --- a/test/scripts/ci-workflow-guards.test.ts +++ b/test/scripts/ci-workflow-guards.test.ts @@ -573,7 +573,11 @@ describe("ci workflow guards", () => { expect(runStep.run).toContain("childEnv[key] = value"); }); - it("uploads a CI timing summary after the run lanes finish", () => { + it("keeps the CI timing summary parked for timing optimization work", () => { + expect(readFileSync(".github/workflows/ci.yml", "utf8")).toContain( + "Re-enable this job when we want to collect CI timing data for timing optimization.", + ); + const workflow = readCiWorkflow(); const timingJob = workflow.jobs["ci-timings-summary"]; @@ -598,6 +602,7 @@ describe("ci workflow guards", () => { "ios-build", "android", ]); + expect(timingJob.if).toContain("false"); expect(timingJob.if).toContain("always()"); expect(timingJob.if).toContain("!cancelled()");