commit c4a7356a469b3b6267965d7bcc318cf35cdd3b62
author Andrii Shyshkalov <tandrii@chromium.org> Tue Sep 25 18:40:17 2018 +0000
committer Commit Bot <commit-bot@chromium.org> Tue Sep 25 18:40:17 2018 +0000
tree b2536427352944f474bc6195d927fdf01d3b3078
parent 8e64369342f22b0541d438bac748c93fe4b8d14e

git cl: cache also LABELS data on a change.

This removes 1 RPC when tbr-ing second or later patchset.

Since tests cover TBR only in first upload, there are
no test changes. Since adding test is real PITA here,
keep them as is.

R=ehmaldonado

Change-Id: I90801b06727ce2131a703d6cd6c24ffad798bc8a
Reviewed-on: https://chromium-review.googlesource.com/1242212
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

git_cl.py

diff --git a/git_cl.py b/git_cl.py
index 5a66386..b9d8acf 100755
--- a/git_cl.py
+++ b/git_cl.py
@@ -2452,7 +2452,7 @@
     # Warm change details cache now to avoid RPCs later, reducing latency for
     # developers.
     self._GetChangeDetail(
-        ['DETAILED_ACCOUNTS', 'CURRENT_REVISION', 'CURRENT_COMMIT'])
+        ['DETAILED_ACCOUNTS', 'CURRENT_REVISION', 'CURRENT_COMMIT', 'LABELS'])
 
     status = self._GetChangeDetail()['status']
     if status in ('MERGED', 'ABANDONED'):