| 14 Dec 2023 |
 matthewcroughan | zram inside of vm tests might be a big boost for all performance actually | 11:09:59 |
| matthewcroughan | I wonder why we don't to that | 11:10:02 |
| matthewcroughan | * I wonder why we don't do that to the nodes by default | 11:10:08 |
| matthewcroughan | lassulus: I enabled zram inside the disk builder VM, and it allowed me to install a closure with 1024M that I could only install with 2048M otherwise | 11:22:34 |
| matthewcroughan | * lassulus: I enabled zram inside the disk builder VM, and it allowed me to install a closure with 1024M that I could only install with 3GB otherwise | 11:22:40 |
 lassulus | uh weird | 11:23:16 |
| lassulus | but ok | 11:23:18 |
| matthewcroughan | @ | 12:09:31 |
| matthewcroughan | * https://github.com/nix-community/disko/pull/467 | 12:09:33 |
| matthewcroughan | I'm thinking something like disko.imageBuilder as an option namespace might begin to make sense, but not for this PR | 12:10:22 |
| matthewcroughan | disko.imageBuilder.memSize and disko.imageBuilder.extraRootModules | 12:11:02 |
 eyJhb | I've not been that productive today, but I'm very much enjoying the adventures of Matthew Croughan. And am currently having a hard time not playing with disko as well. | 12:14:28 |
| matthewcroughan | I'm astounded how fast those tests ran. | 12:15:20 |
| matthewcroughan | Hmm, it looks like f2fs won't work with this method even with extraRootModules = [ "f2fs" ] | 13:23:56 |
| matthewcroughan | since mkfs.f2fs doesn't exist unless f2fs-tools is in the initrd | 13:24:06 |
| matthewcroughan | It gets added to the initrd if the initrd is systemd-boot, but vmTools is too low level to benefit from that | 13:26:57 |
| matthewcroughan | Should we not just ditch vmTools altogether and use NixOS VMs? The vmTools machinery is shocking to me, it's like a whole other OS being maintained alongside NixOS | 13:27:27 |
| matthewcroughan | https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/tasks/filesystems/f2fs.nix#L16-L18 | 13:31:57 |
| matthewcroughan | It does not look as if you can override the utils provided to the initrd.. hmm
https://github.com/NixOS/nixpkgs/blob/master/pkgs/build-support/vm/default.nix#L29 | 13:34:29 |
| matthewcroughan | * It does not look as if you can override the utils provided to the vmTools initrd.. hmm
https://github.com/NixOS/nixpkgs/blob/master/pkgs/build-support/vm/default.nix#L29 | 13:34:38 |
| matthewcroughan | I can't solve this in a preCreateHook because those hooks get executed in a subshell which means they can't export vars? | 13:41:45 |
| matthewcroughan | time for another module option! | 13:42:02 |
| matthewcroughan | https://github.com/nix-community/disko/pull/470 | 13:53:47 |
| matthewcroughan | I would recommend merging all the PRs that have the new module option if you think they're fine, then I'll follow up with another PR to refactor the options into disko.imageBuilder.optionName | 13:54:31 |
| matthewcroughan | I'll also make an f2fs test and example, after these PRs are merged, as well as a simple raspberry pi hybrid boot example | 13:57:55 |
| matthewcroughan | and also a pi4 example using GPT | 13:58:02 |
| matthewcroughan | * and also a pi4 example using GPT partitioning | 13:58:06 |
| matthewcroughan | Oh.. I can't do type = lib.types.functionTo (lib.types.listOf lib.types.package); and set it to an empty list | 14:03:13 |
| matthewcroughan | * Oh.. I can't do type = lib.types.functionTo (lib.types.listOf lib.types.package); and set it to an empty list [] | 14:03:15 |
| matthewcroughan | I saw your suggestion, but it should just be listOf package right? | 14:09:19 |
