commit c1bbdfd7c455d5f940d7907201c35d80a92d7ccc
author Mike Frysinger <vapier@chromium.org> Thu Feb 06 23:54:48 2020 -0500
committer Commit Bot <commit-bot@chromium.org> Sat Feb 08 06:59:17 2020 +0000
tree 39d77e292503e36cd56ecab822f998a72c870000
parent b59a427549b1b723ca64c0f2897bff2ffb5919e4

gerrit: add dryrun for private/public helper

This fixes -n handling with `gerrit private ...`.

BUG=None
TEST=`gerrit -n private 12345 yes` no longer attempts to update the CL

Change-Id: If3734eb22f5c67a51eb265ac5bc0552270289259
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/2044495
Reviewed-by: Alex Klein <saklein@chromium.org>
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 9ca4788..84a26d8 100644
--- a/scripts/gerrit.py
+++ b/scripts/gerrit.py
@@ -502,7 +502,7 @@
     raise RuntimeError('Unknown "boolean" value: %s' % private_str)
 
   helper, cl = GetGerrit(opts, cl)
-  helper.SetPrivate(cl, private)
+  helper.SetPrivate(cl, private, dryrun=opts.dryrun)
 UserActPrivate.usage = '<CL> <private str>'