about summary refs log tree commit diff
path: root/ops/pipelines/depot.nix
diff options
context:
space:
mode:
authorVincent Ambo <mail@tazj.in>2021-08-29T12·25+0300
committertazjin <mail@tazj.in>2021-08-29T12·37+0000
commit43269730e66c9a95467f8699a8b18fce8fef4a79 (patch)
tree2333783d9123f478617d8d21767603baacd70445 /ops/pipelines/depot.nix
parentfe225d48a18f722c03e31fdeb9c6c6c4ac6bb9d8 (diff)
refactor(ops/pipelines): Move failure status zeroing to setup r/2799
We changed the configured pipeline in Buildkite to upload
`static-pipeline.yaml` instead of containing the steps of that
pipeline itself.

This makes it easier to test changes to builds and such, but adds
another build step with scheduling overhead etc.

However - we can work around this by killing one of the existing build
steps. There's no reason the failure status zeroing (required for
status reporting) shouldn't be part of the pipeline setup, so I've
moved it there instead and nuked that step.

This should mean that the pipeline is configurable from within the
repo, but without slowing anything down.

Change-Id: I206ecc02647de42a461e33c02879ab84daf5ed2b
Reviewed-on: https://cl.tvl.fyi/c/depot/+/3461
Tested-by: BuildkiteCI
Reviewed-by: sterni <sternenseemann@systemli.org>
Diffstat (limited to 'ops/pipelines/depot.nix')
-rw-r--r--ops/pipelines/depot.nix11
1 files changed, 1 insertions, 10 deletions
diff --git a/ops/pipelines/depot.nix b/ops/pipelines/depot.nix
index 709ccc9d5f83..6d4b69d5b086 100644
--- a/ops/pipelines/depot.nix
+++ b/ops/pipelines/depot.nix
@@ -75,17 +75,8 @@ let
   #
   # Pipeline steps need to stay in order.
   pipeline.steps =
-    # Zero the failure status
-    [
-      {
-        command = "buildkite-agent meta-data set 'failure' '0'";
-        label = ":buildkite:";
-      }
-      { wait = null; }
-    ]
-
     # Create build steps for each CI target
-    ++ (map mkStep depot.ci.targets)
+    (map mkStep depot.ci.targets)
 
     ++ [
       # Simultaneously run protobuf checks