summary refs log tree commit diff
path: root/nixos
diff options
context:
space:
mode:
authorMaximilian Bosch <maximilian@mbosch.me>2020-08-18 12:24:02 +0200
committerMaximilian Bosch <maximilian@mbosch.me>2020-08-18 12:26:49 +0200
commit2fbddb0ccb9ad539f1b4a96ea5bc0247dff6da25 (patch)
treed93e2ca8d54ae6931830e4ef8be88e065b03eadf /nixos
parent644ecea529209ad1a7b1a1c0db03c81d206d66c0 (diff)
downloadnixpkgs-2fbddb0ccb9ad539f1b4a96ea5bc0247dff6da25.tar
nixpkgs-2fbddb0ccb9ad539f1b4a96ea5bc0247dff6da25.tar.gz
nixpkgs-2fbddb0ccb9ad539f1b4a96ea5bc0247dff6da25.tar.bz2
nixpkgs-2fbddb0ccb9ad539f1b4a96ea5bc0247dff6da25.tar.lz
nixpkgs-2fbddb0ccb9ad539f1b4a96ea5bc0247dff6da25.tar.xz
nixpkgs-2fbddb0ccb9ad539f1b4a96ea5bc0247dff6da25.tar.zst
nixpkgs-2fbddb0ccb9ad539f1b4a96ea5bc0247dff6da25.zip
nixos/test-instrumentation: properly import `options` for `qemu`-check
If `qemu-vm.nix` is imported, the option `virtualisation.qemu.consoles`
should be set to make sure that the machine's output isn't rendered on
the graphical window of QEMU.

This is needed when interactively running a NixOS test or in conjunction
with `nixos-build-vms(8)`.

The patch 257855753073987160c293fdedc9d95a33cbc0e8 tries to only do this
if the option actually exists, however this condition used to be always
false since `options` wasn't imported in the module and pointed to
`lib.options` due to the `with lib;`-clause.
Diffstat (limited to 'nixos')
-rw-r--r--nixos/modules/testing/test-instrumentation.nix11
1 files changed, 1 insertions, 10 deletions
diff --git a/nixos/modules/testing/test-instrumentation.nix b/nixos/modules/testing/test-instrumentation.nix
index a6b34e36d20..30ffb12cbad 100644
--- a/nixos/modules/testing/test-instrumentation.nix
+++ b/nixos/modules/testing/test-instrumentation.nix
@@ -1,22 +1,13 @@
 # This module allows the test driver to connect to the virtual machine
 # via a root shell attached to port 514.
 
-{ config, lib, pkgs, ... }:
+{ options, config, lib, pkgs, ... }:
 
 with lib;
 with import ../../lib/qemu-flags.nix { inherit pkgs; };
 
 {
 
-  # This option is a dummy that if used in conjunction with
-  # modules/virtualisation/qemu-vm.nix gets merged with the same option defined
-  # there and only is declared here because some modules use
-  # test-instrumentation.nix but not qemu-vm.nix.
-  #
-  # One particular example are the boot tests where we want instrumentation
-  # within the images but not other stuff like setting up 9p filesystems.
-  options.virtualisation.qemu = { };
-
   config = {
 
     systemd.services.backdoor =