From 25d665634a1bd38515320beabf85a6e23545bac7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Niklas=20Hamb=C3=BCchen?= <mail@nh2.me>
Date: Thu, 18 Jun 2020 01:49:24 +0200
Subject: [PATCH] consul.passthru.tests: Refactor: Extract variable

---
 nixos/tests/consul.nix | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/nixos/tests/consul.nix b/nixos/tests/consul.nix
index a5188d1283041..eb7dd45923fc4 100644
--- a/nixos/tests/consul.nix
+++ b/nixos/tests/consul.nix
@@ -55,6 +55,7 @@ let
 
   server = index: { pkgs, ... }:
     let
+      numConsensusServers = builtins.length allConsensusServerHosts;
       thisConsensusServerHost = builtins.elemAt allConsensusServerHosts index;
       ip = thisConsensusServerHost; # since we already use IPs to identify servers
     in
@@ -71,12 +72,12 @@ let
             inherit webUi;
             extraConfig = defaultExtraConfig // {
               server = true;
-              bootstrap_expect = builtins.length allConsensusServerHosts;
+              bootstrap_expect = numConsensusServers;
               retry_join =
                 # If there's only 1 node in the network, we allow self-join;
                 # otherwise, the node must not try to join itself, and join only the other servers.
                 # See https://github.com/hashicorp/consul/issues/2868
-                if builtins.length allConsensusServerHosts == 1
+                if numConsensusServers == 1
                   then allConsensusServerHosts
                   else builtins.filter (h: h != thisConsensusServerHost) allConsensusServerHosts;
               bind_addr = ip;