freshlybakedca.ke / patisserie
Your one-stop-cake-shop for everything Freshly Baked has to offer
fork atom

ci: reduce evaluation memory #16

merged opened by a.starrysky.fyi targeting main from private/minion/push-xuwsvklxosst

So, we've done some things here...

  • We're no longer evaluating homes - which was basically a double-eval anyway until we get MacOS/etc. up
  • We're splitting system evals apart from each other, which will take longer over all but reduces the peak memory usage from >10GB to ~3GB

    • 10GB was unsustainable for midnight ... we were constantly OOMing when we accidentally triggered CI twice

    • ~3GB is very sustainable for midnight :)
STACK 3
#16 ci: reduce evaluation memory
0
#2
#15 ci: only allow redistributable package builds
0
#0
#17 style: reformat code
0
#0
Labels

None yet.

requested-reviewers

None yet.

approved

None yet.

tested-working

None yet.

rejected

None yet.

assignee

None yet.

Participants 1
AT URI
at://did:plc:uuyqs6y3pwtbteet4swt5i5y/sh.tangled.repo.pull/3lz4ppc6buy22
+1 -1
Interdiff #1 → #2
+1 -1
.tangled/workflows/packetmix-build.yml
··· 53 53 echo "Evaluating system $system" 54 54 55 55 eval_out=$(nix eval \ 56 - -f ./packetmix/ci.nix "systems.nixos.$system.result.config.build.toplevel.drvPath" \ 56 + -f ./packetmix/ci.nix "systems.nixos.$system.result.config.system.build.toplevel.drvPath" \ 57 57 --show-trace --raw \ 58 58 2>&1 >"/tmp/systems-drv-paths/$system" | tee /dev/stderr) 59 59 eval_status=$?

History

3 rounds 0 comments
sign up or login to add to the discussion
a.starrysky.fyi submitted #2
diff
1 commit
expand
4664f241
ci: reduce evaluation memory
expand 0 comments
pull request successfully merged
a.starrysky.fyi submitted #1
diff interdiff
1 commit
expand
9351929a
ci: reduce evaluation memory
expand 0 comments
a.starrysky.fyi submitted #0
diff
1 commit
expand
0625bbc1
ci: reduce evaluation memory
expand 0 comments