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

speed up ci by splitting off eval and build + fix ci with restricted namespaces (backport #12050) #12055

Merged
merged 3 commits into from
Dec 14, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Dec 14, 2024

  • This speeds up macOS builds from 30 minutes to 11 minutes (3x faster).
  • Also improve error reporting e.g. printing out what actually failed to build.
  • As a result we also no longer need swap.
  • The resulting script is also very useful for running checks locally as it does not need so much RAM as before (we used significantly more memory than what our nixpkgs eval ci these days consume)
    This is an automatic backport of pull request speed up ci by splitting off eval and build + fix ci with restricted namespaces #12050 done by Mergify.

- This speeds up macOS builds from 30 minutes to 11 minutes (3x faster).
- Also improve error reporting e.g. printing out what actually failed to build.
- As a result we also no longer need swap.

(cherry picked from commit 691b1ea)
This is producing quite a bit of log output in CI.

(cherry picked from commit 31b3a34)
For our tests we need to map the root user for some tests.
However ubuntu no longer allows this by default:
https://ubuntu.com/blog/ubuntu-23-10-restricted-unprivileged-user-namespaces

(cherry picked from commit 9f2b925)
@mergify mergify bot requested a review from edolstra as a code owner December 14, 2024 15:24
@mergify mergify bot added the merge-queue label Dec 14, 2024
@mergify mergify bot merged commit fd087ce into 2.25-maintenance Dec 14, 2024
23 of 24 checks passed
@mergify mergify bot deleted the mergify/bp/2.25-maintenance/pr-12050 branch December 14, 2024 15:51
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.

1 participant