Skip to content

Commit 47cb654

Browse files
committed
New runTest interface fixes useNixStoreImage test
The new runTest interface lets us turn off includeTestScriptReferences, which lets the useNixStoreImage test pass. Unfortunately, I then discovered another problem with useNixStoreImage: It builds a new store image every time the VM starts up! This is undesirable for two reasons: 1. It significantly delays VM start-up. For network-service-providing VMs, this is downtime. Longer startup time likely won't trouble high-availability services with fancy load balancers, health checks, and rolling restarts, but for unsophisticated use cases that currently just accept some small amount of downtime during VM restarts for software updates, using useNixStoreImage means much longer periods of interrupted service. 2. This wastes disk I/O, building the same image over and over. This useNixStoreImage design choice was made in order to keep the guest's Nix store image out of the host's Nix store. This is especially important for the public Hydra that populates the public binary cache that's currently hosted on centralized pay-per-byte commercial storage services -- it doesn't make sense to pay to store and host the Nix store itself and also many little copies of overlapping pieces of it. So here in nixos-qemu-vm-isolation, we're happy to pay the cost of storing the guests' Nix store images in the host's Nix store, because we really value the faster start-up, and because we don't have the problem of large storage service bills and no budget to pay them with. :)
1 parent ab670ad commit 47cb654

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

README.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,6 @@ squashfs for the VM's /nix/store that contains only the VM's dependencies
33
(like the installer has) rather than a virtio mount of the host's entire
44
/nix/store.
55

6-
**Update:** `virtualisation.useNixStoreImage` exists now! But it doesn't
7-
work! :( See the note in `checks/mount-grep.nix`
6+
**Update:** `virtualisation.useNixStoreImage` exists now! But it builds
7+
the store image at runtime, rather that at build-time, drastically
8+
increasing VM start-up time.

checks/mount-grep.nix

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@ pkgs: {
33

44
hostPkgs = pkgs;
55

6+
includeTestScriptReferences = false;
7+
68
nodes = {
79
shared = _: { };
810
private = _: { imports = [ ../modules/qemu-vm-isolation.nix ]; };
@@ -26,8 +28,6 @@ pkgs: {
2628
2729
shared.succeed("[[ -e ${pkgs.pv} ]]")
2830
private.fail("[[ -e ${pkgs.pv} ]]")
29-
30-
# useNixStoreImage isn't ready until this works:
31-
# useNixStoreImage.fail("[[ -e ${pkgs.pv} ]]")
31+
useNixStoreImage.fail("[[ -e ${pkgs.pv} ]]")
3232
'';
3333
}

0 commit comments

Comments
 (0)