Skip to content

Avoid building too many jobs at once#10929

Merged
Ericson2314 merged 2 commits intomasterfrom
avoid-building-too-many-jobs-at-once
Jun 17, 2024
Merged

Avoid building too many jobs at once#10929
Ericson2314 merged 2 commits intomasterfrom
avoid-building-too-many-jobs-at-once

Conversation

@Ericson2314
Copy link
Member

Motivation

This will avoid some out-of-memory issues in GitHub actions that result
from num jobs > 1 and num cores = 4. Once we only have the Meson build
system, this problem should go away, and we can reenable these jobs.

Context

(First commit is a separate fix and should be kept, is not temporary.)

Priorities and Process

Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

We were only doing that for the more exotic builds, just forgot.
@Ericson2314 Ericson2314 requested a review from edolstra as a code owner June 17, 2024 13:08
@Ericson2314 Ericson2314 enabled auto-merge June 17, 2024 13:12
This will avoid some out-of-memory issues in GitHub actions that result
from num jobs > 1 and num cores = 4. Once we only have the Meson build
system, this problem should go away, and we can reenable these jobs.
@Ericson2314 Ericson2314 force-pushed the avoid-building-too-many-jobs-at-once branch from beed540 to c9cdc24 Compare June 17, 2024 13:16
@Ericson2314 Ericson2314 merged commit e0b4691 into master Jun 17, 2024
@Ericson2314 Ericson2314 deleted the avoid-building-too-many-jobs-at-once branch June 17, 2024 13:53
@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/2024-06-17-nix-team-meeting-minutes-153/47186/1

roberth added a commit that referenced this pull request Jun 25, 2024
Restore meson CI after #10929
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.

3 participants