about summary refs log tree commit diff
path: root/ops/gerrit-tvl/static/tvl.js
diff options
context:
space:
mode:
authorVincent Ambo <mail@tazj.in>2022-05-28T13·34+0200
committerclbot <clbot@tvl.fyi>2022-05-28T13·39+0000
commitaed1fbeb95c1790ed9faa46bfa410635fb8e8bb6 (patch)
treec1ac6047759df145cac112b7e63d53797bb9877e /ops/gerrit-tvl/static/tvl.js
parent6813598c17959862734e5878d745e7dd8a197717 (diff)
fix(gerrit-tvl): Use only one build filter r/4176
Buildkite can't handle more than one filter for the query; as of the
last commit it just returned an empty list.

I've verified with curl based on the request the previous attempt
constructed that this works as intended with only setting the commit.

Behaviour is probably undefined if there are two builds for the same
commit (i.e. a retry). Which one will you see? Who knows!

However, since the commit hash contains the Change-Id, we can't get a
situation where the build was for two different CLs at the same
commit. Gerrit wouldn't allow that.

Change-Id: I0dcd0ff44c28d3d15cba23461970bfc8483f4e48
Reviewed-on: https://cl.tvl.fyi/c/depot/+/5768
Autosubmit: tazjin <tazjin@tvl.su>
Tested-by: BuildkiteCI
Reviewed-by: sterni <sternenseemann@systemli.org>
Diffstat (limited to 'ops/gerrit-tvl/static/tvl.js')
-rw-r--r--ops/gerrit-tvl/static/tvl.js5
1 files changed, 1 insertions, 4 deletions
diff --git a/ops/gerrit-tvl/static/tvl.js b/ops/gerrit-tvl/static/tvl.js
index 51c3344c270f..684636de3084 100644
--- a/ops/gerrit-tvl/static/tvl.js
+++ b/ops/gerrit-tvl/static/tvl.js
@@ -79,11 +79,10 @@ function jobStateToCheckRunStatus(state) {
 
 const tvlChecksProvider = {
   async fetch(change) {
-    let {changeNumber, patchsetSha, repo} = change;
+    let {patchsetSha, repo} = change;
 
     const experiments = window.ENABLED_EXPERIMENTS || [];
     if (experiments.includes("UiFeature__tvl_check_debug")) {
-      changeNumber = 2872;
       patchsetSha = '76692104f58b849b1503a8d8a700298003fa7b5f';
       repo = 'depot';
     }
@@ -94,8 +93,6 @@ const tvlChecksProvider = {
     }
 
     const params = {
-      // besadii groups different patchsets of the same CL under this fake ref
-      branch: `cl/${changeNumber.toString()}`,
       commit: patchsetSha,
     };
     const url = `https://api.buildkite.com/v2/organizations/tvl/pipelines/depot/builds?${encodeParams(params)}`;