summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2015-11-11cros payload: --signatures flag shows the metadata signaturesAlex Deymo
The existing --signatures flag now dumps the metadata signatures embedded in the payload. BUG=None TEST=Added unittests; manually check a signed payload. Change-Id: I4e2eb5cafeabbafd40af31716840d405a79b1023 Reviewed-on: https://chromium-review.googlesource.com/310654 Commit-Ready: Alex Deymo <deymo@chromium.org> Tested-by: Alex Deymo <deymo@chromium.org> Reviewed-by: Sen Jiang <senj@chromium.org>
2015-11-11chromeos_config: Take lakitu out of brilloDaniel Wang
Considering the direction that the Brillo project is going, Lakitu probably doesn't fit in there any more. There are some recent Brillo related changes that aren't applicable to Lakitu, e.g., chrome-os-partner:45210 that enabled HWTests for all Brillo boards. Things like this will continue to happen so let's move early before we find us in a difficult situation. This change tries to keep the config delta as minimal as just to disable hwtests for Lakitu, so the risk of breakage is very low. BUG=chromium:550601 TEST=cbuildbot/runtest cbuildbot/chromeos_config_unittest and compare the generated config_dump.json with unchanged one. Change-Id: I45613b70332f089af347b93916d40bd73fcfc5ef Reviewed-on: https://chromium-review.googlesource.com/311440 Commit-Ready: Daniel Wang <wonderfly@google.com> Tested-by: Daniel Wang <wonderfly@google.com> Reviewed-by: Amey Deshpande <ameyd@google.com>
2015-11-11pushimage: push keyset loading down to allow for input flexibilityMike Frysinger
Sometimes we do want sub-artifacts to sign with a different keyset, like when a firmware builder also produces accessory firmware. The current code will only check the keysets from the main file and then ignore any setting in the sub-file. Instead, delay the keyset loading to after we know the image type so we can pull it from the specific input file. BUG=chrome-os-partner:46635 TEST=`./cbuildbot/run_tests` passes TEST=`./pushimage -n gs://foo/ --board smaug --debug` with custom input insns shows correct output insns Change-Id: I8ed170e43b6e2c115d39dfdc772fbc93054cc9d7 Reviewed-on: https://chromium-review.googlesource.com/311280 Commit-Ready: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org> Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
2015-11-11validation_pool: Split out throttled tree logic.Don Garrett
ApplyPoolIntoRepo contained the logic to decide what CLs to test if the tree was logic. Move that logic out into it's own method. This is prep work to allow intermediate steps that modify the manifest to run after we know exactly what CLs are being considered, but before any of them are applied into the code base. BUG=chromium:549844 TEST=cbuildbot/run_tests Change-Id: I0eb17459c956e4c1fe9e2995e9bd601f279aee72 Reviewed-on: https://chromium-review.googlesource.com/311212 Commit-Ready: Don Garrett <dgarrett@chromium.org> Tested-by: Don Garrett <dgarrett@chromium.org> Reviewed-by: Aviv Keshet <akeshet@chromium.org>
2015-11-11lkgm/sync: Tweak comments/naming.Don Garrett
Just improve readability of the code slightly after trying to read it. BUG=None TEST=cbuildbot/run_tests Change-Id: I9cec3fcacf7c686033e21a0c8aa2278a6711ce2f Reviewed-on: https://chromium-review.googlesource.com/310543 Commit-Ready: Don Garrett <dgarrett@chromium.org> Tested-by: Don Garrett <dgarrett@chromium.org> Reviewed-by: Paul Hobbs <phobbs@google.com> Reviewed-by: Aviv Keshet <akeshet@chromium.org>
2015-11-11CommitQueueSyncStage: Fix --debug run case.Don Garrett
When run with --debug and no CIDB credentials, the master-paladin config would fail by using a db connection of None. Work around that. BUG=None TEST=bin/cbuildbot --buildroot ~/tmp/build_master --debug --buildbot --branch master --buildnumber 37 master-paladin Change-Id: I9f4e1ef2157124c268252b1c8e5fc59f85049826 Reviewed-on: https://chromium-review.googlesource.com/310874 Commit-Ready: Don Garrett <dgarrett@chromium.org> Tested-by: Don Garrett <dgarrett@chromium.org> Reviewed-by: Paul Hobbs <phobbs@google.com> Reviewed-by: Aviv Keshet <akeshet@chromium.org>
2015-11-11chromeos_config: temporarily disable jetstream testsAviv Keshet
BUG=chrome-os-partner:47563 TEST=None Change-Id: Ic687b97d07f078fca379b094d53cff88a9a50779 Reviewed-on: https://chromium-review.googlesource.com/312098 Trybot-Ready: Aviv Keshet <akeshet@chromium.org> Tested-by: Aviv Keshet <akeshet@chromium.org> Reviewed-by: David James <davidjames@chromium.org>
2015-11-10Fix cleanup function to be a bit more robust.David James
Example failure: https://uberchromegw/i/chromeos/builders/whirlwind%20paladin/builds/3673 TBR=akeshet BUG=chromium:55250 TEST=unit test Change-Id: I079b11ef90e05e23323010fd92e6527885adacbd Reviewed-on: https://chromium-review.googlesource.com/311783 Tested-by: David James <davidjames@chromium.org> Reviewed-by: Fang Deng <fdeng@chromium.org>
2015-11-06Update builders to delete git/obj dirs when gc --auto breaks.David James
This allows builders to recover from git corruption. BUG=chromium:552503 TEST=Unit tests. Trybot run. Change-Id: I06b85c12a6ef4ac1a9193f6759d18b88d8d4724a Reviewed-on: https://chromium-review.googlesource.com/311423 Reviewed-by: Richard Barnette <jrbarnette@chromium.org> Reviewed-by: David James <davidjames@chromium.org> Commit-Queue: David James <davidjames@chromium.org> Trybot-Ready: David James <davidjames@chromium.org> Tested-by: David James <davidjames@chromium.org>
2015-11-03pushimage: add support for signing accessories imagesVincent Palatin
'accessory_usbpd' and 'accessory_rwsig' are used to sign standalone accesory MCU firmwares (right now, the former are all the USB type-C accessories: e.g. Zinger power supply, DingDong and HoHo display dongles, the latter is the Smaug wireless keyboard). Signed-off-by: Vincent Palatin <vpalatin@chromium.org> BUG=chrome-os-partner:46635 TEST=bin/pushimage --debug --test-sign premp --sign-types accessory_rwsig --board smaug gs://chromeos-image-archive/smaug-firmware/R45-7132.239.0 TEST=./scripts/pushimage_unittest Change-Id: I6481c68061f50b9acc3656a1167d64ced79ec011 Reviewed-on: https://chromium-review.googlesource.com/308725 Commit-Ready: Vincent Palatin <vpalatin@chromium.org> Tested-by: Vincent Palatin <vpalatin@chromium.org> Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
2015-11-03paygen: Generate the payload hash and metadata hash together.Sen Jiang
Due to recent changes in delta_generator, payload hash and metadata hash are calculated together, calling it twice are just doing repeated work. BUG=None TEST=./paygen_payload_lib_unittest Change-Id: I62c6a3621e2da882a5acd8209c17686152810806 Reviewed-on: https://chromium-review.googlesource.com/310226 Commit-Ready: Alex Deymo <deymo@chromium.org> Tested-by: Sen Jiang <senj@chromium.org> Reviewed-by: Matthew Sartori <msartori@chromium.org> Reviewed-by: Alex Deymo <deymo@chromium.org>
2015-10-31Cleanup and reapply uprevs in PublishUprev in CQ.David James
Currently the CQ reuses old uprevs from the Uprev stage and this is error-prone because the SHA1s can be out of date. We have robust logic for fixing this but it only executes when the commit queue fails. This has the nice side effect of also making the commit queue more robust to conflicting with other changes, since it resyncs and reapplies all changes from scratch at the end, rather than trying to do a complex rebase that could run into conflicts. This logic is quite well tested and has been in use for a long time, so it's time we switch to it. I've also deleted the old UpdateCommitHashesForChanges function. This logic wasn't working (due to the fact that it was running on the wrong branch after phobbs' changes) and with my change above, is now obsolete. BUG=chromium:549839, chromium:547541 TEST=Unit tests. Change-Id: I9174504db4f468e04ec7a1c60091c16077be32c1 Reviewed-on: https://chromium-review.googlesource.com/310032 Trybot-Ready: David James <davidjames@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org> Tested-by: David James <davidjames@chromium.org>
2015-10-30Revert "Temporarily remove smaug from list of important builders in CQ."Furquan Shaikh
This reverts commit e6c578e5c00a4bff28999370af0ce3eef4b417e8. smaug-paladin is green again. Thus, reverting this commit. Change-Id: I36e28e11584cba8c691d0587f29d9d906cb36361 Reviewed-on: https://chromium-review.googlesource.com/309897 Commit-Ready: Furquan Shaikh <furquan@chromium.org> Tested-by: Furquan Shaikh <furquan@chromium.org> Reviewed-by: Luigi Semenzato <semenzato@chromium.org>
2015-10-30cros_mark_as_stable: Checkout before syncingPaul Hobbs
"git.SyncPushBranch" rebases HEAD onto the updated remote. We need to checkout the stable_branch before syncing in order to make sure that it has the remotes changes. TEST=unittests BUG=chromium:547541 Change-Id: Iaa7fa900ebb85514dc12fb5b12ed92a205e36103 Reviewed-on: https://chromium-review.googlesource.com/309898 Reviewed-by: Mike Frysinger <vapier@chromium.org> Tested-by: Matthew Sartori <msartori@chromium.org>
2015-10-30Surface bugs or CLs responsible for build failures.David James
BUG=none TEST=Run the script. Change-Id: I9cc0db1a82968c443e0f364d7598ac4bb056c33c Reviewed-on: https://chromium-review.googlesource.com/308588 Commit-Ready: David James <davidjames@chromium.org> Tested-by: David James <davidjames@chromium.org> Reviewed-by: Paul Hobbs <phobbs@google.com>
2015-10-30lint: add check for unittest module namesMike Frysinger
Make sure people don't actually try to use foo_unittests.py. BUG=None TEST=tests pass & no new lint errors Change-Id: Ic25f580c608f21aab0f006bc304a92ac8c8e23d9 Reviewed-on: https://chromium-review.googlesource.com/309610 Commit-Ready: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org> Reviewed-by: Don Garrett <dgarrett@chromium.org>
2015-10-30Add jetstream_cq hwtest to the commit queue, nonblocking.John Carey
BUG=chrome-os-partner:45210 TEST=tested in roundabout ways Change-Id: I2d76270c3b94039c98b7510c53bc2d8a35b0b80a Reviewed-on: https://chromium-review.googlesource.com/309577 Commit-Ready: John Carey <ranix@google.com> Tested-by: John Carey <ranix@google.com> Reviewed-by: John Carey <ranix@google.com> Reviewed-by: David James <davidjames@chromium.org> Reviewed-by: Adam Haberlach <ahaberlach@chromium.org>
2015-10-30Add terra builder configBernie Thompson
BUG=chrome-os-partner:46775 TEST=None Change-Id: Idb9a3bd8ea091b93d1763703930a4f3200ae8253 Reviewed-on: https://chromium-review.googlesource.com/308252 Commit-Ready: Bernie Thompson <bhthompson@chromium.org> Tested-by: Bernie Thompson <bhthompson@chromium.org> Reviewed-by: Shawn N <shawnn@chromium.org>
2015-10-30Rename unittests without matching module.Don Garrett
We are considering adding a cros lint test to ensure that unittests match a general library. This CL fixes all existing cases in chromite where that isn't already true. Note, in one case, the unittest was for logic in the __init__.py while, which is now tested as 'init_unittest.py'. This may require specific lint support, or further tweaking. BUG=None TEST=run_tests Change-Id: If8cb3e9f4ab90f4b518c5f7ea7e66c47240de501 Reviewed-on: https://chromium-review.googlesource.com/309860 Commit-Ready: Don Garrett <dgarrett@chromium.org> Tested-by: Don Garrett <dgarrett@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org>
2015-10-30lib/launch_control: Add library.Don Garrett
Move two libraries for working with Launch Control over, so that they can be shared between multiple Site Specific cbuildbot builders. BUG=chromium:547996 TEST=Unitests Change-Id: I812c7573494dfed149d2cc6ffcb184abb78a6f60 Reviewed-on: https://chromium-review.googlesource.com/308754 Commit-Ready: Don Garrett <dgarrett@chromium.org> Tested-by: Don Garrett <dgarrett@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org>
2015-10-30cros_mark_as_stable: checkout stablizing_branchPaul Hobbs
In order to make the commit on stabilizing_branch we have to actually be *on* stablizing_branch. BUG=chromium:547541 TEST=unit tests. Change-Id: I2c64b0c8c4f0061ee90a6e154a182239fd0208b2 Reviewed-on: https://chromium-review.googlesource.com/308731 Reviewed-by: Paul Hobbs <phobbs@google.com> Tested-by: Paul Hobbs <phobbs@google.com> Commit-Queue: Paul Hobbs <phobbs@google.com>
2015-10-30Temporarily remove smaug from list of important builders in CQ.David James
BUG=chromium:549434 TEST=none Change-Id: I55a44c91b7f558ee2463dfde46bddc43eb5180d4 Reviewed-on: https://chromium-review.googlesource.com/309961 Reviewed-by: David James <davidjames@chromium.org> Tested-by: David James <davidjames@chromium.org>
2015-10-27paygen: Fix GenMetadataHash to pass out_hash_file to delta_generator.Sen Jiang
delta_generator now needs out_hash_file flag to be passed when calculating hash. Just a quick fix here, will refactor to combine _GenPayloadHash and _GenMetadataHash together later. BUG=chromium:548257 TEST=./paygen_payload_lib_unittest Change-Id: Ie3b9ec861cb9faa861200b1966aca178a7ecbac4 Reviewed-on: https://chromium-review.googlesource.com/309029 Commit-Ready: Alex Deymo <deymo@chromium.org> Tested-by: Sen Jiang <senj@chromium.org> Reviewed-by: Alex Deymo <deymo@chromium.org>
2015-10-27chromeos_config: Mark glados paladin as importantShawn Nematbakhsh
glados paladin is green, so mark it as important so we get preflight prebuilt updates. BUG=chrome-os-partner:45196 TEST=Pass unittests. Change-Id: I9127a0240d3a9f6780eaf9c25dba27bd813a00aa Signed-off-by: Shawn Nematbakhsh <shawnn@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/309011 Commit-Ready: Shawn N <shawnn@chromium.org> Tested-by: Shawn N <shawnn@chromium.org> Reviewed-by: Bernie Thompson <bhthompson@chromium.org>
2015-10-27Remove toolchain-major builder.Luis Lozano
The compiler branch this builder is using is not being maintained. Does not make sense to try to build this for now. BUG=chromium:538675 TEST=run_tests Change-Id: I430d9b9a6b9987825459689d512d9bba46d2d4bb Reviewed-on: https://chromium-review.googlesource.com/308752 Commit-Ready: Luis Lozano <llozano@chromium.org> Tested-by: Luis Lozano <llozano@chromium.org> Reviewed-by: Han Shen <shenhan@chromium.org>
2015-10-23Test that branch exists before pushing.David James
BUG=chromium:545779 TEST=pre-cq Change-Id: I349167b31891cbf69853417d3afc82bf70daa7fa Reviewed-on: https://chromium-review.googlesource.com/308420 Commit-Ready: David James <davidjames@chromium.org> Tested-by: David James <davidjames@chromium.org> Reviewed-by: Jonathan Ross <jonross@google.com> Reviewed-by: Paul Hobbs <phobbs@google.com> Reviewed-by: Mike Frysinger <vapier@chromium.org>
2015-10-23test_stages: Fix usage of PrintBuildbotStepTextPaul Hobbs
In the code that warns that we failed to generate delta payloads, test_stages misuses "PrintBuildbotStepWarnings" by providing it a string as a file-handle as its argument. This causes an AttributeError when we try to write to it. BUG=chromium:541258 TEST=None Change-Id: I063e47ea5bf7b410e3cc0edf767abac611a0c288 Reviewed-on: https://chromium-review.googlesource.com/304595 Commit-Ready: Paul Hobbs <phobbs@google.com> Tested-by: Paul Hobbs <phobbs@google.com> Reviewed-by: Mungyung Ryu <mkryu@google.com>
2015-10-23commands: Only copy images if not present.Don Garrett
Images are being touched in the archive directory while being tarred. There is no clear explaination, but it's possible they are being copied into location more than once. This CL may/may not fix the referenced bug, but it shouldn't hurt anything. BUG=chromium:547055 TEST=cbuildbot/run_tests Change-Id: I6df9cf13b600b39f881e9ea275c9ce5ec5351235 Reviewed-on: https://chromium-review.googlesource.com/308550 Commit-Ready: Don Garrett <dgarrett@chromium.org> Tested-by: Don Garrett <dgarrett@chromium.org> Reviewed-by: Daniel Wang <wonderfly@google.com>
2015-10-23config_lib: Update misc users to config_lib.GetConfig.Don Garrett
Various scripts were left using the unpreferred (and uncached) config_lib.LoadConfigFromFile when they should be using config_lib.GetConfig. The result is the same, but GetConfig only loads/parses from file once per run. BUG=None TEST=run_tests Change-Id: Ia9932390e79a3dc964eea4ae40f45d270af26ce0 Reviewed-on: https://chromium-review.googlesource.com/308244 Commit-Ready: Don Garrett <dgarrett@chromium.org> Tested-by: Don Garrett <dgarrett@chromium.org> Reviewed-by: Matthew Sartori <msartori@chromium.org>
2015-10-23config_lib: DEFAULT_ARCHIVE_BUCKET -> config.params.ARCHIVE_URLDon Garrett
Make constants.DEFAULT_ARCHIVE_BUCKET configurable as a site parameter. Update all the code that was using the constant. Since cros_chrome_sdk was replicating logic from archive_lib, have it use a helper method, instead of computing a base URL itself. BUG=None TEST=cbuildbot/run_tests Change-Id: I56bfaf1aaaa96777a79219b8d805c99531ced636 Reviewed-on: https://chromium-review.googlesource.com/308267 Commit-Ready: Don Garrett <dgarrett@chromium.org> Tested-by: Don Garrett <dgarrett@chromium.org> Reviewed-by: Matthew Sartori <msartori@chromium.org> Reviewed-by: Fang Deng <fdeng@chromium.org>
2015-10-22Add veyron_{mighty,speedy} to the ChromeOS waterfall.David James
Leave them marked as unimportant for now, so that we can wait for the waterfall to be restarted, and for its stability/speed to be proven. BUG=chromium:545596 TEST=Unit tests. Change-Id: I5266a4312a79bdba80f24c2bd2803a2696b69460 Reviewed-on: https://chromium-review.googlesource.com/308160 Commit-Ready: David James <davidjames@chromium.org> Tested-by: David James <davidjames@chromium.org> Reviewed-by: Daniel Jacques <dnj@chromium.org>
2015-10-22cros_mark_as_stable: remove HEAD state dependencyPaul Hobbs
"cros_mark_as_stable push" requires the repository to already be on stabilizing_branch, otherwise it will not push ebuild uprevs. Unfortunately, due to git-based submission the repo may be on "patch_branch" if changes were re-applied in the CommitQueueCompletion stage. This patch removes the dependency on HEAD being in a certain state. BUG=chromium:545779 TEST=unit tests. Change-Id: I604a46a01657f9bcd47f17b03b638500288dca68 Reviewed-on: https://chromium-review.googlesource.com/307866 Commit-Ready: Paul Hobbs <phobbs@google.com> Tested-by: Paul Hobbs <phobbs@google.com> Reviewed-by: Mike Frysinger <vapier@chromium.org>
2015-10-21manifest_version: Fail the build if slaves timeout after 5 mins.Kevin Cheng
I created a new class SlaveStatus to deal with the logic of deciding whether or not to stop monitoring the build status. The only early kickout situation is if a slave hasn't started within 5 mins and the other slaves have finished building. More situations are coming but those will come in subsequent cls. BUG=chromium:537791 TEST=Existing unittests pass and added new unittests for SlaveStatus. Change-Id: If682f6fff3782353ec8285f15775f4e788e1e5c5 Reviewed-on: https://chromium-review.googlesource.com/306701 Commit-Ready: Kevin Cheng <kevcheng@chromium.org> Tested-by: Kevin Cheng <kevcheng@chromium.org> Reviewed-by: Kevin Cheng <kevcheng@chromium.org>
2015-10-21cros lint: detect & warn about trailing newlinesMike Frysinger
So we don't have to have reviewers catch these errors, add a linter check. BUG=None TEST=`cros lint` detected a few files with bad trailing newlines Change-Id: I6dd90e25516e77902fa91227b2d4c4036cc7d11a Reviewed-on: https://chromium-review.googlesource.com/307175 Commit-Ready: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org> Reviewed-by: Paul Hobbs <phobbs@google.com>
2015-10-21lint: clean up trailing whitespace/disable-msg/unused importsMike Frysinger
A new lint hook will catch these, so make sure they're cleaned up before we land it. Also clean up a few other lint errors that have crept into the codebase somehow. BUG=None TEST=`cros lint` is clean after new trailing newline check Change-Id: Ifdcc8ef72ee944511a821193857bfbf4b2d3ad7b Reviewed-on: https://chromium-review.googlesource.com/307174 Commit-Ready: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org> Reviewed-by: Daniel Wang <wonderfly@google.com>
2015-10-20manifest_version_unittest: convert mox to mockMike Frysinger
BUG=chromium:430770 TEST=`./cbuildbot/run_tests` passes Change-Id: I8b67807b82bbc2d56d5bfdc5dab666246b4f5627 Reviewed-on: https://chromium-review.googlesource.com/307510 Commit-Ready: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org> Reviewed-by: Paul Hobbs <phobbs@google.com> Reviewed-by: David James <davidjames@chromium.org>
2015-10-20Don't use LKGM anymore on Chrome builders.David James
BUG=chromium:239219 TEST=Run it. Change-Id: I146cc0ef329767a9e05964114cb4c8767a1712d4 Reviewed-on: https://chromium-review.googlesource.com/307186 Commit-Ready: David James <davidjames@chromium.org> Tested-by: David James <davidjames@chromium.org> Reviewed-by: Paul Hobbs <phobbs@google.com> Reviewed-by: Mike Frysinger <vapier@chromium.org>
2015-10-19chromeos_config: migrate wizpig from rambi to stragoBernie Thompson
BUG=None TEST=None Change-Id: I67edda71d5f7504c80fc8ed3eaa6ed59749f4bcc Reviewed-on: https://chromium-review.googlesource.com/304670 Commit-Ready: Bernie Thompson <bhthompson@chromium.org> Tested-by: Bernie Thompson <bhthompson@chromium.org> Reviewed-by: Aaron Durbin <adurbin@chromium.org>
2015-10-16Disable vmtests for cyan-cheetsBernie Thompson
VMTest does not currently pass on cyan-cheets, and will not for some time, so we should disable it. BUG=None TEST=None Change-Id: I2a0e4a0b4bde19185f121545d347137489c52f05 Reviewed-on: https://chromium-review.googlesource.com/306730 Commit-Ready: Bernie Thompson <bhthompson@chromium.org> Tested-by: Bernie Thompson <bhthompson@chromium.org> Reviewed-by: Luigi Semenzato <semenzato@chromium.org> Reviewed-by: David James <davidjames@chromium.org>
2015-10-16chromeos_config: make Glados important and add a Glados paladinBernie Thompson
BUG=chrome-os-partner:46453 TEST=None Change-Id: Ie028ef34b1e30e67112247705716aeffaf08b928 Reviewed-on: https://chromium-review.googlesource.com/305515 Commit-Ready: Bernie Thompson <bhthompson@chromium.org> Commit-Ready: Shawn N <shawnn@chromium.org> Tested-by: Bernie Thompson <bhthompson@chromium.org> Tested-by: Shawn N <shawnn@chromium.org> Reviewed-by: David James <davidjames@chromium.org>
2015-10-14compute: Use Git slave checkouts.Dan Jacques
BUG=chromium:533026 TEST=None Change-Id: Ied8f9b500ccada49be90d3d0908b37eff5bcca87 Reviewed-on: https://chromium-review.googlesource.com/305220 Commit-Ready: Daniel Jacques <dnj@chromium.org> Tested-by: Daniel Jacques <dnj@chromium.org> Reviewed-by: Chris Sosa <sosa@chromium.org>
2015-10-14run_tests: run tests in order of timing lengthMike Frysinger
The test runner has no idea how long a particular test might take, so it runs them all in whatever order it finds them on disk. This means that on parallel cpus, we might schedule a long running test later on and then drain the remaining fast tests leaving just the one. If we instead ran that long test first, we'd be able to execute everything else in parallel largely guaranteeing the runtime of run_tests is the length of the longest test. BUG=None TEST=ran ./cbuildbot/run_tests a lot and saw less variation in overall runtime Change-Id: I194a0639d4cd188f8d4616486e7a1708971e1e6b Reviewed-on: https://chromium-review.googlesource.com/241042 Commit-Ready: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org> Reviewed-by: David James <davidjames@chromium.org>
2015-10-14paygen: Re-enable generation of NMO update test payloads.Matthew Sartori
This CL adds more cataloging of NMO/previous builds during the build discovery phase of paygen. This information is used to better control which test payloads we generate. This CL removes any usage of '_previous_versions' in paygen and allows for all discovered 'previous' builds to have update test payloads generated. BUG=chromium:541417 TEST=Unittests. Change-Id: I59601997423f6a839b4e2a88bb5f6d87123746c2 Reviewed-on: https://chromium-review.googlesource.com/305332 Commit-Ready: Matthew Sartori <msartori@chromium.org> Tested-by: Matthew Sartori <msartori@chromium.org> Reviewed-by: Don Garrett <dgarrett@chromium.org>
2015-10-13wrapper: support wrapping scripts outside of chromiteMike Frysinger
Sometimes we want to prototype/land code in other repos, such as when we want to keep something private before launching it. Extend the wrapper logic to support such cases by loading the module directly via imp. BUG=None TEST=created symlink outside of chromite dir and was able to run/import chromite code Change-Id: If0c24854dd7758691d48dfbb4087410a574fc3b4 Reviewed-on: https://chromium-review.googlesource.com/305550 Commit-Ready: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org>
2015-10-13Remove veyron-thea builder configBernie Thompson
BUG=chrome-os-partner:45839 TEST=None Change-Id: I0e58582f1246394a4cc8c2836f7d80136967e38e Reviewed-on: https://chromium-review.googlesource.com/304671 Commit-Ready: Bernie Thompson <bhthompson@chromium.org> Tested-by: Bernie Thompson <bhthompson@chromium.org> Reviewed-by: Bernie Thompson <bhthompson@chromium.org>
2015-10-13Add veyron_minnie-cheets canary builderBernie Thompson
BUG=chromium:538232 TEST=None Change-Id: I9180eb85b0b2a7e0b137e6c5218c77b334f1f163 Reviewed-on: https://chromium-review.googlesource.com/303434 Commit-Ready: Bernie Thompson <bhthompson@chromium.org> Tested-by: Bernie Thompson <bhthompson@chromium.org> Reviewed-by: Bernie Thompson <bhthompson@chromium.org>
2015-10-13workspace_lib: Remove this library.Don Garrett
This library partially implemented the Chrome OS Brillo concept of a workspace. That concept is no longer relevant, so remove the library. Update assorted users, removing the concept of workspace where it seems appropriate. BUG=chromium:530677 TEST=run_tests Change-Id: Icb43d24428f3c35e89e6c90325cbd03382670ffd Reviewed-on: https://chromium-review.googlesource.com/305025 Commit-Ready: Don Garrett <dgarrett@chromium.org> Tested-by: Don Garrett <dgarrett@chromium.org> Reviewed-by: Don Garrett <dgarrett@chromium.org>
2015-10-13brick_lib: Remove this library.Don Garrett
This library partially implemented the Chrome OS Brillo concept of a brick. That concept is no longer relevant, so remove the library. Update assorted users, removing the concept of brick where it seems appropriate. BUG=chromium:530677 TEST=run_tests Change-Id: I3d768ffd8c16b559cca1eb1e459be20d8e912492 Reviewed-on: https://chromium-review.googlesource.com/305024 Commit-Ready: Don Garrett <dgarrett@chromium.org> Tested-by: Don Garrett <dgarrett@chromium.org> Reviewed-by: Don Garrett <dgarrett@chromium.org>
2015-10-13blueprint_lib: Remove this library.Don Garrett
This library partially implemented the Chrome OS Brillo concept of a blueprint. That concept is no longer relevant, so remove the library. Also, remove the related concept of a BSP. BUG=chromium:530677 TEST=run_tests Change-Id: I18fd43232a0093913f533301acaa25658951fe12 Reviewed-on: https://chromium-review.googlesource.com/305023 Commit-Ready: Don Garrett <dgarrett@chromium.org> Tested-by: Don Garrett <dgarrett@chromium.org> Reviewed-by: Don Garrett <dgarrett@chromium.org> Reviewed-by: Bertrand Simonnet <bsimonnet@google.com>
2015-10-13bootstrap_lib: Remove this library.Don Garrett
This library partially implemented the Chrome OS Brillo concept of bootstrapping an SDK checkout, and working with that type of checkout. That concept is no longer relevant, so remove the library. BUG=chromium:530677 TEST=run_tests Change-Id: If719aeecb5b05fdd4e376c5c7ebd1f79ff05fab1 Reviewed-on: https://chromium-review.googlesource.com/305022 Commit-Ready: Don Garrett <dgarrett@chromium.org> Tested-by: Don Garrett <dgarrett@chromium.org> Reviewed-by: Don Garrett <dgarrett@chromium.org>