Move bisect_builds.py to tools/bisect
Currently there are 3 scripts which basically do the same thing: * //tools/bisect-builds.py Most developers use this one. * https://chrome-internal.googlesource.com/chrome/test/python_tests/+/refs/heads/main/bisect_builds.py Test teams use this to support bisection. * https://chrome-internal.googlesource.com/clank/internal/apps/+/refs/heads/main/tools/bisect-archived-builds.py Clank only. Test teams use this to support bisection. We plan to streamline bisection experience documented in go/chrome-bisection-refresh. The end result will be: 1. Merge the three scripts to one under //tools/bisect/. 2. Define clear ownership in //tools/bisect/OWNERS for long term maintenance and support. This change copies the python_tests bisect_builds.py here. Only added a few comments to pass presubmit. There are still presubmit errors that I'll address in the following changes. For reviewers, please review: 1 The security concern in the linked bug was resolved. 2 This plan looks good. Bug: 332933533 Change-Id: Idc56c3a306461f0c02e3542018e673fc79f06d48 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5455951 Reviewed-by: Dirk Pranke <dpranke@google.com> Commit-Queue: Sven Zheng <svenzheng@chromium.org> Cr-Commit-Position: refs/heads/main@{#1289490}
This commit is contained in:

committed by
Chromium LUCI CQ

parent
ea4f588f3b
commit
07dc43c891
@ -42,6 +42,8 @@ _EXCLUDED_PATHS = (
|
||||
r".+_pb2(_grpc)?\.py$",
|
||||
r".+/pnacl_shim\.c$",
|
||||
r"^gpu/config/.*_list_json\.cc$",
|
||||
# TODO(crbug.com/332933533) Remove this exception.
|
||||
r"^tools/bisect/.*",
|
||||
r"tools/md_browser/.*\.css$",
|
||||
# Test pages for Maps telemetry tests.
|
||||
r"tools/perf/page_sets/maps_perf_test.*",
|
||||
@ -3710,6 +3712,7 @@ def CheckPythonDevilInit(input_api, output_api):
|
||||
affected_file,
|
||||
files_to_skip=(_EXCLUDED_PATHS + input_api.DEFAULT_FILES_TO_SKIP + (
|
||||
r'^build/android/devil_chromium\.py',
|
||||
r'^tools/bisect/.*',
|
||||
r'^third_party/.*',
|
||||
)),
|
||||
files_to_check=[r'.*\.py$'])
|
||||
|
1971
tools/bisect/bisect_builds.py
Executable file
1971
tools/bisect/bisect_builds.py
Executable file
File diff suppressed because it is too large
Load Diff
Reference in New Issue
Block a user