Commit Graph

7780 Commits (d548d5a90ec23a801ea001be59e16a64b836a816)
 

Author SHA1 Message Date
recipe-roller d548d5a90e Roll recipe dependencies (trivial).
This is an automated CL created by the recipe roller. This CL rolls
recipe changes from upstream projects (recipe_engine) into this repository.

recipe_engine:
bbbf2ecbd0
  bbbf2ec (olivernewman@google.com)
      [tricium] Fix pylint analyzer executable name

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug.

R=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I4f9b1505d0dfce4289bed1d327ff996dc3281745
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2827152
Commit-Queue: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
Bot-Commit: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
4 years ago
Mirko Bonadei ea4e399c4d Switch gn wrapper scripts to invoke python3.
Bug: 941669
Change-Id: I515459793f2a406a959c361f9884b183e21ebdf7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2825439
Reviewed-by: Dirk Pranke <dpranke@google.com>
Commit-Queue: Mirko Bonadei <mbonadei@chromium.org>
4 years ago
Owen Rodley 3cee7aff78 [vpython] Update to 58604ec32e265
This contains two CLs of interest:
* https://crrev.com/c/2807199, which strips the 'm' ABI tag and fixes
  https://crbug.com/1187476
* https://crrev.com/c/2820342, which generalises the Python 3 tag logic
  to more than just 3.8

Bug: 1187476,1176014
Change-Id: I2138e9a7b2147c8c72cb99daa08642919a30708c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2823254
Auto-Submit: Owen Rodley <orodley@chromium.org>
Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
4 years ago
recipe-roller f9d141ab4d Roll recipe dependencies (trivial).
This is an automated CL created by the recipe roller. This CL rolls
recipe changes from upstream projects (recipe_engine) into this repository.

recipe_engine:
7c1ba3120b
  7c1ba31 (weiweilin@google.com)
      [recipe] support configuring test presentation in MiloApi.

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug.

R=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I22f39bf7f8d263fc3704c55e1a864dd339fc5308
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2824176
Commit-Queue: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
Bot-Commit: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
4 years ago
Josip Sokcevic 3d9cab4fa8 Increase bot_update http low speed time
Windows builders have issues fetching from chromium/src, and increasing
timeout may help - recommended by git team.

Bug: 1198581
Recipe-Nontrivial-Roll: chromiumos
Recipe-Nontrivial-Roll: infra
Recipe-Nontrivial-Roll: build_limited_scripts_slave
Recipe-Nontrivial-Roll: chrome_release
Recipe-Nontrivial-Roll: build
Change-Id: Ic650dea2a175e9642a6d3d5b8f1c54fa9979e12a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2823020
Commit-Queue: Josip Sokcevic <sokcevic@google.com>
Auto-Submit: Josip Sokcevic <sokcevic@google.com>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
4 years ago
Josip Sokcevic d73d069a76 Run yapf with vpython3
R=ehmaldonado@chromium.org

Bug: 1197346
Change-Id: Ifc56ff316954eaa3c532b212371de57f8884ff97
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2822446
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
Commit-Queue: Josip Sokcevic <sokcevic@google.com>
4 years ago
Fumitoshi Ukai 9955936084 Roll out new goma client binary VERSION=217
disable ATS on Windows by default.
fix http_proxy in mac-arm64

Bug: b/184911357, b/183118231, b/183895446
Change-Id: I88fcfa6cbbb49a104525e9b9612a1a8790f4d6a6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2823397
Reviewed-by: Junji Watanabe <jwata@google.com>
Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
Reviewed-by: Yoshisato Yanagisawa <yyanagisawa@google.com>
Commit-Queue: Fumitoshi Ukai <ukai@google.com>
4 years ago
Haiyang Pan 057831ef1f Recipe: Add timeout for fetching gerrit commit message in tryserver API
In crbug.com/1196159, a build got stuck at "gerrit changes" step for 3
hours. It would be good to add a timeout so that it fails early.

Also adjust the timeout for get_changes to 60s, from 600s

Bug: 1196159
Change-Id: Id2327ab40dd2fd8d95aaad6bb724f7ed06e6c244
Recipe-Nontrivial-Roll: infra
Recipe-Nontrivial-Roll: chromiumos
Recipe-Nontrivial-Roll: build_limited_scripts_slave
Recipe-Nontrivial-Roll: build
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2811899
Commit-Queue: Haiyang Pan <hypan@google.com>
Reviewed-by: Andrii Shyshkalov <tandrii@google.com>
4 years ago
Robert Iannucci b2c07b2300 [led] Update led.
This picks up a change to make 'debug sleep' work on luciexe jobs.

R=yiwzhang

Bug: 1197300
Change-Id: Iea7a061e84f7011b6a7f1cc64cca34b0267b6ec1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2818885
Auto-Submit: Robbie Iannucci <iannucci@chromium.org>
Commit-Queue: Scott Lee <ddoman@chromium.org>
Reviewed-by: Scott Lee <ddoman@chromium.org>
4 years ago
Chan Li 5d9af91928 [dirmd] roll dirmd
To include https://chromium-review.googlesource.com/c/infra/infra/+/2815945

Change-Id: I74dcb46948700b02bffb867fd1ff51baa3979d31
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2818896
Auto-Submit: Chan Li <chanli@chromium.org>
Reviewed-by: Nodir Turakulov <nodir@chromium.org>
Commit-Queue: Nodir Turakulov <nodir@chromium.org>
4 years ago
Michael Achenbach c53cbb415e [recipes] Remove wasm_llvm config
The config was only used in a recipe removed here:
https://crrev.com/c/2814607

Bug: v8:11622
Change-Id: Ie22e26ca0b833afc27459de370ff7a918ad89c29
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2814739
Reviewed-by: Derek Schuff <dschuff@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@google.com>
Commit-Queue: Michael Achenbach <machenbach@chromium.org>
4 years ago
Fumitoshi Ukai 00feb941d4 Roll out new goma client binary VERSION=216
Add mac-arm64 platform to provide goma client
on fully supported platforms

Bug: b/182514677, b/183118231
Change-Id: Iae7581883d403a1a28f7d0bb1b871c9cfd0f7e65
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2816481
Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
Reviewed-by: Yoshisato Yanagisawa <yyanagisawa@google.com>
Commit-Queue: Fumitoshi Ukai <ukai@google.com>
4 years ago
Josip Sokcevic 1cabb17575 Set branchref on Changelist if set by user
R=gavinmak@google.com

Bug: 1067814
Change-Id: I592407cd489ade8bffc5158a86ccb061064d3ddd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2815397
Auto-Submit: Josip Sokcevic <sokcevic@google.com>
Commit-Queue: Gavin Mak <gavinmak@google.com>
Reviewed-by: Gavin Mak <gavinmak@google.com>
4 years ago
recipe-roller 98a52e2e31 Roll recipe dependencies (trivial).
This is an automated CL created by the recipe roller. This CL rolls
recipe changes from upstream projects (recipe_engine) into this repository.

recipe_engine:
91f81ea469
  91f81ea (olivernewman@google.com)
      [tricium] Add `run_legacy` option to skip emitting comments

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug.

R=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I2c0827476d48af10bdc44c707dce996215262af8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2812318
Commit-Queue: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
Bot-Commit: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
4 years ago
Mike Frysinger f78dd18ee1 CROS_OWNERS: add more build peeps
Change-Id: I15763fa3b1a42904eb4434f4ca72b6ad3bd0edb8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2812198
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Stephane Belmon <sbelmon@google.com>
4 years ago
Josip Sokcevic e1a98941d3 Add mass-abandon to gerrit_client.py
mass-abandon allows users to mass abandon CLs based on search query.
Owner is always specified (either "me" or user defined via -p).

R=gavinmak@google.com, thakis@chromium.org

Change-Id: I789a3a1f3af22867ebbbc50c35b60486206ff403
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2811005
Reviewed-by: Gavin Mak <gavinmak@google.com>
Commit-Queue: Josip Sokcevic <sokcevic@google.com>
4 years ago
Nico Weber 3f562c04e0 win_toolchain: Write json files only if different from what is on disk
Bug: chromium:1188672
Change-Id: I10aed281afd636acc0c680a7da486ae74aa637df
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2807968
Auto-Submit: Nico Weber <thakis@chromium.org>
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
4 years ago
Robert Iannucci 51eac66a56 Update `bb` tool with enhanced support for experiments.
Relevant changes:

6a6825419 [bb] Add explicit experiment support to `bb ls`.
bccb58660 [bb] Make `bb get` print experiments.
0d2672023 [buildbucket/bb] Add the ability to set/block experiments in `bb add`.
2fa412040 remove is_luci property

R=gavinmak@google.com, vadimsh, yiwzhang

Bug: 1093970,1134774
Change-Id: I2860857cb86626ad235dbb87eccf19ed38800557
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2809195
Auto-Submit: Robbie Iannucci <iannucci@chromium.org>
Commit-Queue: Vadim Shtayura <vadimsh@chromium.org>
Reviewed-by: Vadim Shtayura <vadimsh@chromium.org>
4 years ago
seanmccullough faaed2f486 [depot_tools, chromium/src] mv CheckInclusiveLanguage into canned checks
Also adds a new canned_presubmit_checks_test.py and supporting mocks,
based on the PRESUBMIT_*.py under chromium/src.

If this is OK, there are subsequent CLs for removing the original code
from chromuim/src and infra/infra, and having their PRESUBMIT scripts
just reference this canned check instead.

Change-Id: I67dfb7ac0b4cdc36bd62ec2dc062ca5c78c2244e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2805268
Commit-Queue: Sean McCullough <seanmccullough@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@google.com>
4 years ago
Nico Weber 6900c00a1a depot_tools bootstrap: Don't try to pull non-existent binaries on mac/arm64
With this, `gclient` completes bootstrap after running

    echo "mac-arm64" > .cipd_client_platform

as long as you also run

    export VPYTHON_BYPASS="manually managed python not supported by chrome operations"

Things work enough that you can `fetch chromium`. You can then build
all of chrome (if you want to build all tests, you also need to
`echo "mac-arm64" > third_party/depot_tools/.cipd_client_platform`
in your chromium checkout).

Parts of depot_tools that actually need the env managed by vpython,
such as `git cl`, don't work. So you can't `git cl upload` for now.

Bug: 1184768,1103326,1184766,1184765,1103236
Change-Id: Ib559c35e9665b826db9841dd8a4fe0dea16f37ec
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2807956
Auto-Submit: Nico Weber <thakis@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@google.com>
Reviewed-by: Vadim Shtayura <vadimsh@chromium.org>
Commit-Queue: Dirk Pranke <dpranke@google.com>
4 years ago
Dean Michael Berris 5c3929c352 pinpoint: Update Pinpoint CLI
This change includes the following commits from the infra/infra
repository:

97ee9cecaa Pinpoint: introduce fakelegacy-based smoke test. (chowski@google.com)
7f5c67efcb pinpoint: Translate ExperimentCommit properly (dberris@google.com)
6c32245733 Pinpoint cli: add actionable error when GCS credentials fail. (chowski@google.com)
5ed9722479 pinpoint fakelegacy: support /api/new endpoint. (chowski@google.com)
9d26e75b86 pinpoint: add copyright heading to files which are missing it. (chowski@google.com)
2c07962627 Pinpoint server: use the new ShouldBeStatusError assertion more. (chowski@google.com)
830e31f6b1 Pinpoint server: add unit test for ScheduleJob. (chowski@google.com)
c27f979330 Pinpoint cli: infer some flag defaults from `git cl issue`. (chowski@google.com)
ed69e2428f Pinpoint fakelegacy: support ListJobs. (chowski@google.com)
5f0e1e9a35 Pinpoint: first stab at a fake Legacy Pinpoint implementation. (chowski@google.com)
3aae5c2e6b pinpoint: Support wait+download for experiments (dberris@google.com)
6a634f7926 Replace 'master' with 'main' in test data. (chowski@chromium.org)
81ccbd091d Pinpoint server: support ListJobs' filter argument. (chowski@chromium.org)
575bd02e5a pinpoint: Update docs, support non-chromium/src (dberris@google.com)
e53451b2ca pinpoint server: support Cancel RPC. (chowski@chromium.org)
cb65ecf501 pinpoint: Reduce duplication in server_test (chowski@chromium.org)
e91e4e899d pinpoint: Support optional patchset for CLs (dberris@google.com)

R=dpranke@google.com

Bug: chromium:1175615
Change-Id: I4ac6407d822bf1905cbba02d29649ac4e400aecb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2807336
Auto-Submit: Dean Berris <dberris@chromium.org>
Commit-Queue: Dirk Pranke <dpranke@google.com>
Reviewed-by: Dirk Pranke <dpranke@google.com>
4 years ago
recipe-roller da768751d4 Roll recipe dependencies (trivial).
This is an automated CL created by the recipe roller. This CL rolls
recipe changes from upstream projects (recipe_engine) into this repository.

recipe_engine:
9e296d9de0
  9e296d9 (yiwzhang@google.com)
      Set step end_time to start_time if it is earlier than the start...

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug.

R=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: Iba11781abb42db07f3a43f8020daa10181edf713
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2803201
Commit-Queue: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
Bot-Commit: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
4 years ago
Stephen Martinis 7837307d38 Delete CheckBuildbotPendingBuilds
This check has been disabled basically everywhere since about 2013. It
should be safe to remove this; there will be warnings when trying to
skip it, but it shouldn't fail any builds.

Recipe-Nontrivial-Roll: build
Bug: 1194216
Change-Id: I84878f9e46543ae71f7e4e057b99d15a034e0d0d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2794799
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@google.com>
Reviewed-by: Gavin Mak <gavinmak@google.com>
4 years ago
LaMont Jones 9eed4238d8 gerrit: add createchange changeedit publishchangeedit
Add calls to support creating changes, as well as editing and publishing
them in gerrit_client and gerrit_util.

Bug=b:182613582

Change-Id: I0514cf08dce63ab29d99d4485d96fa124006326a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2800811
Auto-Submit: LaMont Jones <lamontjones@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@google.com>
Reviewed-by: Josip Sokcevic <sokcevic@google.com>
Commit-Queue: LaMont Jones <lamontjones@chromium.org>
4 years ago
Gavin Mak 9a94a95e65 Remove TAGS_REFSPEC if no_fetch_tags is true
Bug: 1188627
Change-Id: I6ee14ee9976b9aab60ce409ddd17f018ad966055
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2797933
Reviewed-by: Josip Sokcevic <sokcevic@google.com>
Commit-Queue: Gavin Mak <gavinmak@google.com>
4 years ago
Gavin Mak b1c08f6c61 Notify if a change auto-CCs more than 100 users
When uploading a large-scale change, many users are automatically CC'ed
on a change. This adds a warning if this is the case and links to a more
appropriate approval for a large-scale change.

Bug: 1194514
Change-Id: Icd336ebdffbccbc9f30a89a33e244a788a768cfa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2798963
Reviewed-by: Josip Sokcevic <sokcevic@google.com>
Commit-Queue: Gavin Mak <gavinmak@google.com>
4 years ago
Solomon Kinard 053a717f02 Quick spelling update to man pages
Change-Id: Icd13bc19a59048ab9d90b6a55f68f052a0e2c4b9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2797939
Commit-Queue: Dirk Pranke <dpranke@google.com>
Reviewed-by: Dirk Pranke <dpranke@google.com>
4 years ago
Gavin Mak 554187a21c Add more exception details in _run_check_function
Bug: 1194603
Change-Id: I4945f815078a681c853e038faecc443762a07bd3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2797935
Reviewed-by: Josip Sokcevic <sokcevic@google.com>
Commit-Queue: Gavin Mak <gavinmak@google.com>
4 years ago
Josip Sokcevic 0642373b27 Update origin/HEAD automatically
If origin/HEAD resolves to `master`, depot_tools can check if
origin/main exists. If so, it's very likely that origin/HEAD should be
updated to point to origin/main.
In the worst case, this will result in extra git remote call while not
changing local state (i.e origin/HEAD remained unchanged).

R=gavinmak@google.com

Bug: 1190702
Change-Id: I51e69d7a95d3534e1820099b4b6983d38b5e2763
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2795075
Reviewed-by: Gavin Mak <gavinmak@google.com>
Commit-Queue: Josip Sokcevic <sokcevic@google.com>
4 years ago
Josip Sokcevic f2cfd3dac0 Reland "Add --push-options in git cl upload"
This is a reland of e2c65d7b75

Original change's description:
> Add --push-options in git cl upload
>
> --push-option parameter is passed to git push as is.
>
> R=ehmaldonado@google.com
>
> Bug: 1184393
> Change-Id: Id1f7da1f0c8e8a23144b547d50d817fe8d4efeb1
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2786080
> Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
> Commit-Queue: Josip Sokcevic <sokcevic@google.com>

R=gavinmak@google.com

Bug: 1184393
Change-Id: I9302a2e908b931a963a63ed2bb3778e21c4d6e53
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2794787
Reviewed-by: Gavin Mak <gavinmak@google.com>
Commit-Queue: Josip Sokcevic <sokcevic@google.com>
4 years ago
Mike Frysinger 5260b2b204 vpython: use the Chromium version on all platforms
We have a python2 binary available on all platforms already, so use
that instead of relying on the system copy.  This provides consistent
behavior across all platforms and avoids the impending doom when host
distros stop providing python2 packages.

Change-Id: Id48fe343291ba70981a1a65526ba80d1f7ae7d73
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2785114
Reviewed-by: Dirk Pranke <dpranke@google.com>
Reviewed-by: Brian Ryner <bryner@google.com>
Commit-Queue: Mike Frysinger <vapier@chromium.org>
4 years ago
Ryan Harrison 9781115e15 Add Tint to list of Monorail projects
Change-Id: I370f023c7c4a5e5b1d1c1322bc443c7c375e1089
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2792341
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Commit-Queue: Dirk Pranke <dpranke@google.com>
Auto-Submit: Ryan Harrison <rharrison@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@google.com>
4 years ago
Josip Sokcevic d0ba91fa49 Revert "Add --push-options in git cl upload"
This reverts commit e2c65d7b75.

Reason for revert: https://crbug.com/1192942

Original change's description:
> Add --push-options in git cl upload
>
> --push-option parameter is passed to git push as is.
>
> R=​ehmaldonado@google.com
>
> Bug: 1184393
> Change-Id: Id1f7da1f0c8e8a23144b547d50d817fe8d4efeb1
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2786080
> Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
> Commit-Queue: Josip Sokcevic <sokcevic@google.com>

Bug: 1184393
Change-Id: Idf5d79fda3c5f917b1de5ca396837f14a401ffb7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2791749
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Commit-Queue: Josip Sokcevic <sokcevic@google.com>
Reviewed-by: Gavin Mak <gavinmak@google.com>
4 years ago
Edward Lesmes e0de6a88e5 Revert "Remove ehmaldonado@chromium.org from OWNERS"
This reverts commit 4a9ce1b527.

Reason for revert:
Testing secondary account can still approve

Original change's description:
> Remove ehmaldonado@chromium.org from OWNERS
>
> Change-Id: I661f8d23dd7cd716e4746fd4ef20de9416fd6f1c
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2787480
> Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
> Commit-Queue: Gavin Mak <gavinmak@google.com>

Change-Id: Ib54084be936f0f66370600f3b520a13079ae763c
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2787596
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Reviewed-by: Gavin Mak <gavinmak@google.com>
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
4 years ago
Gavin Mak 4a9ce1b527 Remove ehmaldonado@chromium.org from OWNERS
Change-Id: I661f8d23dd7cd716e4746fd4ef20de9416fd6f1c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2787480
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
Commit-Queue: Gavin Mak <gavinmak@google.com>
4 years ago
Nico Weber f15a709ff3 Try removing windows-386 from cipd_manifest.txt
See bug for details.

Please revert if this breaks something.

Bug: 1192749
Change-Id: I727e5d04e5c5a235030f93ee8a917d89edb6082b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2785108
Commit-Queue: Nico Weber <thakis@chromium.org>
Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
Auto-Submit: Nico Weber <thakis@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
4 years ago
Yiwei Zhang e21b73eba1 Update lucicfg v1.23.1 -> v1.23.3.
To pick up:
  * https://crrev.com/c/2782857
  * https://crrev.com/c/2783876

R=qyearsley, tandrii

Bug: 1190367,1189795
Change-Id: Ia31c427f2a77aa24dc4ac3201db7f4012f903d5e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2786281
Reviewed-by: Andrii Shyshkalov <tandrii@google.com>
Commit-Queue: Yiwei Zhang <yiwzhang@google.com>
4 years ago
Nodir Turakulov 49d9d8735f [dirmd] Roll dirmd
To pick up
https://chromium-review.googlesource.com/c/infra/infra/+/2757296
https://chromium-review.googlesource.com/c/infra/infra/+/2762697
https://chromium-review.googlesource.com/c/infra/infra/+/2779750

R=sokcevic@google.com

Bug: 1179786
Change-Id: I34c2a28d4325a5004e8cc32d6ac74fb2f2fb4390
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2786133
Auto-Submit: Nodir Turakulov <nodir@chromium.org>
Commit-Queue: Josip Sokcevic <sokcevic@google.com>
Reviewed-by: Josip Sokcevic <sokcevic@google.com>
4 years ago
Josip Sokcevic e2c65d7b75 Add --push-options in git cl upload
--push-option parameter is passed to git push as is.

R=ehmaldonado@google.com

Bug: 1184393
Change-Id: Id1f7da1f0c8e8a23144b547d50d817fe8d4efeb1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2786080
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>
Commit-Queue: Josip Sokcevic <sokcevic@google.com>
4 years ago
recipe-roller 273c9cfaca Roll recipe dependencies (trivial).
This is an automated CL created by the recipe roller. This CL rolls
recipe changes from upstream projects (recipe_engine) into this repository.

recipe_engine:
3e3066d09b
  3e3066d (iannucci@chromium.org)
      Do not open $debug until the engine is committed to running the...

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug.

R=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I089e6ac0444c465987c43103dc1634fd61d8f5b1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2785349
Commit-Queue: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
Bot-Commit: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
4 years ago
recipe-roller cc5d231570 Roll recipe dependencies (trivial).
This is an automated CL created by the recipe roller. This CL rolls
recipe changes from upstream projects (recipe_engine) into this repository.

recipe_engine:
e9c8ffe3e5
  e9c8ffe (yiwzhang@google.com)
      cq: Expose method for setting reuse

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug.

R=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: Ie0ef956e3f06cab41744df16a51998e8f4f43a2e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2782115
Commit-Queue: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
Bot-Commit: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
4 years ago
recipe-roller 68a0333ec7 Roll recipe dependencies (nontrivial).
This is an automated CL created by the recipe roller. This CL rolls
recipe changes from upstream projects (recipe_engine) into this repository.

Please review the expectation changes, and LGTM+CQ.

recipe_engine:
3faa1bab09
  3faa1ba (yiwzhang@google.com)
      cq: Use proto Output properties in cq recipe_module

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug.

R=dpranke@google.com

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: I5cb2fc5927de1cecad74f2c7a25be875983556a2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2781195
Auto-Submit: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
Commit-Queue: Dirk Pranke <dpranke@google.com>
Reviewed-by: Dirk Pranke <dpranke@google.com>
4 years ago
Brian Ryner 84ce160786 Bump Xcode version to 12.4.
This will allow us to target Big Sur for Mac ARM64 builds.

Bug: 1124119
Change-Id: I0aa17c744e13d45699e79eb17336ebcafb9ea719
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2777472
Reviewed-by: Mark Mentovai <mark@chromium.org>
Reviewed-by: Jeff Yoon <jeffyoon@google.com>
Commit-Queue: Brian Ryner <bryner@google.com>
4 years ago
Josip Sokcevic 848a5361d7 Stop using /usr/bin/python directly
Use /usr/bin/env python instead

R=dpranke@google.com

Bug: 1191100
Change-Id: I8de37ea7a02e7236d3bd36d4bae8cf4ffcf75ab9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2780631
Auto-Submit: Josip Sokcevic <sokcevic@google.com>
Commit-Queue: Dirk Pranke <dpranke@google.com>
Reviewed-by: Dirk Pranke <dpranke@google.com>
4 years ago
Vadim Shtayura 8db027f851 Update luci-auth tool.
To pick up ID tokens support in `luci-auth context`.

R=tandrii@chromium.org

Change-Id: Ia2fc9cac96256bdd74186dcf584a47bfd97f7a0f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2780632
Auto-Submit: Vadim Shtayura <vadimsh@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@google.com>
Commit-Queue: Andrii Shyshkalov <tandrii@google.com>
4 years ago
recipe-roller 726f4d7a22 Roll recipe dependencies (trivial).
This is an automated CL created by the recipe roller. This CL rolls
recipe changes from upstream projects (recipe_engine) into this repository.

recipe_engine:
3c382e042b
  3c382e0 (yiwzhang@google.com)
      engine: Support set proto output properties

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug.

R=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Change-Id: Iab3f85b66bfff4df0b87b26056a2c3c41d077822
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2779774
Commit-Queue: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
Bot-Commit: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
4 years ago
Yiwei Zhang b2659abe19 recipe: Adopt the breaking change in cq recipe_module
cq changes: https://crrev.com/c/2773637

R=tandrii

Bug: 1177864
Recipe-Manual-Change: build
Recipe-Manual-Change: build_limited_scripts_slave
Change-Id: Ic89a2cd6c53bc69050dca4b1e3da8560a5f37dea
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2776469
Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
Auto-Submit: Yiwei Zhang <yiwzhang@google.com>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@google.com>
4 years ago
Vadim Shtayura 4cbfb4b6cf Update lucicfg v1.22.2 -> v1.23.1.
To pick up an experiment with short bucket names in
luci-scheduler.cfg.

R=tandrii@chromium.org
BUG=1182002

Change-Id: Ibf8870e24a41d2d2ad158b856fff973aa3a79cdf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2780371
Reviewed-by: Andrii Shyshkalov <tandrii@google.com>
Commit-Queue: Vadim Shtayura <vadimsh@chromium.org>
4 years ago
Nico Weber d4da7ca919 gclient: Correctly set host_cpu to arm64 on arm macs
platform.machine() is 'arm64' on arm macs, and the `.startswith('arm')`
branch converted that to 'arm' before this CL.

Bug: 1103236,1190880
Change-Id: Idd75a724f059ecd2dd873737e4998fe9bc937e04
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2779414
Commit-Queue: Nico Weber <thakis@chromium.org>
Commit-Queue: Dirk Pranke <dpranke@google.com>
Auto-Submit: Nico Weber <thakis@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@google.com>
4 years ago
Edward Lesmes 743e98ce87 gerrit_util: Fix checking if code-owners enabled on repo.
Change-Id: I5f518c22be4c5496f91202015c9caf18d0fa1be9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2778638
Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org>
Reviewed-by: Jason Clinton <jclinton@chromium.org>
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
4 years ago