From 1094e356df316e28d8b7ee8ef9844d0ec7853872 Mon Sep 17 00:00:00 2001 From: Luke Granger-Brown Date: Sat, 1 Aug 2020 14:30:25 +0000 Subject: fix(besadii): Don't add people back to attention set. This uses the not very documented ignore_default_attention_set_rules review API. Change-Id: I650777bbbd24a1922f26967fbbd7da06d14b6782 Reviewed-on: https://cl.tvl.fyi/c/depot/+/1513 Tested-by: BuildkiteCI Reviewed-by: tazjin --- ops/besadii/main.go2 | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/ops/besadii/main.go2 b/ops/besadii/main.go2 index 27dd76b42f..998c677010 100644 --- a/ops/besadii/main.go2 +++ b/ops/besadii/main.go2 @@ -223,9 +223,10 @@ func refUpdatedMain() { // // https://gerrit-review.googlesource.com/Documentation/rest-api-changes.html#review-input type reviewInput struct { - Message string `json:"message"` - Labels map[string]int `json:"labels"` - OmitDuplicateComments bool `json:"omit_duplicate_comments"` + Message string `json:"message"` + Labels map[string]int `json:"labels"` + OmitDuplicateComments bool `json:"omit_duplicate_comments"` + IgnoreDefaultAttentionSetRules bool `json:"ignore_default_attention_set_rules"` } func postCommandMain() { @@ -269,6 +270,9 @@ func postCommandMain() { Labels: map[string]int{ "Verified": verified, }, + + // Update the attention set if we are failing this patchset. + IgnoreDefaultAttentionSetRules: verified == 1, } body, _ := json.Marshal(review) -- cgit 1.4.1