Skip to content

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Jan 2, 2026

To avoid running this 9 minute long build for each curl/curl master
and PR commit. Keep running it locally within this repo.

To avoid running this 9 minute long build for each curl/curl master
and PR commit.
@vszakats vszakats changed the title GHA: make the Testi386 workflow local GHA: move the Testi386 job to a local workflow Jan 2, 2026
@vszakats vszakats merged commit 0106023 into curl:master Jan 2, 2026
29 checks passed
@vszakats vszakats deleted the curli386 branch January 2, 2026 09:57
vszakats added a commit to curl/curl that referenced this pull request Jan 2, 2026
…h in 10m

To make CI turnaround time shorter, by cutting the longest running jobs.
After this patch all jobs should finish around 10-11 minutes. Down from
15-16 minutes before this patch.

Suggested-by: Stefan Eissing

The fuzzing workflow is now the slowest (with a 7-minute startup time
needed to build deps from source on each run), followed by macOS
and Windows torture tests (both split in two now). Without fuzzing, it's
under 10 minutes.

Notes:
- an extra cost with job-pairs is installing prereqs,
  configuring/building curl and tests twice. GitHub doesn't support
  making a matrix job a prereq for another workflow that may fix this:
    https://github.com/orgs/community/discussions/42335
  This overhead is significant on Windows: 11m20 -> 9m20 + 8m40
- job-pairs are annoying to maintain and keep in sync.
- splitting tests into halves is a manual process and needs to be
  revisited from time to time. Possibly something to automate with
  a runtests option, e.g. with `1 of 50%` and `2 of 50%`?

Also:
- split torture tests in two equal pieces, replacing the `FTP` + `!FTP`
  split used earlier.

Related perf improvements from today:
- make scan-build 2x fast:
  ff958fc #20159
- drop build-only Testi386 fuzz workflow (from curl), saving 9m per run:
  curl/curl-fuzzer@0106023
  curl/curl-fuzzer#236

Closes #20153
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant