3
0
Fork 0
forked from mirrors/nixpkgs

nixos/yggdrasil: fix for configFile option

The configFile was not being merged with the declarative configuration at
runtime.
This commit is contained in:
Emery Hemingway 2019-12-01 12:32:23 +01:00 committed by Lassulus
parent fdca231242
commit 6c1c99d6b4

View file

@ -12,11 +12,11 @@ let
configFileProvided = (cfg.configFile != null);
generateConfig = (
if configProvided && configFileProvided then
"${pkgs.jq}/bin/jq -s add /run/yggdrasil/configFile.json ${configAsFile}"
"${pkgs.jq}/bin/jq -s add ${configAsFile} ${cfg.configFile}"
else if configProvided then
"cat ${configAsFile}"
else if configFileProvided then
"cat /run/yggdrasil/configFile.json"
"cat ${cfg.configFile}"
else
"${cfg.package}/bin/yggdrasil -genconf"
);
@ -147,7 +147,7 @@ in {
RuntimeDirectory = "yggdrasil";
RuntimeDirectoryMode = "0700";
BindReadOnlyPaths = mkIf configFileProvided
[ "${cfg.configFile}:/run/yggdrasil/configFile.json" ];
[ "${cfg.configFile}" ];
# TODO: as of yggdrasil 0.3.8 and systemd 243, yggdrasil fails
# to set up the network adapter when DynamicUser is set. See