- 03 Nov, 2017 4 commits
-
-
Matt Mueller authored
Bug: 374928 Change-Id: I8f633ce172fc644e251193eb3fa4b67d62fb9dfa Reviewed-on: https://chromium-review.googlesource.com/747365 Commit-Queue: Matt Mueller <mattm@chromium.org> Reviewed-by: Aaron Gable <agable@chromium.org>
-
Ryan Tseng authored
The test output was incorrect, this matches what it actually should look like. Bug: 772529 Change-Id: I1ca9ceeb6d6b6f55b2ac7c90667c6a9904d2a43b Recipe-Nontrivial-Roll: build Recipe-Nontrivial-Roll: build_limited Recipe-Nontrivial-Roll: build_limited_scripts_slave Recipe-Nontrivial-Roll: infra Recipe-Nontrivial-Roll: skia Recipe-Nontrivial-Roll: release_scripts Change-Id: I1ca9ceeb6d6b6f55b2ac7c90667c6a9904d2a43b Reviewed-on: https://chromium-review.googlesource.com/751694 Commit-Queue: Ryan Tseng <hinoka@chromium.org> Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
-
Raphael Kubo da Costa authored
In Python land, the method run once a test method runs is called tearDown(), not cleanUp(). In other words, the existing methods were never called and we were always leaving lots of "gstools_test*" directories in /tmp. Change-Id: Ib1de95c7ba92922b98571780f389237de0dcb253 Reviewed-on: https://chromium-review.googlesource.com/753383 Commit-Queue: Aaron Gable <agable@chromium.org> Reviewed-by: Aaron Gable <agable@chromium.org>
-
Sergiy Byelozyorov authored
This check will be used in Chromium, V8 and Catapult PRESUBMIT scripts. R=maruel@chromium.org, tandrii@chromium.org Bug: 777893 Change-Id: I2ca1e774b89787c4d3b5f336315d145571858864 Reviewed-on: https://chromium-review.googlesource.com/738169 Commit-Queue: Sergiy Byelozyorov <sergiyb@google.com> Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org> Reviewed-by: Daniel Jacques <dnj@chromium.org>
-
- 02 Nov, 2017 3 commits
-
-
Aaron Gable authored
Bug: 749709 Change-Id: I17beca5c374c080502833b30a26b68a0774689ef Reviewed-on: https://chromium-review.googlesource.com/749633Reviewed-by: Robbie Iannucci <iannucci@chromium.org> Commit-Queue: Aaron Gable <agable@chromium.org>
-
Aaron Gable authored
Bug: 770327 Change-Id: I9dd0ed0be0761de4968136f563e8986b979c35cf Reviewed-on: https://chromium-review.googlesource.com/723824 Commit-Queue: Aaron Gable <agable@chromium.org> Reviewed-by: Ravi Mistry <rmistry@chromium.org>
-
Raphael Kubo da Costa authored
Instead of calling re.search() for every entry in WATCHLIST_DEFINITIONS for every file being processed, create the regular expressions object beforehand when parsing the watchlist file. Processing a Chromium commit with 17k files went down from about 25 minutes to 10 seconds with this change. Bug: 780055 Change-Id: I6493971b67a7466ce8e1e3b28537018a724bbf47 Reviewed-on: https://chromium-review.googlesource.com/751463Reviewed-by: Aaron Gable <agable@chromium.org> Commit-Queue: Raphael Kubo da Costa (rakuco) <raphael.kubo.da.costa@intel.com>
-
- 01 Nov, 2017 2 commits
-
-
Ryan Tseng authored
This relands commit 47b67c42. Also: * Fix bot_update.py output of manifest Bug: 772529,776299 Recipe-Nontrivial-Roll: infra Recipe-Nontrivial-Roll: build_limited_scripts_slave Recipe-Nontrivial-Roll: skiabuildbot Recipe-Nontrivial-Roll: release_scripts Recipe-Nontrivial-Roll: skia Recipe-Nontrivial-Roll: skiabuildbot Recipe-Nontrivial-Roll: build Change-Id: I31828ea5bfbc284623be8e553980f83d0889de92 Reviewed-on: https://chromium-review.googlesource.com/748308 Commit-Queue: Ryan Tseng <hinoka@chromium.org> Reviewed-by: Robbie Iannucci <iannucci@chromium.org> Reviewed-by: Ryan Tseng <hinoka@chromium.org>
-
Ryan Tseng authored
This reverts commit 47b67c42. Reason for revert: Bug in bot_update.py resource, breaks recipe roller. Original change's description: > Reland "bot_update recipe: Upload source manifest" > > This reverts commit c3d1208d. > > Also: > * Instead of replacing "manifest", just add a new "source_manifest" > to the output JSON. This allow transition without breakage. > * Change the test api so test for recipe output changes. > > The plan is to land this first, switch all downstream to "source_manifest", > and then remove the original "manifest" key. > > Bug: 772529,776299 > Change-Id: Iffb75f18046f8e4c058afe077872d4257b9dd754 > Recipe-Nontrivial-Roll: infra > Recipe-Nontrivial-Roll: build_limited_scripts_slave > Recipe-Nontrivial-Roll: skiabuildbot > Recipe-Nontrivial-Roll: release_scripts > Recipe-Nontrivial-Roll: skia > Recipe-Nontrivial-Roll: skiabuildbot > Recipe-Nontrivial-Roll: build > Reviewed-on: https://chromium-review.googlesource.com/731378 > Reviewed-by: Robbie Iannucci <iannucci@chromium.org> > Commit-Queue: Ryan Tseng <hinoka@chromium.org> TBR=iannucci@chromium.org,hinoka@chromium.org Change-Id: I7a4ee904075e8b75b8a47f9ef0cd8a633af85a9c No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 772529, 776299 Reviewed-on: https://chromium-review.googlesource.com/748312Reviewed-by: Ryan Tseng <hinoka@chromium.org> Commit-Queue: Ryan Tseng <hinoka@chromium.org>
-
- 31 Oct, 2017 3 commits
-
-
Ryan Tseng authored
This reverts commit c3d1208d. Also: * Instead of replacing "manifest", just add a new "source_manifest" to the output JSON. This allow transition without breakage. * Change the test api so test for recipe output changes. The plan is to land this first, switch all downstream to "source_manifest", and then remove the original "manifest" key. Bug: 772529,776299 Change-Id: Iffb75f18046f8e4c058afe077872d4257b9dd754 Recipe-Nontrivial-Roll: infra Recipe-Nontrivial-Roll: build_limited_scripts_slave Recipe-Nontrivial-Roll: skiabuildbot Recipe-Nontrivial-Roll: release_scripts Recipe-Nontrivial-Roll: skia Recipe-Nontrivial-Roll: skiabuildbot Recipe-Nontrivial-Roll: build Reviewed-on: https://chromium-review.googlesource.com/731378Reviewed-by: Robbie Iannucci <iannucci@chromium.org> Commit-Queue: Ryan Tseng <hinoka@chromium.org>
-
Tom Anderson authored
Change-Id: I66bfe94e5625715e03ee00b534a27003ca688047 Reviewed-on: https://chromium-review.googlesource.com/747073 Commit-Queue: Dirk Pranke <dpranke@chromium.org> Reviewed-by: Nico Weber <thakis@chromium.org> Reviewed-by: Dirk Pranke <dpranke@chromium.org>
-
Chris Nardi authored
The vpython/CIPD work created some errors when attempting to build on Cygwin. This patch fixes these errors by converting Cygwin paths to Windows paths or using the depot_tools python instead of Cygwin python. Bug: Change-Id: Id62f59a357fa2937c7eb843bf306ef4303b0910e Reviewed-on: https://chromium-review.googlesource.com/745727Reviewed-by: Robbie Iannucci <iannucci@chromium.org> Reviewed-by: Vadim Shtayura <vadimsh@chromium.org> Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
-
- 27 Oct, 2017 1 commit
-
-
Dominic Battre authored
If you name your branches "bug-123-fix-foobar" or "123-fix-foobar", git cl upload will automatically populate the "Bug: " line in the cl description with 123. Bug: 778252 Change-Id: I5bf570d5d44e8681c552a52a8b5ed0c64ce82284 Reviewed-on: https://chromium-review.googlesource.com/738237 Commit-Queue: Aaron Gable <agable@chromium.org> Reviewed-by: Aaron Gable <agable@chromium.org>
-
- 26 Oct, 2017 1 commit
-
-
David Benjamin authored
Bug: Change-Id: Id7509ae08bc09c6cb291f29b55c60d6d96e79121 Reviewed-on: https://chromium-review.googlesource.com/721359Reviewed-by: Robbie Iannucci <iannucci@chromium.org> Commit-Queue: David Benjamin <davidben@chromium.org>
-
- 25 Oct, 2017 2 commits
-
-
Mun Yong Jang authored
Bug:509672 Change-Id: I74bbaa725be829c625cd617c58b3e849acb18102 Reviewed-on: https://chromium-review.googlesource.com/726847 Commit-Queue: Mun Yong Jang <myjang@google.com> Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org> Reviewed-by: Nodir Turakulov <nodir@chromium.org>
-
Andrew Moylan authored
This change inserts a flush after "..." in lines like: Failed! Attempting to squash mybranch ... Failed! and: Failed! Attempting to squash mybranch ... Success! This way the user can follow the progress better and has an indication of why the rebase is taking longer than they might expect. Bug: Change-Id: Ie04db2879fccd363d7b0f13b7a5217f1c13fd0e2 Reviewed-on: https://chromium-review.googlesource.com/737009Reviewed-by: Robbie Iannucci <iannucci@chromium.org> Commit-Queue: Andrew Moylan <amoylan@chromium.org>
-
- 23 Oct, 2017 1 commit
-
-
Ryan Tseng authored
the tests haven't been ran by presumbit for a while because of the plural in the filename. At some point some post "gsutil cp" file checking happened, which broke the tests. This adds a callback to the fake gsutil cp so that the expect file is copied over. This also removes "gsutil version" checking from gsutil.py and just assume that if the file exists, then it's good, which should shave about 1-2s off of each gsutil.py call. Bug: 772740,776311 Change-Id: I4fef62cfd46a849afed1f095dd6a96069376d13d Reviewed-on: https://chromium-review.googlesource.com/707758 Commit-Queue: Ryan Tseng <hinoka@chromium.org> Reviewed-by: Robbie Iannucci <iannucci@chromium.org> Reviewed-by: Aaron Gable <agable@chromium.org>
-
- 20 Oct, 2017 5 commits
-
-
Benjamin Pastene authored
We only compile and tag cipd packages with armv6, so armv7 hosts get 404s from cipd. This will use armv6 packages on armv7 bots, which should work. Bug: 776703 Change-Id: I6e54c2f9e1b220ce038a99af1df28cb6c0a9e579 Reviewed-on: https://chromium-review.googlesource.com/730850Reviewed-by: Vadim Shtayura <vadimsh@chromium.org> Reviewed-by: Robbie Iannucci <iannucci@chromium.org> Commit-Queue: Benjamin Pastene <bpastene@chromium.org>
-
Robert Iannucci authored
From https://chromium.googlesource.com/infra/luci/luci-go: git log a90d36d07f65098da34b600f6cfb8a5f05199ac5..63bd11762eec0b8604bc89dcfb9de4f5dad3777a --oneline vpython e638db97 [vpython] Normalize spec before verification. 22c141ce [vpython] Remove maximum script path length check. b1e7bc36 [vpython] Better failure output. 3edc75c8 [vpython] Better VirtualEnv isolation. R=dnj@chromium.org, nodir@chromium.org Bug: Change-Id: I20d06c51760b27e70b9ce65c7ddd856943fab7eb Reviewed-on: https://chromium-review.googlesource.com/731512Reviewed-by: Nodir Turakulov <nodir@chromium.org> Reviewed-by: Ryan Tseng <hinoka@chromium.org> Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
-
Daniel Cheng authored
This will let progress show the name of the hook while it's running. Change-Id: I14ef2a9b5fd2bc4d8d778f5402f78a5827ffbb08 Reviewed-on: https://chromium-review.googlesource.com/731469Reviewed-by: Dirk Pranke <dpranke@chromium.org> Reviewed-by: Nico Weber <thakis@chromium.org> Commit-Queue: Daniel Cheng <dcheng@chromium.org>
-
Daniel Cheng authored
This is a reland of 5fd1defb Original change's description: > gclient runhooks: now with less spammy progress notifications > > Bug: 772741 > Change-Id: I62ad64219ba27ee4ea26ceb603b1133dff4a2e04 > Reviewed-on: https://chromium-review.googlesource.com/727194 > Reviewed-by: Nico Weber <thakis@chromium.org> > Reviewed-by: Dirk Pranke <dpranke@chromium.org> > Commit-Queue: Daniel Cheng <dcheng@chromium.org> Bug: 772741 Change-Id: Idd6fcd46a8c97716d68ed0975940705a90540a4d Reviewed-on: https://chromium-review.googlesource.com/728961Reviewed-by: Dirk Pranke <dpranke@chromium.org> Commit-Queue: Daniel Cheng <dcheng@chromium.org>
-
Jao-ke Chin-Lee authored
If the revision has not yet propagated, then we never retry, because the amount of time to fetch HEAD already exceeds the deadline. Bug: 771007 Change-Id: I91ac30b2b53b4f75729357b3298c52856a4d350e Reviewed-on: https://chromium-review.googlesource.com/729328 Commit-Queue: Jao-ke Chin-Lee <jchinlee@chromium.org> Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
-
- 19 Oct, 2017 4 commits
-
-
Daniel Cheng authored
This reverts commit 5fd1defb. Reason for revert: breaks v8 because hook names are optional Original change's description: > gclient runhooks: now with less spammy progress notifications > > Bug: 772741 > Change-Id: I62ad64219ba27ee4ea26ceb603b1133dff4a2e04 > Reviewed-on: https://chromium-review.googlesource.com/727194 > Reviewed-by: Nico Weber <thakis@chromium.org> > Reviewed-by: Dirk Pranke <dpranke@chromium.org> > Commit-Queue: Daniel Cheng <dcheng@chromium.org> TBR=dcheng@chromium.org,thakis@chromium.org,dpranke@chromium.org Change-Id: I7dee0ca188f5ca34232f16e6496249fc9f49fa96 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 772741 Reviewed-on: https://chromium-review.googlesource.com/728959Reviewed-by: Dirk Pranke <dpranke@chromium.org> Commit-Queue: Daniel Cheng <dcheng@chromium.org>
-
Daniel Cheng authored
Bug: 772741 Change-Id: I62ad64219ba27ee4ea26ceb603b1133dff4a2e04 Reviewed-on: https://chromium-review.googlesource.com/727194Reviewed-by: Nico Weber <thakis@chromium.org> Reviewed-by: Dirk Pranke <dpranke@chromium.org> Commit-Queue: Daniel Cheng <dcheng@chromium.org>
-
Michael Achenbach authored
This reverts commit 037f9b80. Reason for revert: The structural change to manifest breaks the recipe tester: https://cs.chromium.org/chromium/infra/recipes/recipes/recipe_roll_tryjob.py?q=recipe_roll_tryjob.py&sq=package:chromium&dr&l=149 https://crbug.com/776299 Original change's description: > bot_update recipe: Upload source manifest > > Bug: 772529 > Change-Id: I3f833740ddfbc71be42b140c99dcd4bbfeed54fe > Reviewed-on: https://chromium-review.googlesource.com/706071 > Commit-Queue: Ryan Tseng <hinoka@chromium.org> > Reviewed-by: Robbie Iannucci <iannucci@chromium.org> TBR=iannucci@chromium.org,hinoka@chromium.org,agable@chromium.org Change-Id: Iba9653d73b7f117ca2f94619ae5da900cd0c8c98 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 772529,776299 Reviewed-on: https://chromium-review.googlesource.com/728059Reviewed-by: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org>
-
Ryan Tseng authored
Bug: 772529 Change-Id: I3f833740ddfbc71be42b140c99dcd4bbfeed54fe Reviewed-on: https://chromium-review.googlesource.com/706071 Commit-Queue: Ryan Tseng <hinoka@chromium.org> Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
-
- 18 Oct, 2017 2 commits
-
-
Marc-Antoine Ruel authored
When 'gclient recurse' is called from a unconfigured directory. No functional change otherwise. R=agable@chromium.org Bug:774749 Change-Id: Ifb2b74c042ce31faf3e1287c5614739e58e9178c Reviewed-on: https://chromium-review.googlesource.com/726419Reviewed-by: Aaron Gable <agable@chromium.org> Commit-Queue: Marc-Antoine Ruel <maruel@chromium.org>
-
Michael Achenbach authored
Bug: 775854 Change-Id: I6f831a2536de3cadbeaede7b78cfcf7d41f77052 Reviewed-on: https://chromium-review.googlesource.com/725328Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org> Reviewed-by: Dirk Pranke <dpranke@chromium.org> Reviewed-by: Aaron Gable <agable@chromium.org> Commit-Queue: Aaron Gable <agable@chromium.org>
-
- 17 Oct, 2017 5 commits
-
-
Dirk Pranke authored
R=estaab@chromium.org Change-Id: I318a1897fa1d10fce4aaad2dd1f4a553d14aff47 Reviewed-on: https://chromium-review.googlesource.com/723999Reviewed-by: Erik Staab <estaab@chromium.org> Commit-Queue: Dirk Pranke <dpranke@chromium.org>
-
Andrew Grieve authored
Follow-up to d86c8037. The error message was still saying "src" rather than the repository root. Bug: 768962 Change-Id: I781191e561db889a28acf51ee49a20d6d352b489 Reviewed-on: https://chromium-review.googlesource.com/719718Reviewed-by: Dirk Pranke <dpranke@chromium.org> Commit-Queue: agrieve <agrieve@chromium.org>
-
Dan Jacques authored
Update "vpython" and "led" pins to version which includes PPC64 support: d1e9a52e1c8414cd0a0df8bec6c37f325e0fbaa1 TBR=iannucci@chromium.org BUG=chromium:773857 TEST=presubmit Change-Id: I7d5fd91b08b114e534926db45e1d499638659fbd Reviewed-on: https://chromium-review.googlesource.com/723165Reviewed-by: Daniel Jacques <dnj@chromium.org> Commit-Queue: Daniel Jacques <dnj@chromium.org>
-
Dan Jacques authored
The uname was reported incorrectly, and should have read "ppc64le". Update the script to reference this. TBR=iannucci@chromium.org BUG=chromium:773857 TEST=None Change-Id: Ice5ffe1257333b2bc1bfef7da60dcea4ead4d4c8 Reviewed-on: https://chromium-review.googlesource.com/723162Reviewed-by: Daniel Jacques <dnj@chromium.org> Commit-Queue: Daniel Jacques <dnj@chromium.org>
-
Dan Jacques authored
BUG=chromium:773857 TEST=None Change-Id: I9deb95f7f6c30b5970d74bf3d43df2038db3087e Reviewed-on: https://chromium-review.googlesource.com/721165Reviewed-by: Vadim Shtayura <vadimsh@chromium.org> Reviewed-by: Robbie Iannucci <iannucci@chromium.org> Commit-Queue: Daniel Jacques <dnj@chromium.org>
-
- 14 Oct, 2017 1 commit
-
-
Ryan Tseng authored
To pick up flag changes in kitchen: https://chromium-review.googlesource.com/c/infra/infra/+/718117 Bug: Change-Id: Ib6ce2d76a2e688e1f7da56cf03e1b57d95600e2c Reviewed-on: https://chromium-review.googlesource.com/720250Reviewed-by: Vadim Shtayura <vadimsh@chromium.org> Commit-Queue: Ryan Tseng <hinoka@chromium.org>
-
- 13 Oct, 2017 1 commit
-
-
Michael Moss authored
This fixes an issue introduced by crrev.com/b40a4514 where a 404 result might return None contents. R=agable@google.com, phajdan@google.com Bug:774355 Change-Id: I993a029adb8d92cb27e043cd3d9b17cc8aee2432 Reviewed-on: https://chromium-review.googlesource.com/717937 Commit-Queue: Michael Moss <mmoss@chromium.org> Reviewed-by: Paweł Hajdan Jr. <phajdan.jr@chromium.org>
-
- 12 Oct, 2017 5 commits
-
-
Dirk Pranke authored
Previously, gclient would attempt to write an args file after a dependency was checked out, but before any sub-dependencies had been checked out. If the args file path pointed at something inside a sub-dependency, this wouldn't work, because the directory might not yet exist. This most obviously happened for buildspec clobber builds. The fix is to wait until after the sub-dependencies have been checked out to write the file. R=phajdan.jr@chromium.org, mmoss@chromium.org BUG=773933 Change-Id: I0cf4564204f7dabd9f843dc7904db7050fcc0d23 Reviewed-on: https://chromium-review.googlesource.com/714644Reviewed-by: Paweł Hajdan Jr. <phajdan.jr@chromium.org> Commit-Queue: Dirk Pranke <dpranke@chromium.org>
-
Stefan Zager authored
R=agable@chromium.org,tandrii@chromium.org Bug: Change-Id: I83c285c333a21f2ed0138723a1cd5347e65f41ae Reviewed-on: https://chromium-review.googlesource.com/710742 Commit-Queue: Stefan Zager <szager@chromium.org> Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org> Reviewed-by: Aaron Gable <agable@chromium.org>
-
Michael Moss authored
This is needed in the release recipes because there are times when we want to check that a file is _not_ there yet (i.e. 404 response). R=agable@chromium.org, dnj@chromium.org Change-Id: Iedf646d6c73ebf898e90afd2009840b1e5c5b1ab Reviewed-on: https://chromium-review.googlesource.com/709916Reviewed-by: Paweł Hajdan Jr. <phajdan.jr@chromium.org> Commit-Queue: Michael Moss <mmoss@chromium.org>
-
Ryan Tseng authored
Normally, if dfgs is started with multiple threads, the first time gsutil.py is called, it may be called by many different threads at the same time and hit a race condition where multiple threads are all trying to download gsutil at the same time. This works around the problem by calling it for the first time in the main thread before anything else happens. Bug: 773671 Change-Id: Ie75d90d6d20d11f46a601500b75a198fa5e5f111 Reviewed-on: https://chromium-review.googlesource.com/713750Reviewed-by: Jao-ke Chin-Lee <jchinlee@chromium.org> Commit-Queue: Ryan Tseng <hinoka@chromium.org>
-
Michael Achenbach authored
Bug: 773769 Change-Id: I3c1a394c5995e0c56046da5f2509e63060aa6a29 Reviewed-on: https://chromium-review.googlesource.com/712935Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org> Reviewed-by: Dirk Pranke <dpranke@chromium.org> Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org>
-