Commits · 2b71832f6d8dc74119589992836cf95aeb8a9842 · Linshizhi / Chromium_Depot_tools

24 Oct, 2018 1 commit

Hide standard output from the browser when running git-cl-web · 2b71832f

Sergiy Byelozyorov authored Oct 24, 2018

R=nodir@chromium.org

Change-Id: I0816a109bc50cbc5c06339b832748f03abee372a
Reviewed-on: https://chromium-review.googlesource.com/c/1296511
Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org>
Reviewed-by: Nodir Turakulov <nodir@chromium.org>

2b71832f

23 Oct, 2018 2 commits

Roll recipe dependencies (trivial). · 879d5e37

recipe-roller authored Oct 23, 2018

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).


More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug.
recipe_engine:
  https://crrev.com/04620158d8f45c68cc8bc39e30ce4a85c368046a [archive] Add extended-length path support for windows. (iannucci@chromium.org)


TBR=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: Ic26a7c96c2eda21b232e11c407a198fe928568e7
Reviewed-on: https://chromium-review.googlesource.com/c/1297194Reviewed-by: Recipe Roller <recipe-roller@chromium.org>
Commit-Queue: Recipe Roller <recipe-roller@chromium.org>

879d5e37

metrics: Add function to extract metrics from HTTP requests · 03d6d118

Edward Lemur authored Oct 23, 2018

Bug: 897394
Change-Id: I92d2514e4347f02dbcf2884bef3a073095ee03a3
Reviewed-on: https://chromium-review.googlesource.com/c/1292242
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

03d6d118

22 Oct, 2018 1 commit

metrics: Add function to collect repeated metrics. · 149834e6

Edward Lemur authored Oct 22, 2018

Bug: 897394
Change-Id: Iece4507c7bf92d7a8aad045d31b4f06e54b3af15
Reviewed-on: https://chromium-review.googlesource.com/c/1292243Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>

149834e6

19 Oct, 2018 1 commit

[git] Stop using git-retry wrapper on LUCI. · 98f1e59b

Robert Iannucci authored Oct 19, 2018

The LUCI environment alreadly does `git retry` style retries by virtue
of it's Go git-wrapper program. No need to put an extra python
interpreter in the middle.

R=hinoka@chromium.org, nodir@chromium.org

Recipe-Nontrivial-Roll: infra
Recipe-Nontrivial-Roll: build
Change-Id: I2918d7f413dde667fccd45c83ad6f4b96c2afe2e
Reviewed-on: https://chromium-review.googlesource.com/c/1292236
Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
Reviewed-by: Ryan Tseng <hinoka@chromium.org>

98f1e59b

18 Oct, 2018 7 commits

git-cl: Don't report metrics for not-known repos. · 93277a7f

Edward Lesmes authored Oct 18, 2018

It is not recorded by the server, but causes a 500 status anyway.

Bug: 896868
Change-Id: Ic3bf0d989bf3c3641dae9ea49a58ffb28f52e865
Reviewed-on: https://chromium-review.googlesource.com/c/1289769Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>

93277a7f

[roll-dep] Use Bug: instead of BUG= · c1e6594d

Robert Iannucci authored Oct 18, 2018

This is the more modern syntax and everything should already work with
it.

R=tandrii@chromium.org

Change-Id: I7acbe6cc7c663e03e9a72c2b6c155a7982477146
Reviewed-on: https://chromium-review.googlesource.com/c/1289233
Auto-Submit: Robbie Iannucci <iannucci@chromium.org>
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

c1e6594d

Support target_arch for WindowsSDK · e660e3fb

Scott Graham authored Oct 18, 2018

Bug: chromium:892712
Change-Id: I68e1822a9a7205aaab3836be3f57a2f1feb1c9a6
Reviewed-on: https://chromium-review.googlesource.com/c/1289231
Commit-Queue: Scott Graham <scottmg@chromium.org>
Auto-Submit: Scott Graham <scottmg@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>

e660e3fb

metrics: Add a mechanism to notify users when we want to collect additional metrics. · 48836262

Edward Lemur authored Oct 18, 2018

When we change the version number in metrics_utils:

If the user is not a Googler, or has opted out explicitly, nothing happens
and we still don't collect metrics.

If we're collecting metrics from the user, we stop collecting metrics
and display a notice telling them what has changed.
That notice will be displayed ten times, after which we will
resume collecting metrics. A notice telling them we're collecting metrics
will still be displayed.

Bug: None
Change-Id: If1cc12b2fc06f0d6237714c4f182367b1afdf9fb
Reviewed-on: https://chromium-review.googlesource.com/c/1285395
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

48836262

git cl land: remove Rietveld-only options. · 3e002fb8

Andrii Shyshkalov authored Oct 18, 2018

R=ehmaldonado

Bug: 770408
Change-Id: I6f460902e3282451f5761bebf176d180a8f88d26
Reviewed-on: https://chromium-review.googlesource.com/c/1279141
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

3e002fb8

Reland "git cl: delete no longer used Rietveld methods." · 51bdf8c8

Andrii Shyshkalov authored Oct 18, 2018

This is a reland of 38d36a61
but now
 $ git cl status
 $ git cl archive
actually work.

Original change's description:
> git cl: delete no longer used Rietveld methods.
>
> R=ehmaldonado
>
> Bug: 770408
> Change-Id: I4dc034839180a7587ee368793547c5b74c2572f6
> Reviewed-on: https://chromium-review.googlesource.com/c/1279139
> Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
> Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

Bug: 770408, 895784
Change-Id: Id1dbf31dc19d693f1c4636c9558e1ac99fe6a77b
Reviewed-on: https://chromium-review.googlesource.com/c/1287245
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

51bdf8c8

Use lowercase machine architecture. · 4ac892e4

Edward Lemur authored Oct 18, 2018

Bug: 896355
Change-Id: I83a3c27478029d90206b5bb26e26797df47a90f6
Reviewed-on: https://chromium-review.googlesource.com/c/1287244Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>

4ac892e4

17 Oct, 2018 2 commits

bot_update: stop failing with a KeyError in the recipe on failure. · 13339332

John Budorick authored Oct 17, 2018

This has been low-key driving me bananas for a while, and I've seen
it confuse sheriffs etc many times.

Change-Id: If46ab55821447a3a70a3f0ea774c017f29fc7400
Reviewed-on: https://chromium-review.googlesource.com/c/1286819Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: John Budorick <jbudorick@chromium.org>

13339332

Set unique build ID. · 08faab99

Yoshisato Yanagisawa authored Oct 17, 2018

For ease of distinguishing a series of build from others in Goma,
let me make autoninja set unique build ID.
The ID can be usable for statistics purpose to see how much time is
usually spent for each build.

Since the unique ID is not specific to Goma, i.e. anybody invoked by
ninja can use it to distinguish builds, I chose AUTONINJA_BUILD_ID.
Since we do not need to distinguish each node, I chose uuid4.

Bug: b/77176746
Change-Id: I0b6a67811b420c51d1fd3fd113bf7f039a41e2ab
Reviewed-on: https://chromium-review.googlesource.com/c/1275685Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Fumitoshi Ukai <ukai@chromium.org>
Commit-Queue: Yoshisato Yanagisawa <yyanagisawa@chromium.org>

08faab99

16 Oct, 2018 10 commits

Add instructions to check for metrics collection. · 63517c85

Andy Perelson authored Oct 16, 2018

Change-Id: Ie747dcf08692ec2d5c077ca70c97391b8d63787e
Reviewed-on: https://chromium-review.googlesource.com/c/1285390Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
Commit-Queue: Andy Perelson <ajp@chromium.org>

63517c85

git-cl: Log response IP and host for Gerrit RPC failures. · b0b43f3a

Edward Lemur authored Oct 16, 2018

Bug: 881860
Change-Id: I32a8c27ba912bf76feda86940c7f30a5462f527f
Reviewed-on: https://chromium-review.googlesource.com/c/1279128Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>

b0b43f3a

Revert "git cl: delete no longer used Rietveld methods." · 1e488131

Andrii Shyshkalov authored Oct 16, 2018

This reverts commit 38d36a61.

Reason for revert: breaks git cl archive for old rietveld branches.

Original change's description:
> git cl: delete no longer used Rietveld methods.
> 
> R=ehmaldonado
> 
> Bug: 770408
> Change-Id: I4dc034839180a7587ee368793547c5b74c2572f6
> Reviewed-on: https://chromium-review.googlesource.com/c/1279139
> Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
> Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

TBR=tandrii@chromium.org,ehmaldonado@chromium.org

Change-Id: Ifff7b5e13b104841ecf2c190b043ad23f9dbcccd
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 770408
Reviewed-on: https://chromium-review.googlesource.com/c/1284990Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

1e488131

Roll recipe dependencies (trivial). · 74b70246

recipe-roller authored Oct 16, 2018

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).


More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug.
recipe_engine:
  https://crrev.com/f46a599f105746e2f0ce7068c630e312f2113893 [cipd] Do not assume that all packages have refs or tags. (iannucci@chromium.org)


TBR=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: Ic63aa46f0dd00a0c6bc3315bcbddb0ae871e57eb
Reviewed-on: https://chromium-review.googlesource.com/c/1283809Reviewed-by: Recipe Roller <recipe-roller@chromium.org>
Commit-Queue: Recipe Roller <recipe-roller@chromium.org>

74b70246

git cl try-results: delete Rietveld support. · c68a1753

Andrii Shyshkalov authored Oct 16, 2018

R=ehmaldonado

Bug: 770408
Change-Id: I94ff37f198937dc7d341a859a71becd91cf3a706
Reviewed-on: https://chromium-review.googlesource.com/c/1279140
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

c68a1753

git cl: delete no longer used Rietveld methods. · 38d36a61

Andrii Shyshkalov authored Oct 16, 2018

R=ehmaldonado

Bug: 770408
Change-Id: I4dc034839180a7587ee368793547c5b74c2572f6
Reviewed-on: https://chromium-review.googlesource.com/c/1279139
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

38d36a61

git cl: rename and relocate CreateDescriptionFromLog func. · b07575f6

Andrii Shyshkalov authored Oct 16, 2018

R=ehmaldonado

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

b07575f6

git cl description: rename gerrit test. · dd672fb3

Andrii Shyshkalov authored Oct 16, 2018

R=ehmaldonado

Bug: 770408
Change-Id: I211ad8e67e9486b974eed7694436afb15ecf21a2
Reviewed-on: https://chromium-review.googlesource.com/c/1279136
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

dd672fb3

git cl comments: remove Rietveld support. · 642641d0

Andrii Shyshkalov authored Oct 16, 2018

R=ehmaldonado

Bug: 770408
Change-Id: Ib3120b5fd09c35be691c025d0d69ceda60f2ec7f
Reviewed-on: https://chromium-review.googlesource.com/c/1279135
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

642641d0

git cl: no longer support --rietveld flag when forcing codereview. · feec80e9

Andrii Shyshkalov authored Oct 16, 2018

R=ehmaldonado

Bug: 770408
Change-Id: I980487aacd115535d0ca855cd1edfcfc18fc5cbe
Reviewed-on: https://chromium-review.googlesource.com/c/1279138
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

feec80e9

15 Oct, 2018 7 commits

git cl upload: delete no longer supported flags. · 7d7bbc3b

Andrii Shyshkalov authored Oct 15, 2018

--email
--emulate-svn-auto-props

R=ehmaldonado

Bug: 770408
Change-Id: I645342e5735313055c713bc8f8b68eee353aba49
Reviewed-on: https://chromium-review.googlesource.com/c/1279134
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

7d7bbc3b

git cl: add reviewers and ccs to git push command if possible. · 2f727917

Andrii Shyshkalov authored Oct 15, 2018

Also fix a typo in ValidateEmail function which didn't support
email addresses with '-' in them, e.g., infra-dev@chromium.org.

R=ehmaldonado

Bug: 875089
Change-Id: I2d73c1473527c9bf62e25e9f88250196b783fcb0
Reviewed-on: https://chromium-review.googlesource.com/c/1242849
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

2f727917

git cl: partially remove Rietveld support for uploading code. · 9f274436

Andrii Shyshkalov authored Oct 15, 2018

Also deletes lots of tests which provided coverage for Rietveld only,
and hence no longer useful.

R=ehmaldonado

Bug: 770408
Change-Id: I31195f7819a52d1063ed28064a74fd70fbc39357
Reviewed-on: https://chromium-review.googlesource.com/c/1279133
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

9f274436

git cl: delete Rietveld support for tryjobs. · 03da1506

Andrii Shyshkalov authored Oct 15, 2018

R=ehmaldonado

Bug: 770408
Change-Id: Id070e6084b6beba7309bdc35f0f750decd77e0c9
Reviewed-on: https://chromium-review.googlesource.com/c/1279132
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

03da1506

git cl: start deleting Rietveld support. · f5569d26

Andrii Shyshkalov authored Oct 15, 2018

R=ehmaldonado

Bug: 770408
Change-Id: I5020596ada0c6fef2797fed6b78e7256a35ffc40
Reviewed-on: https://chromium-review.googlesource.com/c/1279131
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

f5569d26

git cl: find which reviewers and ccs are OK for Gerrit before upload. · ba7b0a46

Andrii Shyshkalov authored Oct 15, 2018

This adds O(reviewers+cc) RPC to Gerrit before upload,
though done concurrently in up to 10 threads. This information isn't
used yet, but will be in follow up CLs.

R=ehmaldonado

Bug: 877717
Change-Id: I2374f249ee874a71089244309e50e8e88a3dee7d
Reviewed-on: https://chromium-review.googlesource.com/c/1242847Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

ba7b0a46

git cl: compute reviewers and ccs before pushing change to Gerrit. · 76988a8c

Andrii Shyshkalov authored Oct 15, 2018

R=ehmaldonado

Bug: 877717
Change-Id: Iccfba594369d8c726fd9b077d3ac18046757e4a2
Reviewed-on: https://chromium-review.googlesource.com/c/1242845
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

76988a8c

13 Oct, 2018 1 commit

git cl: bring label setting closer to git push in Gerrit case. · dd788444

Andrii Shyshkalov authored Oct 13, 2018

R=ehmaldonado

Bug: 877717
Change-Id: I81e574068687576cc44e41ba00ba650de8aae31b
Reviewed-on: https://chromium-review.googlesource.com/c/1242844Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

dd788444

12 Oct, 2018 1 commit

git cl upload: also explicitly use eu1 mirror for retries. · 94faf328

Andrii Shyshkalov authored Oct 12, 2018

R=ehmaldonado

Bug: 881860
Change-Id: Ia02905a8b71486d771a691c31114d692e62a369f
Reviewed-on: https://chromium-review.googlesource.com/c/1277796Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

94faf328

11 Oct, 2018 2 commits

Roll recipe dependencies (trivial). · 066e1107

recipe-roller authored Oct 11, 2018

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).


More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug.
recipe_engine:
  https://crrev.com/dffa0b83f071c910d625955d6bb61335696e3a08 Add UUID recipe module (smut@google.com)


TBR=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: If0a3cdcf3cf05894b71f73aa15121bdf353193af
Reviewed-on: https://chromium-review.googlesource.com/c/1278325
Commit-Queue: Recipe Roller <recipe-roller@chromium.org>
Reviewed-by: Recipe Roller <recipe-roller@chromium.org>

066e1107

git-cl: Make request to upload logs more visible. · 47faa068

Edward Lemur authored Oct 11, 2018

Print the message asking the user to upload gerrit logs for git-cl upload
failures after the exception.

Also, initialize the error logger only for git-cl upload, so that the
headers are not overwritten unless git-cl upload is executed again.

Bug: 881860
Change-Id: I8e63432eecca5253e56096e3ad0218eec4058e12
Reviewed-on: https://chromium-review.googlesource.com/c/1277426
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

47faa068

10 Oct, 2018 2 commits

[git recipe_module] Add tags flag for fetch · 2fddb956

Ryan Tseng authored Oct 10, 2018

Some projects that use the git module (eg flutter) require tags.

Bug:  870558
Change-Id: Idae50ef5c8a092e5f0109b6c14c562268a2ad236
Reviewed-on: https://chromium-review.googlesource.com/c/1274245Reviewed-by: Jao-ke Chin-Lee <jchinlee@chromium.org>
Commit-Queue: Ryan Tseng <hinoka@chromium.org>

2fddb956

git-cl: Print response headers on 404 Gerrit RPC status. · 83bd7f4c

Edward Lemur authored Oct 10, 2018

And ask the user to report the printed headers to the bug.

Bug: 881860
Change-Id: I0a6cc0d90b4a721f72cd7eee4cba59934facbb1e
Reviewed-on: https://chromium-review.googlesource.com/c/1266837
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

83bd7f4c

09 Oct, 2018 2 commits

Update yapf version 0.22 -> 0.24 · db1e69ef

Aiden Benner authored Oct 09, 2018

0.24 has just been uploaded see bug.

Bug: 890349
Change-Id: Ice1e0c54d3fb60c0da5fde14aafae20c3ac449ce
Reviewed-on: https://chromium-review.googlesource.com/c/1261756Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Commit-Queue: Aiden Benner <abenner@google.com>

db1e69ef

[led] Roll led to correctly pick swarming host from buildbucket. · dce74762

Robert Iannucci authored Oct 09, 2018

TBR=tandrii@chromium.org, vadimsh@chromium.org

Change-Id: Ie976820701359bb4976d251b3de4f1077d2357ac
Reviewed-on: https://chromium-review.googlesource.com/c/1271304
Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>

dce74762

05 Oct, 2018 1 commit

Roll recipe dependencies (trivial). · 71e3be7a

recipe-roller authored Oct 05, 2018

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).


More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug.
recipe_engine:
  https://crrev.com/acbe9e3142610d958181460254565ef27ad47dc0 [StepPresentation] Remove all logs from memory during finalization. (iannucci@chromium.org)


TBR=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: Id65a77df9f8508f57a28aa77433b141b1a691c9f
Reviewed-on: https://chromium-review.googlesource.com/c/1264996Reviewed-by: Recipe Roller <recipe-roller@chromium.org>
Commit-Queue: Recipe Roller <recipe-roller@chromium.org>

71e3be7a