feat(ops/buildkite): Allow skip of individual steps
Using the `skip` attribute, it lets the user decide to temporarily skip a step. Change-Id: If63ce60a2b4148f041655912730acc4ae9e28534 Reviewed-on: https://cl.tvl.fyi/c/depot/+/5971 Tested-by: BuildkiteCI Reviewed-by: tazjin <tazjin@tvl.su>
This commit is contained in:
parent
c7d966c57b
commit
b9d79109dd
1 changed files with 4 additions and 2 deletions
|
@ -323,6 +323,7 @@ rec {
|
||||||
|
|
||||||
# TODO(tazjin): Turn into hard-failure after 2022-10-01.
|
# TODO(tazjin): Turn into hard-failure after 2022-10-01.
|
||||||
, postBuild ? null
|
, postBuild ? null
|
||||||
|
, skip ? false
|
||||||
}:
|
}:
|
||||||
let
|
let
|
||||||
parent = overridableParent parentOverride;
|
parent = overridableParent parentOverride;
|
||||||
|
@ -346,7 +347,8 @@ rec {
|
||||||
label
|
label
|
||||||
needsOutput
|
needsOutput
|
||||||
parent
|
parent
|
||||||
parentLabel;
|
parentLabel
|
||||||
|
skip;
|
||||||
|
|
||||||
# //nix/buildkite is growing a new feature for adding different
|
# //nix/buildkite is growing a new feature for adding different
|
||||||
# "build phases" which supersedes the previous `postBuild`
|
# "build phases" which supersedes the previous `postBuild`
|
||||||
|
@ -386,7 +388,7 @@ rec {
|
||||||
let
|
let
|
||||||
step = {
|
step = {
|
||||||
label = ":gear: ${cfg.label} (from ${cfg.parentLabel})";
|
label = ":gear: ${cfg.label} (from ${cfg.parentLabel})";
|
||||||
skip = if cfg.alwaysRun then false else cfg.parent.skip or false;
|
skip = if cfg.alwaysRun then false else cfg.skip or cfg.parent.skip or false;
|
||||||
|
|
||||||
depends_on = lib.optional
|
depends_on = lib.optional
|
||||||
(buildEnabled && !cfg.alwaysRun && !cfg.needsOutput)
|
(buildEnabled && !cfg.alwaysRun && !cfg.needsOutput)
|
||||||
|
|
Loading…
Reference in a new issue