Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

attempt to make Appveyor's Cygwin test faster #2623

Merged
merged 4 commits into from
May 7, 2021
Merged

attempt to make Appveyor's Cygwin test faster #2623

merged 4 commits into from
May 7, 2021

Conversation

Cyan4973
Copy link
Contributor

@Cyan4973 Cyan4973 commented May 6, 2021

Cygwin is the longest Appveyor test.

Appveyor is typically the CI which finishes last, and therefore our limiting factor,
presumably because it doesn't parallelize its workload.

Longer term, we should considering transferring some of these tests to github actions,
where parallelization is more effective.

edit : well, this attempt is not greatly effective.
The tests are running noticeably faster, saving > 70 seconds,
but gains are more than offset by extended compilation duration,
likely due to -O1 (instead of "default", aka -O0).
That's unexpected, I was not expecting -O1 to make such a large difference.

edit 2 : tested on my local workstation
compilation of zstd costs about ~x4 more time at -O1 than at -O0.
clang is slightly faster than gcc (about -10%), but it doesn't change the overall picture.

edit 3 : so I scaled back ambitions, and got after the easy wins,
saving approximately 20 sec to Cygwin tests, which is very little.
I'm now wondering if the majority of the delay rather comes from initial package installation.
I see in traces that there are 114 packages being updated before any test actually begins.
That seems like a lot, and is most likely source of substantial delays.
To be investigated, in a separate PR.

Cygwin is the longest Appveyor test
Appveyor is typically the CI which finish last
@Cyan4973 Cyan4973 merged commit b4637d1 into dev May 7, 2021
@Cyan4973 Cyan4973 deleted the fasterCygwin branch December 9, 2021 00:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants