From a4b3f9af9364d6e39a0f93ddbb5135dbff056b09 Mon Sep 17 00:00:00 2001 From: Vincent Ambo Date: Sun, 14 Jun 2020 18:58:53 +0100 Subject: [PATCH] fix(3p/gerrit): Fix Gerrit derivation name and module configuration Without these changes, the NixOS module isn't able to use the new Gerrit derivation. These changes are already deployed as I needed to make them to get Gerrit back up. Change-Id: Iad3aa6158789a014134fddccd40b508b81486100 Reviewed-on: https://cl.tvl.fyi/c/depot/+/301 Reviewed-by: lukegb --- ops/nixos/monorepo-gerrit.nix | 2 ++ third_party/gerrit/default.nix | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/ops/nixos/monorepo-gerrit.nix b/ops/nixos/monorepo-gerrit.nix index 58fbb8d20..add4b18c7 100644 --- a/ops/nixos/monorepo-gerrit.nix +++ b/ops/nixos/monorepo-gerrit.nix @@ -22,6 +22,8 @@ in { owners-autoassign ]; + package = config.depot.third_party.gerrit; + settings = { core.packedGitLimit = "100m"; log.jsonLogging = true; diff --git a/third_party/gerrit/default.nix b/third_party/gerrit/default.nix index c7ab47cd6..98373582b 100644 --- a/third_party/gerrit/default.nix +++ b/third_party/gerrit/default.nix @@ -31,7 +31,7 @@ let version = "3.2.1"; in pkgs.buildBazelPackage { - name = "gerrit"; + name = "gerrit-${version}"; src = pkgs.fetchgit { url = "https://gerrit.googlesource.com/gerrit";