about summary refs log tree commit diff
path: root/nix/buildkite/default.nix
diff options
context:
space:
mode:
authorEvgeny Zemtsov <eze@resoptima.com>2022-10-10T11·20+0200
committerezemtsov <eugene.zemtsov@gmail.com>2022-10-11T09·17+0000
commitb319e008313d68f2639de0f656d346ed9a823e41 (patch)
tree8ca86426dddf61f2d7c8b4ad01220093cf9e5885 /nix/buildkite/default.nix
parent1181bd78fc7d41a4786a0355244dbdee327560a7 (diff)
fix(nix/buildkite): split extra steps based on all known phases r/5104
Fix a bug introduced by 2ca153141 (I merged the wrong patchset).

Issue happens when pipeline is split by phases into independent
evals (e.g. build/release/deploy).

Splitting extra steps requires knowledge of all known phases,
otherwise pipeline evaluation fails due to extra steps from inactive
phases.

Change-Id: Iab0f2dc3eadda281e483055e26f00a95442e15b0
Reviewed-on: https://cl.tvl.fyi/c/depot/+/6942
Tested-by: BuildkiteCI
Reviewed-by: tazjin <tazjin@tvl.su>
Diffstat (limited to 'nix/buildkite/default.nix')
-rw-r--r--nix/buildkite/default.nix2
1 files changed, 1 insertions, 1 deletions
diff --git a/nix/buildkite/default.nix b/nix/buildkite/default.nix
index 834e4f7a42aa..f0f760b3ceb1 100644
--- a/nix/buildkite/default.nix
+++ b/nix/buildkite/default.nix
@@ -197,7 +197,7 @@ rec {
 
           # Split extra steps by phase.
           splitExtraSteps = lib.groupBy ({ phase, ... }: phase)
-            (attrValues (mapAttrs (normaliseExtraStep enabledPhases overridable)
+            (attrValues (mapAttrs (normaliseExtraStep phases overridable)
               (target.meta.ci.extraSteps or { })));
 
           extraSteps = mapAttrs