commit c7e281618a51165c41a7b462781d585ea22e26ad
author Mike Frysinger <vapier@chromium.org> Tue Feb 25 03:23:04 2020 -0500
committer Commit Bot <commit-bot@chromium.org> Wed Feb 26 00:58:31 2020 +0000
tree d257d96e25e64f01f299f44389e0204364791c38
parent b99f86c876351572d3dfdcd22a875c9b4c473a8e

gerrit: plumb notify support to add/remove reviewers

We have to switch the remove API to use POST so we can include a body.

BUG=None
TEST=adding & removing user to a CL works

Change-Id: I57352ffb44856b0b460a403c00f24efab8eb6f68
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/2071322
Reviewed-by: Michael Mortensen <mmortensen@google.com>
Tested-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Mike Frysinger <vapier@chromium.org>

scripts/gerrit.py

diff --git a/scripts/gerrit.py b/scripts/gerrit.py
index 37c3a41..cfe8321 100644
--- a/scripts/gerrit.py
+++ b/scripts/gerrit.py
@@ -471,7 +471,7 @@
   if add_list or remove_list:
     helper, cl = GetGerrit(opts, cl)
     helper.SetReviewers(cl, add=add_list, remove=remove_list,
-                        dryrun=opts.dryrun)
+                        dryrun=opts.dryrun, notify=opts.notify)
 UserActReviewers.usage = '<CL> <emails...>'