From 82c64d154b2743f34092bc1be519845c4496142d Mon Sep 17 00:00:00 2001 From: Martin Puppe Date: Thu, 4 Aug 2022 21:48:27 +0200 Subject: nixos/nixos-containers: Fix ineffective warning A warning regarding enabling NixOS containers and virtualisation.containers at the same time with state versions < 22.05 had been added in commit 3c49151f154a3872eb278c214863d926a4f8abf6. But this warning had accidentally been defined in the wrong place, and the warning has therefore not actually been in effect. This commit fixes that. --- nixos/modules/virtualisation/nixos-containers.nix | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) (limited to 'nixos/modules/virtualisation/nixos-containers.nix') diff --git a/nixos/modules/virtualisation/nixos-containers.nix b/nixos/modules/virtualisation/nixos-containers.nix index e2fb28ed633..4f074d24fda 100644 --- a/nixos/modules/virtualisation/nixos-containers.nix +++ b/nixos/modules/virtualisation/nixos-containers.nix @@ -742,12 +742,6 @@ in config = mkIf (config.boot.enableContainers) (let - warnings = flatten [ - (optional (config.virtualisation.containers.enable && versionOlder config.system.stateVersion "22.05") '' - Enabling both boot.enableContainers & virtualisation.containers on system.stateVersion < 22.05 is unsupported. - '') - ]; - unit = { description = "Container '%i'"; @@ -771,6 +765,11 @@ in serviceConfig = serviceDirectives dummyConfig; }; in { + warnings = + (optional (config.virtualisation.containers.enable && versionOlder config.system.stateVersion "22.05") '' + Enabling both boot.enableContainers & virtualisation.containers on system.stateVersion < 22.05 is unsupported. + ''); + systemd.targets.multi-user.wants = [ "machines.target" ]; systemd.services = listToAttrs (filter (x: x.value != null) ( -- cgit 1.4.1