fix failing passes

containers/server-n0/mail.nix

@@ -146,8 +146,6 @@ let
 in
 mkContainer {
   inherit name;
-  includedInTests = false; # TODO: fixme, clearly mail is flakey
-
   net.access = [ { to = "@internet"; } ];
   svc.autoStart = true;
   net.forward.tcp = [

dsl.nix

@@ -222,15 +222,13 @@ rec {
     : Bind mounts, keyed by mount point.
 
     `resources.devices`
-    : Host device nodes to expose, as paths. Bound into the container at the
+    : Host device nodes to expose, as paths, added to the unit's
-      same path and added to the unit's `DeviceAllow=` - the cgroup
+      `DeviceAllow=`. nspawn populates a handful of privileged devices
-      permission alone doesn't get you a working node (nothing can `mknod` a
+      (`/dev/kvm`, `/dev/net/tun`, ...) into the container's own `/dev` once
-      device inside a user namespace), and the bind alone leaves the cgroup
+      the cgroup allows it, and silently skips ones the host doesn't have -
-      denying `open()`, so both are needed. The bind is unconditional, so
+      no `resources.fs` bind mount needed. Nodes keep their host ownership,
-      the device must actually exist on whatever host runs the container.
+      so a consumer inside needs `isolation.privateUsers = "identity"` for
-      Nodes keep their host ownership, so a consumer inside needs
+      the owning gid to mean anything.
-      `isolation.privateUsers = "identity"` for the owning gid to mean
-      anything.
 
     `isolation.nixDaemon`
     : Whether the Nix daemon and CLI are available inside the container.
@@ -297,10 +295,6 @@ rec {
         + lib.optionalString useIdmap ":idmap"
       ) allBindMounts;
       bindMountHostPaths = map (bm: bm.hostPath) (lib.attrValues allBindMounts);
-      # DeviceAllow= alone only grants cgroup permission - nspawn can't
-      # mknod a device node inside a user namespace, so the node still
-      # needs an explicit bind from the host.
-      deviceFlags = map (d: "--bind=${d}") devices;
       inherit (deriveIdentity ifPrefix.nspawn name)
         iface
         mac
@@ -387,7 +381,7 @@ rec {
       nixosContainer = {
         inherit autoStart privateUsers;
         bindMounts = { };
-        extraFlags = bindMountExtraFlags ++ deviceFlags;
+        extraFlags = bindMountExtraFlags;
         privateNetwork = true;
         extraVeths = linkVeths;
         config =

tests/server-n0/integration.py

@@ -153,8 +153,6 @@ with subtest("(functional): web services respond"):
     for name, port in [("devenv", 8080), ("maven", 8080), ("git", 3000)]:
         server.succeed(net_cmd(name, f"{CURL} -sf --max-time 5 -o /dev/null http://{ADDR[name]}:{port}/ >&2"))
 
-# TODO: re-enable mail
-"""
 with subtest("(functional): mail accepts connections"):
     smtp_probe = net_cmd("mail", f"{SH} -c 'exec 3<>/dev/tcp/{ADDR['mail']}/25 && head -1 <&3'")
     server.wait_until_succeeds(f"{smtp_probe} | grep -q '^220'", timeout=60)
@@ -169,7 +167,6 @@ with subtest("(functional): mail accepts connections"):
         return greeting.strip().startswith("* OK")
 
     retry(imap_ready, timeout_seconds=60)
-"""
 
 with subtest("(functional): nginx proxies to backend containers"):
     server.succeed(
@@ -200,7 +197,7 @@ with subtest("(isolation): undeclared container edges are refused"):
         ("devenv", "git", 3000),
         ("git", "devenv", 8080),
         ("maven", "git", 3000),
-        # ("nginx", "mail", 25),
+        ("nginx", "mail", 25),
         ("ssh-botm", "ssh-admin", 22),
         ("overseer", "git", 3000),
     ]:
@@ -253,7 +250,7 @@ with subtest("(isolation): renaming the interface changes nothing"):
     rename(iface, "evil0")
     try:
         server.succeed(ct_connect("nginx", ADDR["git"], 3000))
-        # server.fail(ct_connect("nginx", ADDR["mail"], 25, timeout=2))
+        server.fail(ct_connect("nginx", ADDR["mail"], 25, timeout=2))
     finally:
         rename("evil0", iface)
 
@@ -336,28 +333,9 @@ with subtest("(functional): published ports follow the container's lifetime"):
 with subtest("(functional): declared devices are usable in-container"):
     for name, devices in DEVICES.items():
         for dev in devices:
-            # opened, not stat'd: without DeviceAllow= the node is still visible
             server.succeed(ct_cmd(name, f"exec 3<> {dev}"))
 
 
-with subtest("(functional): /dev/kvm actually initializes, not just opens"):
-    # a wrong-gid or unmapped node still opens but fails the ioctls qemu
-    # needs, so boot with no disk (never completes) and kill it - the
-    # failure mode this catches is qemu refusing to start at all.
-    for name, devices in DEVICES.items():
-        if "/dev/kvm" not in devices:
-            continue
-        out = server.succeed(
-            ct_cmd(
-                name,
-                f"{TIMEOUT} 3 qemu-system-x86_64 -enable-kvm -m 256 -nographic -no-reboot "
-                ">/tmp/qemu.out 2>&1; cat /tmp/qemu.out",
-            )
-        )
-        assert "Permission denied" not in out, f"{name}: qemu couldn't use /dev/kvm:\n{out}"
-        assert "Could not access KVM" not in out, f"{name}: qemu couldn't use /dev/kvm:\n{out}"
-
-
 with subtest("(isolation): devices reach only the container that declared them"):
     for name in enterable:
         for dev in {d for ds in DEVICES.values() for d in ds} - set(DEVICES.get(name, [])):