From fcf81f91a3a90c2ee009d180368d9ecdca178334 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bj=C3=B8rn=20Forsman?= Date: Tue, 26 Jul 2022 09:58:24 +0200 Subject: [PATCH] nixos/jenkins-job-builder: better defaults for accessUser/accessTokenFile The new defaults allows jenkins-job-builder to reload the configuration out-of-the-box, whereas the previous defaults required users to manually reload/restart jenkins, or configure accessUser/accessTokenFile themselves. (If `extraJavaOptions = [ "-Djenkins.install.runSetupWizard=false" ]` then the initial admin user is *not* created and you have to use JCasC or something else to bootstrap.) --- .../services/continuous-integration/jenkins/job-builder.nix | 5 +++-- nixos/tests/jenkins.nix | 2 -- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/nixos/modules/services/continuous-integration/jenkins/job-builder.nix b/nixos/modules/services/continuous-integration/jenkins/job-builder.nix index 8dc06bf26416..3a1c6c1a371d 100644 --- a/nixos/modules/services/continuous-integration/jenkins/job-builder.nix +++ b/nixos/modules/services/continuous-integration/jenkins/job-builder.nix @@ -30,7 +30,7 @@ in { }; accessUser = mkOption { - default = ""; + default = "admin"; type = types.str; description = lib.mdDoc '' User id in Jenkins used to reload config. @@ -48,7 +48,8 @@ in { }; accessTokenFile = mkOption { - default = ""; + default = "${config.services.jenkins.home}/secrets/initialAdminPassword"; + defaultText = literalExpression ''"''${config.services.jenkins.home}/secrets/initialAdminPassword"''; type = types.str; example = "/run/keys/jenkins-job-builder-access-token"; description = lib.mdDoc '' diff --git a/nixos/tests/jenkins.nix b/nixos/tests/jenkins.nix index 3f111426db38..a1ede6dc917b 100644 --- a/nixos/tests/jenkins.nix +++ b/nixos/tests/jenkins.nix @@ -18,8 +18,6 @@ import ./make-test-python.nix ({ pkgs, ...} : { enable = true; jobBuilder = { enable = true; - accessUser = "admin"; - accessTokenFile = "/var/lib/jenkins/secrets/initialAdminPassword"; nixJobs = [ { job = { name = "job-1";