From 890ffaa1aed7e5a36dd19ba1a047daa4d66f410b Mon Sep 17 00:00:00 2001 From: Tom Siewert Date: Wed, 10 Jan 2024 11:36:36 +0100 Subject: [PATCH] nixos/gitlab: fix database config when no passwordfile is defined In commit 41ca9c1, the database config got an additional config entry for a separate CI database connection [1]. Unfortunately, the main connection must always be the first entry. The fix for this was only applied when databasePasswordFile has been set. This commit fixes the order of the config for deployments without a databasePasswordFile. [1] https://docs.gitlab.com/ee/update/deprecations.html#single-database-connection-is-deprecated --- nixos/modules/services/misc/gitlab.nix | 1 + 1 file changed, 1 insertion(+) diff --git a/nixos/modules/services/misc/gitlab.nix b/nixos/modules/services/misc/gitlab.nix index ca6910b795d8..d8e4aab4feea 100644 --- a/nixos/modules/services/misc/gitlab.nix +++ b/nixos/modules/services/misc/gitlab.nix @@ -1359,6 +1359,7 @@ in { '' else '' jq <${pkgs.writeText "database.yml" (builtins.toJSON databaseConfig)} \ + '${if lib.versionAtLeast (lib.getVersion cfg.packages.gitlab) "15.9" then ".production.main as $main | del(.production.main) | .production |= {main: $main} + ." else ""}' \ >'${cfg.statePath}/config/database.yml' '' }