commit d8bf20269cd088ce5df75d0c54cd25461ffda9ef
author Guenter Roeck <groeck@chromium.org> Tue Jan 19 07:31:54 2021 -0800
committer Commit Bot <commit-bot@chromium.org> Tue Jan 19 20:01:08 2021 +0000
tree 60466a1bbbdf08a745c1afc68a19a990dd719ccd
parent d1aaede4778f67fa4124e9234d951eb518070187

findmissing: getopen: Fix get_change_id() call

Since commit 7f5b4fc66e ("findmissing: Do not push already abandoned
changes into Gerrit"), get_change_id has a branch parameter. Also,
it now returns more than one value.

BUG=b:176923589
TEST=Run robot

Change-Id: Ia09062ed2c4ce5afecaf4a9cf774548903e31024
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/dev-util/+/2637917
Reviewed-by: Curtis Malainey <cujomalainey@chromium.org>
Reviewed-by: Guenter Roeck <groeck@chromium.org>
Commit-Queue: Guenter Roeck <groeck@chromium.org>
Tested-by: Guenter Roeck <groeck@chromium.org>

contrib/findmissing/getopen.py

diff --git a/contrib/findmissing/getopen.py b/contrib/findmissing/getopen.py
index 25a2002..24b30bf 100755
--- a/contrib/findmissing/getopen.py
+++ b/contrib/findmissing/getopen.py
@@ -160,7 +160,7 @@
 
         print('Upstream commit %s ("%s")' % (fixedby_sha, fixedby_description))
 
-        changeid = missing.get_change_id(db, fixedby_sha)
+        changeid, _ = missing.get_change_id(db, branch, fixedby_sha)
         if changeid:
             print('  CL:%s' % changeid)