| 7 Jul 2023 |
Vladimír Čunát | I expect we want to use 2.3.x with just default features? (nixpkgs/lib/minver.nix) | 16:21:13 |
Vladimír Čunát | It's nothing urgent. It just came up in NixOS release-managing discussion. | 16:23:03 |
cole-h | Maybe... | 16:23:10 |
cole-h | Is there a case that would have been caught with 2.3 that wasn't? | 16:23:56 |
Vladimír Čunát | I'm not aware of any, but I expect it won't be hard to create some artificial one. | 16:24:41 |
Artturin | remember that 2.3 may have worse performance than 2.4+ | 17:08:41 |
Artturin | likely has | 17:08:53 |
cole-h | Which is especially important when evals take 10 minutes on recent masters... | 17:09:08 |
cole-h | So I'd be hesitant to even entertain the idea of pinning to 2.3 until I see an actual, non-contrived / non-forced issue caused by not using the Nixpkgs minver | 17:09:59 |
Vladimír Čunát | We should probably measure the difference in performance on the particular expensive eval ofBorg is doing. Otherwise I don't trust it really. | 17:12:05 |
cole-h | I'll do so. On 5dd0190da9b5aecb5238c1bea1ebaa77f15c6a45, here are the time stats:
822.67user 14.59system 10:02.16elapsed 139%CPU (0avgtext+0avgdata 58247012maxresident)k
72088inputs+0outputs (0major+14564664minor)pagefaults 0swaps
| 17:13:06 |
cole-h | Running 2.4pre20210810_a6ba313 (not an ofborg machine, but same type) | 17:14:08 |
cole-h | (I already bisected the largest increase in eval time to 4af22aab8e239b1ca28da851755c6da1a35fc91b, which made the leap from ~4-5m to like 7m, but not yet the remaining increase to 10m) | 17:17:20 |
cole-h | * (I already bisected the largest increase in eval time to 4af22aab8e239b1ca28da851755c6da1a35fc91b, which made the leap from ~4-5m to like 7m, but not yet the remaining increase to 10m) | 17:17:28 |
Artturin | 2.3.16 and 2.4pre-rc1 have a difference of 3,902 commits https://github.com/NixOS/nix/compare/2.3.16...2.4pre-rc1
| 17:24:14 |
Artturin | * fun fact 2.3.16 and 2.4pre-rc1 have a difference of 3,902 commits https://github.com/NixOS/nix/compare/2.3.16...2.4pre-rc1
| 17:26:46 |
cole-h | $ GC_INITIAL_HEAP_SIZE=8g NIX_REMOTE= time nix-env -f ./outpaths.nix -qaP --no-name --out-path --arg checkMeta true >/dev/null
warning: unknown setting 'experimental-features'
805.99user 13.80system 13:39.89elapsed 99%CPU (0avgtext+0avgdata 54731940maxresident)k
0inputs+8outputs (1major+13685712minor)pagefaults 0swaps
13:40 on the same commit using Nix 2.13
| 17:31:47 |
cole-h | * $ GC_INITIAL_HEAP_SIZE=8g NIX_REMOTE= time nix-env -f ./outpaths.nix -qaP --no-name --out-path --arg checkMeta true >/dev/null
warning: unknown setting 'experimental-features'
805.99user 13.80system 13:39.89elapsed 99%CPU (0avgtext+0avgdata 54731940maxresident)k
0inputs+8outputs (1major+13685712minor)pagefaults 0swaps
13m40s on the same commit using Nix 2.13
| 17:31:51 |
cole-h | Err Nix 2.3 | 17:33:15 |
cole-h | lol | 17:33:16 |
cole-h | * $ GC_INITIAL_HEAP_SIZE=8g NIX_REMOTE= time nix-env -f ./outpaths.nix -qaP --no-name --out-path --arg checkMeta true >/dev/null
warning: unknown setting 'experimental-features'
805.99user 13.80system 13:39.89elapsed 99%CPU (0avgtext+0avgdata 54731940maxresident)k
0inputs+8outputs (1major+13685712minor)pagefaults 0swaps
13m40s on the same commit using Nix 2.3
| 17:33:17 |
cole-h | (coming back) using 2.15.1:
$ GC_INITIAL_HEAP_SIZE=8g NIX_REMOTE= time nix-env -f ./outpaths.nix -qaP --no-name --out-path --arg checkMeta true >/dev/null
680.15user 11.32system 11:31.55elapsed 99%CPU (0avgtext+0avgdata 43949464maxresident)k
0inputs+0outputs (0major+11026151minor)pagefaults 0swaps
| 20:59:25 |
| 8 Jul 2023 |
Vladimír Čunát | Hmm, so both CPU and RAM usage would be significantly worse. I thought of doing this for the tarball job instead, but the RAM requirements wouldn't most likely become a problem with the current infra. | 08:05:06 |
Vladimír Čunát | * Hmm, so both CPU and RAM usage would be significantly worse. I thought of doing this for the tarball job instead, but the RAM requirements would most likely become a problem with the current infra. | 08:05:18 |
Vladimír Čunát | The corresponding machines have only 64 GiB physical RAM. | 08:07:24 |
sterni | In reply to @cole-h:matrix.org Is there a case that would have been caught with 2.3 that wasn't? We already had path interpolation slip into master before: https://github.com/NixOS/ofborg/issues/612 | 08:55:27 |
| 10 Jul 2023 |
| Simon Menke joined the room. | 08:57:06 |
| 11 Jul 2023 |
Artturin | https://github.com/NixOS/ofborg/pull/642 | 01:00:41 |
Artturin | will be a great addition | 01:00:52 |
| 13 Jul 2023 |
| Vladimír Čunát changed their display name from @vcunat to vcunat. | 08:27:38 |