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

CI: Reduce container and machine usage #2079

Merged

Conversation

ericoporto
Copy link
Member

@ericoporto ericoporto commented Aug 7, 2023

  • make all macOS builds in a single machine
  • CMake Linux task builds Debug and Release
  • Additionally, in CMake Linux task, pass j2 for minimal parallelism
  • Windows Engine and Editor tasks builds Debug and Release in the same task
  • Make sure a test runs right after one build, so we can fail earlier
  • for Linux make task (which doesn't make into release), drop the i386 build - we don't run tests in it
  • for the engine and tests, use VS builds with multiprocessor compilation

This is in-line with #2076, with the idea of reducing a bit the usage of Cirrus CI, the idea is we can possibly still stay under it's set limit of compute credits by doing few changes - I know these list above looks like a lot, but the general of the tasks done is the same and it produces the same binaries using the same containers.

This is safe to merge. I can prepare a proper version for merging in ags4 branch too - because of the 2nd compiler there are slight differences there.

port to 3.6.0 branch in #2115.

- make all macOS builds in a single machine
- CMake Linux task builds Debug and Release
- In CMake Linux task, pass j2 for minimal parallelism
- Windows Engine and Editor tasks builds Debug and Release in the same task
- Make sure a test runs right after one build, so we can fail earlier
- for Linux make task (which doesn't make into release), drop the i386 build - we don't run tests in it
- for the engine and tests, use VS builds with multiprocessor compilation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants