commit 8ac8e0936b8ab6cbb96aa2c3a7304ef86bba48b6
author Patrick Williams <patrick@stwcx.xyz> Tue Jan 04 12:05:50 2022 -0600
committer Patrick Williams <patrick@stwcx.xyz> Tue Jan 04 13:48:48 2022 -0600
tree 8753cfc772724577241a3ff5dadef0bd0b463a5a
parent f8ee6f079c8191f337e732e634cc90a17498ab2f

userid-validation: reduce email noise from Jenkins

Jenkins userid-validation review causes a lot of low value emails.
Leverage the `--notify=NONE` option on the typical review indications
to reduce these emails.

```
 --notify (-n) [NONE | OWNER |          : Who to send email notifications to
 OWNER_REVIEWERS | ALL]                   after the review is stored.
```

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I9ae596ff7d297ddfe04beb024132847c3d0d5a64

jenkins/userid-validation

diff --git a/jenkins/userid-validation b/jenkins/userid-validation
index 026b877..1c24aaa 100755
--- a/jenkins/userid-validation
+++ b/jenkins/userid-validation
@@ -46,7 +46,7 @@
 # Reset the vote to 0 so jenkins will detect a new +1 on retriggers
 "${GERRIT_SSH_CMD[@]}" review \
     "${GERRIT_CHANGE_NUMBER},${GERRIT_PATCHSET_NUMBER}" \
-    --ok-to-test=0 -t autogenerated:jenkins
+    --ok-to-test=0 -t autogenerated:jenkins --notify=NONE
 
 # Write full list of users to a file
 GERRIT_CI_GROUPS=( \
@@ -93,13 +93,13 @@
 if grep -q -w "${COMMITTER_USERNAME}" "$WORKSPACE/users.txt"; then
     "${GERRIT_SSH_CMD[@]}" review \
         "${GERRIT_CHANGE_NUMBER},${GERRIT_PATCHSET_NUMBER}" \
-        --ok-to-test=1 -t autogenerated:jenkins \
+        --ok-to-test=1 -t autogenerated:jenkins --notify=NONE \
         "--message='User approved, CI ok to start'"
 
     # Immediately erase the score to prevent infinite triggers.
     "${GERRIT_SSH_CMD[@]}" review \
         "${GERRIT_CHANGE_NUMBER},${GERRIT_PATCHSET_NUMBER}" \
-        --ok-to-test=0 -t autogenerated:jenkins
+        --ok-to-test=0 -t autogenerated:jenkins --notify=NONE
 
     exit 0
 fi