Commits · f05fcf7051e12e5b561b8811e299096f332157f7 · Linshizhi / Chromium_Depot_tools

18 Jul, 2022 3 commits

Update lucicfg from version 1.31.5 to 1.31.6 · f05fcf70

Quinten Yearsley authored Jul 18, 2022

Changes:
  - Allow direct use of location_filters in luci.cq_tryjob_verifier(...)

Bug: 1171945
Change-Id: I0aca6dd01976c66f106906958bc63a81dbd5e6a5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3770293Reviewed-by: Vadim Shtayura <vadimsh@chromium.org>
Commit-Queue: Quinten Yearsley <qyearsley@chromium.org>

f05fcf70

Fix typos in PDFium fetch config. · 26460d81

Lei Zhang authored Jul 18, 2022

- The extra comma at the end of the url string unintentionally turns
  the url into a list.
- The project name should be "pdfium".

Change-Id: I8944d59d06751716c512030145d29aac10cf13fd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3770290Reviewed-by: Aravind Vasudevan <aravindvasudev@google.com>
Commit-Queue: Lei Zhang <thestig@chromium.org>

26460d81

Remove apply_patch_on_gclient property · ea11cc31

Gavin Mak authored Jul 18, 2022

This property isn't used anymore.

Bug: 891917
Change-Id: I005f5b320cb33afddb1d3272f3f626542996bb62
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3766100Reviewed-by: Aravind Vasudevan <aravindvasudev@google.com>
Commit-Queue: Gavin Mak <gavinmak@google.com>
Reviewed-by: Joanna Wang <jojwang@chromium.org>

ea11cc31

15 Jul, 2022 2 commits

Update lucicfg from version 1.31.4 to 1.31.5 · 8168f767

Quinten Yearsley authored Jul 15, 2022

Changes:
  - Fix a bug in location_filter generation which affects cases where
    location_regexp matches gerrit host.

Bug: 1171945
Change-Id: Icd4d724b91e0de4652803d4223beaa290d37b810
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3763791Reviewed-by: Vadim Shtayura <vadimsh@chromium.org>
Commit-Queue: Quinten Yearsley <qyearsley@chromium.org>

8168f767

Roll recipe dependencies (trivial). · 7307e19d

recipe-roller authored Jul 15, 2022

This is an automated CL created by the recipe roller. This CL rolls
recipe changes from upstream projects (recipe_engine) into this repository.

The build that created this CL was
https://ci.chromium.org/b/8808545442368508161

recipe_engine:
https://chromium.googlesource.com/infra/luci/recipes-py.git/+/428eeaebf2e1de58ed1c7cae50daba3384d730cd
  428eeae (iannucci@chromium.org)
      Update buildbucket protos.

Please check the following references for more information:
- autoroller, https://chromium.googlesource.com/infra/luci/recipes-py/+/main/doc/workflow.md#autoroller
- rollback, https://chromium.googlesource.com/infra/luci/recipes-py/+/main/doc/workflow.md#rollback
- cross-repo dependencies, https://chromium.googlesource.com/infra/luci/recipes-py/+/main/doc/cross_repo.md

Use https://goo.gl/noib3a to file a bug.

R=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Ignore-Freeze: Autoroller
Bugdroid-Send-Email: False
Change-Id: Iafabacb36f16643a716979b718093986efc137cd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3766137
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>

7307e19d

14 Jul, 2022 3 commits

Roll out goma client VERSION=250 · 26bc518e

Fumitoshi Ukai authored Jul 14, 2022

Bug: b/237221299
Change-Id: Ic35b9e7e46a91c656c79c249db4ab38d1250dd17
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3758931Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
Commit-Queue: Fumitoshi Ukai <ukai@google.com>

26bc518e

Roll recipe dependencies (trivial). · 398a1e81

recipe-roller authored Jul 14, 2022

This is an automated CL created by the recipe roller. This CL rolls
recipe changes from upstream projects (recipe_engine) into this repository.

The build that created this CL was
https://ci.chromium.org/b/8808707136866269553

recipe_engine:
https://chromium.googlesource.com/infra/luci/recipes-py.git/+/f2ff4699652ca80cd66c9b99f129d4dcf591a253
  f2ff469 (chromium-autoroll@skia-public.iam.gserviceaccount.com)
      Roll CAS Client from 32fe5c8d8532 to a0ba80649473

Please check the following references for more information:
- autoroller, https://chromium.googlesource.com/infra/luci/recipes-py/+/main/doc/workflow.md#autoroller
- rollback, https://chromium.googlesource.com/infra/luci/recipes-py/+/main/doc/workflow.md#rollback
- cross-repo dependencies, https://chromium.googlesource.com/infra/luci/recipes-py/+/main/doc/cross_repo.md

Use https://goo.gl/noib3a to file a bug.

R=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Ignore-Freeze: Autoroller
Bugdroid-Send-Email: False
Change-Id: I3949e02ed6016d46daac9f10ad7f76edbca9d715
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3760949
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>

398a1e81

New vpython to depot_tools. · c8fadda5

Brian Ryner authored Jul 14, 2022

This release contains commits:
e74f8f68337ee619d400e0b6607cbe9df37f78ee Fix so that Python inherits file handles again on Windows.
059bcab0fb866cf8b9bc25b5845beeb8ee90fe3d Bundle the python interpreter on Mac.
e2552b1a6ebed6651aceb5a7ab4639c0d69c0c0e Resolve symlinks when finding the vpython executable.
d5fc7bca58e458258f8c3b8a25ecd070c77581ac Roll infra/go/src/go.chromium.org/luci/ ee2c632ce..82d549a81 (1 commit)
5d9b6ecf87cdfb928e1112d2838d26bc7ede2b48 Roll infra/go/src/go.chromium.org/luci/ 0480658b5..0535bf685 (5 commits)
3076679301d2b9b040bce2c871a63ddc49273c41 [environ] Rename Get => Lookup, GetEmpty => Get.
37f02f9f91aa3918b195931fbcf02750290d0571 Search bundled python paths before looking in PATH.
a4136aa4d9e8c359bcf234fcd6ae026fb5b92925 [go] Format all files.
8349853d4517895a2c5838a00aea3e368ce3f865 [vpython] Make TestLoadForScript pass on OSX when running locally.
d94b5704d8ad151a0306974a1a949d8057c44a18 Revert "Add support for bootstrapping the Python interpreter package."
6dacee57fbe6b5bc5a2fca23b5b7d8ccfea54ee7 Revert "Pass packages to vpython to bootstrap."
03e0afedc841bad5fed8024c4b7fd38d332701ad Pass packages to vpython to bootstrap.
fa3d6effe8906e93944bf54ea6978adcd022f75c go generate all
b5eee61d0153295f3efb2e67b319066c2306ac5b Add support for bootstrapping the Python interpreter package.
cc5fb5e4934445a0a5206bfc324ca486c1285415 [vpython] Skip flaky test.
c04b64ca743703ebbfc064ec5b5c35d3bc378fa4 Add vpython3-only cipd packages.
f14d61370f1d7302a797feb2adc3a2f2449130d1 Remove the VENVOmitUseWheel field now that nothing uses it.
7c739ae11aa23bc37601e6923add677600ae1329 Remove support for old pip versions in pep425 tag probing.
51e3f0a01454d2e18b36de28c7e0ca2f6127ea2b Stop setting the unused VENVOmitUseWheel option.
e0e35a4e2227fb3429f3483c4719b6370eca7e1e Hardcode the venv OmitUseWheel option to true.

Bug: 1318617, 1340653
Change-Id: Ifb94c81492edfa98e1b59fc82e6d43678b82d776
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3756933Reviewed-by: Gavin Mak <gavinmak@google.com>
Commit-Queue: Brian Ryner <bryner@google.com>

c8fadda5

13 Jul, 2022 4 commits

Update Pinpoint CLI for Octopus · 968a19bc

Ryan Heise authored Jul 13, 2022

Bug:1159092
Change-Id: I626b7faa21f073030594a82fd98f510177e2f388
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3760945Reviewed-by: Vadim Shtayura <vadimsh@chromium.org>
Auto-Submit: Ryan Heise <heiserya@google.com>
Commit-Queue: Ryan Heise <heiserya@google.com>
Commit-Queue: Vadim Shtayura <vadimsh@chromium.org>

968a19bc

Teach "fetch pdfium" about the checkout_configuration property. · b4a8efbe

Lei Zhang authored Jul 13, 2022

Allow someone interested in checking out a subset of the PDFium source
to do so via:

fetch pdfium --checkout_configuration=small

Along the way, also add support for the target_os and target_os_only
properties, which are carried over from the Chromium fetch config.

Change-Id: I11153fe7cb828c748055328ab2b60224b665ae84
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3758798
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Aravind Vasudevan <aravindvasudev@google.com>

b4a8efbe

Rearrange presubmit timing message for easier scanning · 6757d469

Bruce Dawson authored Jul 13, 2022

Some presubmits can be take orders of magnitude longer than others, but
finding these presubmits is difficult if the time is printed at the end
of the output lines. Moving the time to the start, with enough alignment
for all plausible times, makes scanning the output easy.

Bug: 1309977
Change-Id: I9ba1a599e9825801e1128844a39c7ccafe046f91
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3744130Reviewed-by: Gavin Mak <gavinmak@google.com>
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>

6757d469

Remove 'Fetching' check in testTrackTag · c3ac73eb

Gavin Mak authored Jul 13, 2022

This check has been failing for a while, and blocks submission of other
depot_tools CLs.

Change-Id: I0f61ff1657c8cc01d0004ed5df9e37465b73c237
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3759629Reviewed-by: Aravind Vasudevan <aravindvasudev@google.com>
Commit-Queue: Gavin Mak <gavinmak@google.com>

c3ac73eb

11 Jul, 2022 1 commit

Add number of presubmit errors/warnings in header · d22bf607

Gavin Mak authored Jul 11, 2022

Headers currently have the format: `** Presubmit ${name} **`. Make these
more helpful by including the number of total errors there are.

Bug: 1341987
Change-Id: Ib41a133c31568d4264d73c14350f0bbd9590356a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3756169Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Commit-Queue: Gavin Mak <gavinmak@google.com>

d22bf607

08 Jul, 2022 1 commit

git_cache: wrap "git remote show" call in retries · 1601b3eb

Jack Neus authored Jul 08, 2022

BUG=b:236709670
TEST=none

Change-Id: I575dd82260acdbeef3c4c7607e2c2ac8020a44c9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3749247
Commit-Queue: Jack Neus <jackneus@google.com>
Reviewed-by: Aravind Vasudevan <aravindvasudev@google.com>
Reviewed-by: Gavin Mak <gavinmak@google.com>

1601b3eb

07 Jul, 2022 1 commit

Remove use_rbe gn arg support · bb07d9eb

Richard Wang authored Jul 07, 2022

As it is removed in https://crrev.com/c/3744967, this cleans up the check for a no longer existing arg.

Bug: chromium:1247781
Change-Id: Ic7288cf34e63cd52087e2b51f1b4c816bbbc0363
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3745201Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Commit-Queue: Richard Wang <richardwa@google.com>

bb07d9eb

01 Jul, 2022 7 commits

Roll recipe dependencies (trivial). · 78c53d11

recipe-roller authored Jul 01, 2022

This is an automated CL created by the recipe roller. This CL rolls
recipe changes from upstream projects (recipe_engine) into this repository.

The build that created this CL was
https://ci.chromium.org/b/8809798072144355457

recipe_engine:
https://chromium.googlesource.com/infra/luci/recipes-py.git/+/bb01260dc91d33aa60051a3790a1abc67d26334c
  bb01260 (alexschulze@chromium.org)
      Revert "[file] Add fail_silently to copy and copytree"

Please check the following references for more information:
- autoroller, https://chromium.googlesource.com/infra/luci/recipes-py/+/main/doc/workflow.md#autoroller
- rollback, https://chromium.googlesource.com/infra/luci/recipes-py/+/main/doc/workflow.md#rollback
- cross-repo dependencies, https://chromium.googlesource.com/infra/luci/recipes-py/+/main/doc/cross_repo.md

Use https://goo.gl/noib3a to file a bug.

R=iannucci@chromium.org

Recipe-Tryjob-Bypass-Reason: Autoroller
Ignore-Freeze: Autoroller
Bugdroid-Send-Email: False
Change-Id: I8a9a0b99dc04b341eda782dce6b1f8b272159f75
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3741122
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>

78c53d11

Fix presubmit_support.py to handle CRLF in PRESUBMIT.py · 6e70e86d

Bruce Dawson authored Jul 01, 2022

PRESUBMIT.py files should not have CRLF line endings, but if a git repo
is misconfigured this may happen and it is important for the presubmits
to run properly so that they can report on this issue.

In particular, the regex which detects USE_PYTHON3 = True did not work
with CRLF line endings.

It was supposed to handle these, and there is even a comment saying:

  # Accept CRLF presubmit script.

However the line after that uses 'rU' as the file open mode in order to
try to get universal newlines, but FileRead ignores the file mode. So,
the replace method is now used instead.

Bug: angleproject:7475, angleproject:4905
Change-Id: I38321176feca089a5796b24756b371709d852243
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3739269Reviewed-by: Eddie Hatfield <eddiehatfield@google.com>
Reviewed-by: Cody Northrop <cnorthrop@google.com>
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>

6e70e86d

[no-sync] Add method to check for diffs between the current checkout and another commit. · 5a7c824f

Joanna Wang authored Jul 01, 2022

Bug: 1339472

Change-Id: If97d2fad7e5c61ec907e15a00db225b653a62bef
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3738365Reviewed-by: Gavin Mak <gavinmak@google.com>
Commit-Queue: Joanna Wang <jojwang@chromium.org>

5a7c824f

[depot tools] Update six version. · c6798c89

Joanna Wang authored Jul 01, 2022

Change-Id: Ic776edca9a399b4667451e702388c2747d397f9b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3738368
Auto-Submit: Joanna Wang <jojwang@chromium.org>
Reviewed-by: Aravind Vasudevan <aravindvasudev@google.com>
Commit-Queue: Aravind Vasudevan <aravindvasudev@google.com>

c6798c89

[no-sync] Update ParseDepsFile to process deps and hooks appropriately with should_sync. · 18af7efd

Joanna Wang authored Jul 01, 2022

Bug: 1339472
Change-Id: Iea077cdb655105d27431ff4f677d93eec121bc31
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3738361
Commit-Queue: Joanna Wang <jojwang@chromium.org>
Reviewed-by: Gavin Mak <gavinmak@google.com>

18af7efd

Roll recipe dependencies (trivial). · 9af90cb5

recipe-roller authored Jul 01, 2022

This is an automated CL created by the recipe roller. This CL rolls
recipe changes from upstream projects (recipe_engine) into this repository.

The build that created this CL was
https://ci.chromium.org/b/8809860362074427393

recipe_engine:
https://chromium.googlesource.com/infra/luci/recipes-py.git/+/2fdd5ba812beb59d941d96e5920bda658764d9f9
  2fdd5ba (jwata@google.com)
      Add fast recipe rollback instrucitons

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
Ignore-Freeze: Autoroller
Bugdroid-Send-Email: False
Change-Id: Icd28d520df42d17a965a30ad06b5aa836a0fe2b7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3739931
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>

9af90cb5

Roll recipe dependencies (trivial). · 9dec1b42

recipe-roller authored Jul 01, 2022

This is an automated CL created by the recipe roller. This CL rolls
recipe changes from upstream projects (recipe_engine) into this repository.

The build that created this CL was
https://ci.chromium.org/b/8809884896443954673

recipe_engine:
https://chromium.googlesource.com/infra/luci/recipes-py.git/+/fe626574f31049199406b9328f2e8ce9fba4693d
  fe62657 (chromium-autoroll@skia-public.iam.gserviceaccount.com)
      Roll CAS Client from 361f01c81b49 to d3fd72180a64

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
Ignore-Freeze: Autoroller
Bugdroid-Send-Email: False
Change-Id: I25853499048f64c87c231e0e97ed1c469828d717
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3739278
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>

9dec1b42

30 Jun, 2022 3 commits

[no-sync] Add a skip_sync_revisions and process it before running deps. · 6628661e

Joanna Wang authored Jun 30, 2022

Bug: 1339472
Change-Id: I429489f7deea035c47e27e32ada858fb8a827643
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3735487Reviewed-by: Gavin Mak <gavinmak@google.com>
Commit-Queue: Joanna Wang <jojwang@chromium.org>

6628661e

[git-cl] If set, use `options.title` for initial uploads. · 566e9d01

manukh authored Jun 30, 2022

Change-Id: Ia57b7a7b769d352e05820af0e40a52fcab471ddb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3736424Reviewed-by: Gavin Mak <gavinmak@google.com>
Commit-Queue: manuk hovanesian <manukh@chromium.org>

566e9d01

[gclient] Specify which DEPS is still using `use_relative_hooks`. · 4e6264cc

Joanna Wang authored Jun 30, 2022

Bug:1107325, 1340011
Change-Id: I90bce90cba2ccf2a37b703b9bad49b5f79cfde33
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3738359Reviewed-by: Aravind Vasudevan <aravindvasudev@google.com>
Commit-Queue: Joanna Wang <jojwang@chromium.org>

4e6264cc

29 Jun, 2022 4 commits

[cipd] Update the client v2.6.5 => v2.6.7. · 0e5881fb

Vadim Shtayura authored Jun 29, 2022

Adds support loading the machine-global config file to pick up
location of the admission plugin binary. See the bug for details.

BUG=1310363
R=iannucci@chromium.org
CC=akashmukherjee@google.com

Change-Id: I7a0cec6a994b79929e204766d32e0f9649b9dc8f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3736694Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Commit-Queue: Vadim Shtayura <vadimsh@chromium.org>

0e5881fb

[verified-builds]: Update lucicfg to 1.31.4 · e7fd0ca4

Yulan Lin authored Jun 29, 2022

Changes:
- add the "wrapper" parameter to luci.recipe()
- add shadow bucket

Bug:1339006
Change-Id: I0f5c23f631b23b46fb40c728c432fb759bcefd7c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3736695
Commit-Queue: Yulan Lin <yulanlin@google.com>
Reviewed-by: Vadim Shtayura <vadimsh@chromium.org>

e7fd0ca4

Allow depot_tools/.git to be a file · 25303119

Will Cassella authored Jun 29, 2022

This CL relaxes the check for the .git folder on Windows so that it
may be either a file or a folder, as may be the case if depot_tools
is a git submodule or a worktree.

Bug: 1333997
Change-Id: Ib805912903696fc247f85d15c35b6e3093cd57f9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3736684
Commit-Queue: Will Cassella <cassew@chromium.org>
Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: Aravind Vasudevan <aravindvasudev@google.com>

25303119

Roll recipe dependencies (trivial). · 4b272722

recipe-roller authored Jun 29, 2022

This is an automated CL created by the recipe roller. This CL rolls
recipe changes from upstream projects (recipe_engine) into this repository.

The build that created this CL was
https://ci.chromium.org/b/8809989966553312161

recipe_engine:
https://chromium.googlesource.com/infra/luci/recipes-py.git/+/fd20a01913328acafd79aa0cfd0df43fd6564dd6
  fd20a01 (atyfto@google.com)
      [protoc] Use infra/3pp/tools/protoc

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
Ignore-Freeze: Autoroller
Bugdroid-Send-Email: False
Change-Id: I75523ad6996a45239ac0e9a51bbb83acfb95cca0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3736686
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>

4b272722

28 Jun, 2022 2 commits

[depot_tools] roll led · a83ac939

Yuanjun Huang authored Jun 28, 2022

To include https://crrev.com/c/3727771.

all changes:
https://chromium.googlesource.com/infra/infra/+log/498086360..675ae8e

Bug:1339383
Change-Id: I54849b921964dea83a3d21a1966de2aa872c643a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3732792Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Commit-Queue: Yuanjun Huang <yuanjunh@google.com>

a83ac939

Roll recipe dependencies (trivial). · 3c4d401c

recipe-roller authored Jun 28, 2022

This is an automated CL created by the recipe roller. This CL rolls
recipe changes from upstream projects (recipe_engine) into this repository.

The build that created this CL was
https://ci.chromium.org/b/8810158576984191137

recipe_engine:
https://chromium.googlesource.com/infra/luci/recipes-py.git/+/606ae4d1deabddcf3b11c5fbb9791523dea61eeb
  606ae4d (yiwzhang@google.com)
      cv: export config proto from recipe engine

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
Ignore-Freeze: Autoroller
Bugdroid-Send-Email: False
Change-Id: Ia2b5d78582073ce06e94c9f76ad67fe13f1b9a32
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3731249
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>

3c4d401c

27 Jun, 2022 3 commits

Print helpful error message when updating depot_tools on Windows · 418f021a

Will Cassella authored Jun 27, 2022

The depot_tools directory may get into a state in which its no longer a
git repo, in which case we can't update it. On Windows this would cause
the update_depot_tools.bat script to fail with a somewhat confusing
error message, this CL fixes that to inform the user what happened and
what they should do about it.

Bug: 1333997
Change-Id: I2c60fc46f1fae7811ff6610e25380d08010d0597
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3718926Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Commit-Queue: Will Cassella <cassew@chromium.org>
Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
Reviewed-by: Aravind Vasudevan <aravindvasudev@google.com>

418f021a

Set default pylint version to 2.7 · 3ffbdfdb

Josip Sokcevic authored Jun 27, 2022

This was announced in infra-announce@chromium.org.

R=gavinmak@google.com

Bug: 1105747
Change-Id: I5ac21683ae8b72d7b8f1caacda5dc5faf131fd2a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3722041Reviewed-by: Gavin Mak <gavinmak@google.com>
Commit-Queue: Gavin Mak <gavinmak@google.com>

3ffbdfdb

Roll out goma client VERSION=249 · 2f3c96d5

Fumitoshi Ukai authored Jun 27, 2022

Bug: b/234780712
Change-Id: I61e654896979438a2e8a7de6f7d11c473df0092e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3721609
Commit-Queue: Fumitoshi Ukai <ukai@google.com>
Reviewed-by: Takuto Ikuta <tikuta@chromium.org>

2f3c96d5

24 Jun, 2022 2 commits

Remove tmp gclient files on fetch · ebccac71

Josip Sokcevic authored Jun 24, 2022

Regression was introduced as part of gclient_scm change to support cog.
The directory should be cleaned up regardless if is empty or not, and if
statement is used only to warn user about non-empty directory, which si
not expected.

R=jojwang@google.com

Fixed: 1339077
Change-Id: Ie3bc6bcac168bad9cf2cb6adc6ae4024fe9ef167
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3724051
Commit-Queue: Joanna Wang <jojwang@chromium.org>
Auto-Submit: Josip Sokcevic <sokcevic@google.com>
Reviewed-by: Joanna Wang <jojwang@chromium.org>

ebccac71

Remove redundant should_process check. · 0a2356cd

Joanna Wang authored Jun 24, 2022

AFAICT
- the code never reaches this point because we already check should process before adding it to the queue. http://shortn/_PWFae17qw4

- this might be necessary if the value of should_process changes between adding it to the queue and running it. But I don't see that happening anywhere. should_process is only ever set and modified here: http://shortn/_eirozAoIm4 which happens during in the parent's run() before the deps get added to the queue.

The should_process check that happens at the beginning of run() was added here: https://codereview.chromium.org/3124017/diff/2001/gclient.py

The should_process check before a deps gets enqueued was added later here: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/559150/

where several should_process checks were added and I think the author just didn't realize that, while the other methods might need them, run() already had a should_process check.

Alternatively, we could remove the should_process check before enqueue-ing. But that would result in enqueuing things that we currently don't enqueue, even if we ultimately don't do anything with it.

Change-Id: Ib3be61310c09a681988baa75fd562c7596e9e044
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3721600Reviewed-by: Josip Sokcevic <sokcevic@google.com>
Commit-Queue: Joanna Wang <jojwang@chromium.org>

0a2356cd

23 Jun, 2022 3 commits

Roll recipe dependencies (trivial). · d9a9c407

recipe-roller authored Jun 23, 2022

This is an automated CL created by the recipe roller. This CL rolls
recipe changes from upstream projects (recipe_engine) into this repository.

The build that created this CL was
https://ci.chromium.org/b/8810528512950350097

recipe_engine:
https://chromium.googlesource.com/infra/luci/recipes-py.git/+/5a7f940c8e17572245e886877b58038135f76409
  5a7f940 (yiwzhang@google.com)
      roll the pin of luci-go proto (excluding buildbucket proto for...

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
Ignore-Freeze: Autoroller
Bugdroid-Send-Email: False
Change-Id: I3cc6b248d7b6e38cf05a61f413ceb1b5458d82ef
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3722721
Bot-Commit: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>
Commit-Queue: Recipe Roller <recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com>

d9a9c407

lucicfg: Update from 1.31.2 to 1.31.3 · 3ef340ac

Yiwei Zhang authored Jun 23, 2022

Notable change:
  https://crrev.com/c/3720728: fix bugs when generating location_filter

R=qyearsley, vadimsh

Bug: 1171945
Change-Id: Ifa19ac2b49b0aa1062e0a9c624587c2e78a84a29
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3722032Reviewed-by: Vadim Shtayura <vadimsh@chromium.org>
Commit-Queue: Vadim Shtayura <vadimsh@chromium.org>
Auto-Submit: Yiwei Zhang <yiwzhang@google.com>

3ef340ac

Make not-run tests an error again · 1d5d7803

Bruce Dawson authored Jun 23, 2022

If a presubmit test is misconfigured then it may end up running on
neither Python 2 nor Python 3. This was supposed to trigger a warning
but the warning code was broken for almost a year. It was then fixed so
that it issued an error, but that caused some breakage, so it was made a
warning.

The known problems with tests not running at all have been fixed and
the warning has been on for over two weeks so this change turns the
warning back into an error. This is appropriate because a test that
is not run at all is a serious bug. This change will allow us to move
more confidently when switching tests to Python 3, and it shouldn't
cause any additional breakage.

Bug: 1330859
Change-Id: I51028bb9c896c60c5cf3ccb6f472ade0bb1e0c16
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3717242
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: Gavin Mak <gavinmak@google.com>

1d5d7803

22 Jun, 2022 1 commit

[lucicfg] Update 1.30.11 => 1.31.2. · dd8da399

Vadim Shtayura authored Jun 22, 2022

Notable changes:
* Updated `buildifier` dependency, adding more linting checks.
* Experiment to generate `location_filters` in CQ configs:
  https://chromium.googlesource.com/infra/luci/luci-go.git/+/6915bcc1

R=yiwzhang@google.com

Change-Id: I09ba35cf572eaa8cb0478db2bee19f7496f8eaee
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3718931
Commit-Queue: Vadim Shtayura <vadimsh@chromium.org>
Reviewed-by: Yiwei Zhang <yiwzhang@google.com>

dd8da399