aboutsummaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2013-12-10Use default results for Telemetry tests; add option to use all results.cmtice
When running telemetry_Crosperf, use the json file of report defaults to determine which results to actually include in the report. This CL also adds a flag to Crospref, --show_all_results=True, that allows the user to override the defaults and generate a report showing all the values. BUG=None TEST=Tested this both with and without the --show_all_results flag. It works as expected for me. Change-Id: I91364572ebdf3d54bb8b408fbd5a2580923f6829 Reviewed-on: https://chrome-internal-review.googlesource.com/149457 Reviewed-by: Yunlian Jiang <yunlian@google.com> Commit-Queue: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com>
2013-12-07Check for file before prepending directory path.cmtice
Sometimes _GetDataMeasurementsFiles returns the file with the full directory path, and sometimes it doesn't. We were assuming it always did not. This patch checks to see if we can already find the file before attempting to prepend the directory path. BUG=None TEST=Tested this in the nightly infrastructure tests area, where we were seeing the problem. This fixed that problem. (Before this fix we were not seeing the successful test results; after it we are.) Change-Id: Ic0a2df4c9aff7b820f652f7ecad56976ed97a4b0 Reviewed-on: https://chrome-internal-review.googlesource.com/149256 Reviewed-by: Luis Lozano <llozano@chromium.org> Commit-Queue: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com>
2013-12-04Make weekly sheriff email point to new web page.cmtice
BUG=None TEST=None Change-Id: I20eb69c42da4b96909c09931c8c7f4fda1b57abd Reviewed-on: https://chrome-internal-review.googlesource.com/149000 Reviewed-by: Han Shen <shenhan@google.com> Commit-Queue: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com>
2013-12-02Add the ability to pass arguments through test_that to Telemetry'scmtice
run_benchmark. BUG=None TEST=I have tested this with my own crosperf, and test_that is passing the arguments along properly. The telemetry_Crosperf autotest also needs to be updated before this completely works. Change-Id: Ic887380b297d966229ac21ba4e7e909e7248fcb0 Reviewed-on: https://chrome-internal-review.googlesource.com/148485 Reviewed-by: Yunlian Jiang <yunlian@google.com> Commit-Queue: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com>
2013-12-02Update machine_manager.py to deal with new 'ifconfig' output.cmtice
BUG=None TEST=Tested in on updated lab parrot machines and it worked. Change-Id: I709989a977130bbeeb667e825acdc874c6bfaf50 Reviewed-on: https://chrome-internal-review.googlesource.com/148714 Reviewed-by: Yunlian Jiang <yunlian@google.com> Commit-Queue: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com> Reviewed-by: Bhaskar Janakiraman <bjanakiraman@google.com>
2013-11-22Update report generation to check for a 'perf_measurements' file.cmtice
The latest version of telemetry_Crosperf generates a perf_measurements file, containing the test results, rather than a keyval file. This CL checks for perf_measurements, and reads it, if it is there. This change is backwards compatible with the older versions of telemetry_Crosperf that used the keyval file. BUG=None TEST=Tested this with the results only in a keyval file and only in a perf_measurements file. Got acceptable test reports either way. Change-Id: If10a62b2f7fbe89d35c1ebe789a81764c9d134e4 Reviewed-on: https://chrome-internal-review.googlesource.com/148295 Reviewed-by: Luis Lozano <llozano@chromium.org> Commit-Queue: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com>
2013-11-19Add example experiment files for Telemetry & test_that.cmtice
Change-Id: I7bbf5e3816bafe7fa8edb39c7a77c0c9b527fc82 Reviewed-on: https://chrome-internal-review.googlesource.com/147866 Reviewed-by: Luis Lozano <llozano@chromium.org> Commit-Queue: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com>
2013-11-06Make changes to run Telemetry tests in nightly infrastructure tests.cmtice
BUG=None TEST=Tested this running the nightly infrastructure cron jobs. Change-Id: I6cccee222b074e9410b45df6750c097430207746 Reviewed-on: https://chrome-internal-review.googlesource.com/147515 Reviewed-by: Luis Lozano <llozano@chromium.org> Commit-Queue: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com>
2013-11-05Add file to bypass most of the commit queue tests.cmtice
Since changes to toolchain-utils will not affect them anyway, bypass the commit queue tests, so if they fail for some other reason it won't prevent toolchain-utils tests from being committed. This is according to the instructions on: www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium-os/commit-queue-overview/ bypassing-tests-on-a-per-project-basis BUG=None Test=None Change-Id: I17ae4591639f9113da9e85354ce81e90eba4aa4b Reviewed-on: https://chrome-internal-review.googlesource.com/147525 Reviewed-by: Caroline Tice <cmtice@google.com> Commit-Queue: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com>
2013-10-23Accept hyphens in test names; fix profiler args string.cmtice
Updated regular expression parser to accept hyphens in the test names. Also fixed the quotes on profiler args, for proper parsing by test_that. BUG=None TEST=Ran crosperf successfully with benchmark name that had hyphen in it; successfully ran crosperf with profiler args. Change-Id: I4a154e390a5fabfc47e0783212fed928eb5e9873 Reviewed-on: https://chrome-internal-review.googlesource.com/146967 Reviewed-by: Yunlian Jiang <yunlian@google.com> Commit-Queue: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com>
2013-10-23Fixed name of telemetry benchmark page_cycler.basic_insertLuis Lozano
BUG=None TEST=Tested by hand using the new name Change-Id: I7246f9f5d2a5e17133613bc4f9038fe1b42332e9 Reviewed-on: https://chrome-internal-review.googlesource.com/146909 Reviewed-by: Caroline Tice <cmtice@google.com> Commit-Queue: Luis Lozano <llozano@chromium.org> Tested-by: Luis Lozano <llozano@chromium.org>
2013-10-10Pin max and min frequencies on machine before running experiment.Luis Lozano
Find out what is the highest static frequency (turbo mode is considered a dynamic frequency) and set the min and max frequency to the highest static. This reduces benchmark noise in particular for some machines like daisy and lumpy i5. BUG=None TEST=Tested with runs of perf_v2. Verified frequencies are pinned. Change-Id: Idd0627a0f53fa986d0ccf1da2bee53d51a5c8ccb Reviewed-on: https://chrome-internal-review.googlesource.com/146206 Reviewed-by: Simon Que <sque@google.com> Tested-by: Luis Lozano <llozano@chromium.org> Commit-Queue: Luis Lozano <llozano@chromium.org>
2013-10-09Added the test name patterns for Telemetry perf v2 benchmarks directionLuis Lozano
Added test name patterns to find the direction (improvement/degradation) of benchmarks for all the perf_v2 except memory.top25 (which is broken right now). This way of detecting benchmark direction is pretty bridle. Telemetry should provide this information explicitly. Change-Id: Ifdccec1d9955f9b7a5790a1fa5ebabe98669496d Reviewed-on: https://chrome-internal-review.googlesource.com/146216 Reviewed-by: Caroline Tice <cmtice@google.com> Commit-Queue: Luis Lozano <llozano@chromium.org> Tested-by: Luis Lozano <llozano@chromium.org>
2013-10-09dejagnu: change gdb baseline for gdb 7.6.1Yunlian Jiang
BUG=None TEST=None Change-Id: I6ce965f9de03edf6147f9eb4791e31b4762dc720 Reviewed-on: https://chrome-internal-review.googlesource.com/146219 Reviewed-by: Yunlian Jiang <yunlian@google.com> Commit-Queue: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com>
2013-10-07crosperf: remove cache if --rerun=True is set.Yunlian Jiang
If rerun=True, the data in the cache is useless, so remove it first to avoid data corrupture. BUG=None TEST=The cache dir is removed. Change-Id: I59b219c57cc99aaea7a734ef6e7fd91007cedfd3 Reviewed-on: https://chrome-internal-review.googlesource.com/146065 Reviewed-by: Luis Lozano <llozano@chromium.org> Reviewed-by: Yunlian Jiang <yunlian@google.com> Commit-Queue: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com>
2013-10-03gdb_dejagnu: print failed tests out.Yunlian Jiang
This prints failed test name out. Also, it ignores the tests that only appears in the baseline. BUG=None TEST=the name of failed tests were printed. Change-Id: Ie0c3f3d3a092a184a2e941020af62d4e2de40da2 Reviewed-on: https://chrome-internal-review.googlesource.com/145848 Reviewed-by: Simon Que <sque@google.com> Reviewed-by: Han Shen <shenhan@google.com> Commit-Queue: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com>
2013-10-03Remove (at least temporarily) the --minilayout argument for checking out ↵Caroline Tice
ChromeOS, so the nightly infrastructure tests can build. BUG=None TEST=None Change-Id: I7a6c68142e0c9af106a047986a95a545ece6a094 Reviewed-on: https://chrome-internal-review.googlesource.com/145917 Reviewed-by: Yunlian Jiang <yunlian@google.com> Commit-Queue: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com>
2013-10-03gdb_dejagnu: update base line.Yunlian Jiang
BUG=None TEST=None Change-Id: I11c0c2ed6af9bd7a395dede27668046af008942c Reviewed-on: https://chrome-internal-review.googlesource.com/145937 Reviewed-by: Yunlian Jiang <yunlian@google.com> Commit-Queue: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com>
2013-10-02gdb_dejagnu: add space to ssh parameter.Yunlian Jiang
BUG=None TEST=It passes. Change-Id: If7413a602b58fce3b0f9472d45ca4e1a35501551 Reviewed-on: https://chrome-internal-review.googlesource.com/145847 Reviewed-by: Yunlian Jiang <yunlian@google.com> Commit-Queue: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com>
2013-10-02gdb_dejiagnu: use machine.name instead of machine.Yunlian Jiang
BUG=None TEST=It can run now. Change-Id: I53b4596c46341b600097f7cd4154b6945e970492 Reviewed-on: https://chrome-internal-review.googlesource.com/145895 Reviewed-by: Yunlian Jiang <yunlian@google.com> Commit-Queue: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com>
2013-10-02Allow "." in benchmark names; Allow multiple test sets.Caroline Tice
This fixes a few minor problems with crosperf: 1). When using run_remote_tests.sh, it adds the "--use_emerged" flag, which is a recent requirement. 2). It updates the regular expression parsing to not mess up escaped characters, and to allow "." in benchmark names (common in Telemetry). 3). It allows there to be multiple "sets" of benchmarks that can be specified together, e.g. perfv2 or pagecyclers (or more in the future). BUG=None TEST=I've run all the various telemetry_Crosperf variations (individual perf_v2 tests; individual telemetry page cycler tests; 'all_perf v2', and 'all_pagecyclers'. Change-Id: Idd45caa6487780b92b8eb321b37b873978f96dca Reviewed-on: https://chrome-internal-review.googlesource.com/145305 Reviewed-by: Yunlian Jiang <yunlian@google.com> Tested-by: Caroline Tice <cmtice@google.com> Commit-Queue: Caroline Tice <cmtice@google.com>
2013-10-01crosperf: fix unittest for crosperf.Yunlian Jiang
This fixes several issues for unittesting. Now unittest seems to be working. BUG=None TEST=crosperf/run_tests.sh passes. Change-Id: I1dcef7ba55d45fbd00e453f749a8b10dea0332e6 Reviewed-on: https://chrome-internal-review.googlesource.com/145635 Reviewed-by: Caroline Tice <cmtice@google.com> Commit-Queue: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com>
2013-10-01Rewrite git/gerrit/gerrit-int urls to googlesourceAaron Gable
BUG=chromium:281665,chromium:290413 TEST=None Change-Id: I0770e6dc963afaaf012d4925ac1b9bbcc17d01ea Reviewed-on: https://chrome-internal-review.googlesource.com/145692 Reviewed-by: Mike Frysinger <vapier@google.com> Commit-Queue: Aaron Gable <agable@google.com> Tested-by: Aaron Gable <agable@google.com>
2013-09-23Update crosperf to run telemetry test via autotest.Caroline Tice
This updates crosperf to recognize the suite 'telemetry_Crosperf', which allows users to run telemetry performance tests via test_that and the telemetry_Crosperf autotest. This won't be usable in general until CL 168491 (the telemetry_Crosperf CL) gets committed. BUG=None TEST=Invoked telemetry tests using this change. Note: Profilers don't quite work properly with telemetry_Crosperf yet, but everything else does. Change-Id: Id14edc89db79bc244496e8f6a5b614300a00be54 Reviewed-on: https://chrome-internal-review.googlesource.com/144440 Reviewed-by: Caroline Tice <cmtice@google.com> Commit-Queue: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com>
2013-09-10gdb_dejagnu: Use TryAcquireMachine from run_dejagnu.Yunlian Jiang
This fixes some gpylint warnings and uses TryAcquireMachine from run_dejagnu. BUG=None TEST=gdb_dejagnu passes. Change-Id: I1e7230b00619daec9cea28ca2926e18fb8ac8345 Reviewed-on: https://chrome-internal-review.googlesource.com/144426 Reviewed-by: Han Shen <shenhan@google.com> Commit-Queue: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com>
2013-09-10For the new git repositories, we need to use https instead of http and ssh.Luis Lozano
Bug=None Test=Tested setup_chromeos, build_chromes, remote_gcc_build Change-Id: I95833ed07692ea04f396233ab87aa7b47ff6b106 Reviewed-on: https://chrome-internal-review.googlesource.com/144397 Reviewed-by: Han Shen <shenhan@google.com> Commit-Queue: Luis Lozano <llozano@chromium.org> Tested-by: Luis Lozano <llozano@chromium.org>
2013-09-10update daisy gdb baselineYunlian Jiang
BUG=None TEST=None Change-Id: Icefcffcc40057954bf7b51c3921248c43ef2b810 Reviewed-on: https://chrome-internal-review.googlesource.com/144366 Reviewed-by: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com> Commit-Queue: Yunlian Jiang <yunlian@google.com>
2013-09-04Modified the script to use the new internal and external gerrit repositories.Luis Lozano
BUG=None TEST=Minimal testing done. Need to let the nightly testing test this. Change-Id: I8273d5e90e5fb3ed96d9782cae651d4144f135dc Reviewed-on: https://chrome-internal-review.googlesource.com/144015 Reviewed-by: Han Shen <shenhan@google.com> Tested-by: Luis Lozano <llozano@chromium.org> Commit-Queue: Luis Lozano <llozano@chromium.org>
2013-08-26add vanilla_image option to build_chromeos.pyYunlian Jiang
This adds vanilla_iamge option to build_chromeos.py. With this option, it runs build_packages with --usepkg. In the test_toolchains.py, it uses this option when building vanilla image. BUG=None TEST=None Change-Id: I54f85e60ccae833ab6682476e19365a51df5ef29 Reviewed-on: https://gerrit-int.chromium.org/43088 Reviewed-by: Luis Lozano <llozano@chromium.org> Commit-Queue: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com>
2013-08-22Make test_that available for running tests.Caroline Tice
Add a new flag, --use_test_that=True/False, to crosperf, to allow users to tell crosperf to use the test_that script rather than the run_remote_tests.sh script for running their tests. BUG=None TEST=Tested with and without the flag, to verify that it does the right thing. Change-Id: I1a040d23adbf364b4a47320d9fd759a2d46e6689 Reviewed-on: https://gerrit-int.chromium.org/43109 Reviewed-by: Luis Lozano <llozano@chromium.org> Reviewed-by: Yunlian Jiang <yunlian@google.com> Commit-Queue: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com>
2013-08-22toolchian_utils: add --accept_licenses=@CHROMEOS flag.Yunlian Jiang
This adds --accept_licenses=@CHROMEOS to setup_board and build_packages. With this patch, we can build daisy from role account. BUG=None TEST=role account can build daisy now. Change-Id: Ib08bf44d3355f667154f78f07e67d2aeb6d97a31 Reviewed-on: https://gerrit-int.chromium.org/43330 Reviewed-by: Caroline Tice <cmtice@google.com> Reviewed-by: Luis Lozano <llozano@chromium.org> Commit-Queue: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com>
2013-08-19Added new daisy machines in the lab.Luis Lozano
BUG=None TEST=None Change-Id: Id88cee787d55dff8f7fb60e1de68347a26a8d56f Reviewed-on: https://gerrit-int.chromium.org/43076 Reviewed-by: Caroline Tice <cmtice@google.com> Commit-Queue: Luis Lozano <llozano@chromium.org> Tested-by: Luis Lozano <llozano@chromium.org>
2013-08-19Add the omnetpp example.Yuheng Long
BUG=None TEST=unit testings for the pipeline stage, pipeline workers, generation, steering, task, flag, hill climbing and genetic algorithm. Change-Id: I69f10c79e87db1ce58c14a4c827ea5f5dd71e4bf Reviewed-on: https://gerrit-int.chromium.org/43047 Commit-Queue: Luis Lozano <llozano@chromium.org> Reviewed-by: Luis Lozano <llozano@chromium.org> Tested-by: Luis Lozano <llozano@chromium.org>
2013-08-19Fixed typo in last commitLuis Lozano
BUG=None TESTS=Did several succesful remote builds Change-Id: Icff4f20ec44c962f52cd4cec85c3e2ab70eeb4a8 Reviewed-on: https://gerrit-int.chromium.org/43077 Reviewed-by: Yunlian Jiang <yunlian@google.com> Commit-Queue: Luis Lozano <llozano@chromium.org> Tested-by: Luis Lozano <llozano@chromium.org>
2013-08-16Add the negative flag elimination algorithm.Yuheng Long
BUG=None TEST=unit testings for the pipeline stage, pipeline workers, generation, steering, task, flag, hill climbing, genetic algorithm and iterative flag elimination algorithm. Change-Id: I2d400e304570de8a15556529eff823d40c36b185 Reviewed-on: https://gerrit-int.chromium.org/42798 Reviewed-by: Luis Lozano <llozano@chromium.org> Commit-Queue: Yuheng Long <yuhenglong@google.com> Tested-by: Yuheng Long <yuhenglong@google.com>
2013-08-16Add an example that flows through the framework.Yuheng Long
BUG=None TEST=unit testings for the pipeline stage, pipeline workers, generation, steering, task, flag, hill climbing and genetic algorithm. Change-Id: I997770204feda543a3424f7f69cf2ee7f45fccc2 Reviewed-on: https://gerrit-int.chromium.org/43004 Reviewed-by: Luis Lozano <llozano@chromium.org> Commit-Queue: Yuheng Long <yuhenglong@google.com> Tested-by: Yuheng Long <yuhenglong@google.com>
2013-08-16Fixed the problems in task and steering.Yuheng Long
BUG=None TEST=unit testings for the pipeline stage, pipeline workers, generation, steering, task, flag, hill climbing and genetic algorithm. Change-Id: Ib34ef05518124b6ffa03f0c35795109e22581b89 Reviewed-on: https://gerrit-int.chromium.org/43002 Reviewed-by: Luis Lozano <llozano@chromium.org> Commit-Queue: Yuheng Long <yuhenglong@google.com> Tested-by: Yuheng Long <yuhenglong@google.com>
2013-08-15Initiate all the class fields in the init method.Yuheng Long
BUG=None TEST=unit testings for the pipeline stage, pipeline workers, generation, steering, task, flag and hill climbing. Change-Id: I170f4c3a719e0b6aedfdbd90aa29803c011c9ba0 Reviewed-on: https://gerrit-int.chromium.org/42835 Reviewed-by: Simon Que <sque@google.com> Reviewed-by: Luis Lozano <llozano@chromium.org> Tested-by: Yuheng Long <yuhenglong@google.com> Commit-Queue: Yuheng Long <yuhenglong@google.com>
2013-08-15Add the content to the README.Yuheng Long
BUG=None TEST=unit testings for the pipeline stage, pipeline workers, generation, steering, task, flag and hill climbing. Change-Id: I49ac28745fc8bdf1a04f26fb468c420e8d2c7b9c Reviewed-on: https://gerrit-int.chromium.org/42833 Reviewed-by: Simon Que <sque@google.com> Reviewed-by: Luis Lozano <llozano@chromium.org> Tested-by: Yuheng Long <yuhenglong@google.com> Commit-Queue: Yuheng Long <yuhenglong@google.com>
2013-08-15Refining the documentations.Yuheng Long
BUG=None TEST=unit testings for the pipeline stage, pipeline workers, generation, steering, task, flag, hill climbing and genetic algorithm. Change-Id: I45c83c9dc74e54b75492ceb281e6912eb347d75e Reviewed-on: https://gerrit-int.chromium.org/42802 Reviewed-by: Simon Que <sque@google.com> Reviewed-by: Luis Lozano <llozano@chromium.org> Tested-by: Yuheng Long <yuhenglong@google.com> Commit-Queue: Yuheng Long <yuhenglong@google.com>
2013-08-15Fix results processing to handle "JSON.stringify" in Telemetry results.Caroline Tice
This is the same as the previous CL; I had to abandon the previous CL because it erroneously depended on a different abandoned CL, so it refused to commit. BUG=none TEST=I ran the telemetry tests with the modification and now ALL of them give good output (before only about half of them did). Change-Id: I567f51176c522bc1c41676f6dc560f3827b9cef4 Reviewed-on: https://gerrit-int.chromium.org/42930 Reviewed-by: Bhaskar Janakiraman <bjanakiraman@google.com> Commit-Queue: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com>
2013-08-11Refine the module hill_climb_best_neighbor.Yuheng Long
BUG=None TEST=unit testings for the pipeline stage, pipeline workers, generation, steering, task, flag and hill climbing. Change-Id: I95fd82072f1474b73735a643f2da589eb930b838 Reviewed-on: https://gerrit-int.chromium.org/42663 Reviewed-by: Luis Lozano <llozano@chromium.org> Tested-by: Yuheng Long <yuhenglong@google.com> Commit-Queue: Yuheng Long <yuhenglong@google.com>
2013-08-10Add more examples for better documentation.Yuheng Long
BUG=None TEST=unit testings for the pipeline stage, pipeline workers, generation, steering, task, flag and hill climbing. Change-Id: I1f8f361388d3c3171a7135d1d0ef38ee5f695829 Reviewed-on: https://gerrit-int.chromium.org/42584 Reviewed-by: Luis Lozano <llozano@chromium.org> Commit-Queue: Yuheng Long <yuhenglong@google.com> Tested-by: Yuheng Long <yuhenglong@google.com>
2013-08-09Added new parrot machines in the labLuis Lozano
Change-Id: I93fdcf4ac064ef1a89289f88ee7d6b6360b28103 Reviewed-on: https://gerrit-int.chromium.org/42622 Reviewed-by: Caroline Tice <cmtice@google.com> Commit-Queue: Luis Lozano <llozano@chromium.org> Tested-by: Luis Lozano <llozano@chromium.org>
2013-08-07Add the Genetic Algorithm.Yuheng Long
BUG=None TEST=unit testings for the pipeline stage, pipeline workers, generation, steering, task, flag, hill climbing and genetic algorithm. Change-Id: I2864d6a6859fff43bc2d3afb059c672c54bbe385 Reviewed-on: https://gerrit-int.chromium.org/42472 Reviewed-by: Simon Que <sque@google.com> Commit-Queue: Yuheng Long <yuhenglong@google.com> Tested-by: Yuheng Long <yuhenglong@google.com>
2013-08-07Initialize machine_id_checksum. Make unlocking unlocked machine a warning.Caroline Tice
Fix issues causing errors in nightly tester: Attempting to use machine_id_checksum when it hasn't been initialized; and attempting to unlock an already unlocked machine causes a fatal error. Change-Id: Ia149953d4e25c7abf97a189bb5fdd129de865c34 Reviewed-on: https://gerrit-int.chromium.org/42489 Reviewed-by: Yunlian Jiang <yunlian@google.com> Commit-Queue: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com>
2013-08-07Add underscore to separate words in variables.Yuheng Long
BUG=None TEST=unit testings for the pipeline stage, pipeline workers, generation, steering, task, flag and hill climbing. Change-Id: I0efdf5e7ad85b42f37519b6e0cebea8352bbf563 Reviewed-on: https://gerrit-int.chromium.org/42430 Reviewed-by: Simon Que <sque@google.com> Tested-by: Yuheng Long <yuhenglong@google.com> Commit-Queue: Yuheng Long <yuhenglong@google.com>
2013-08-07Encapsulate a couple inner fields in a unittest.Yuheng Long
BUG=None TEST=unit testings for the pipeline stage, pipeline workers, generation, steering, task, flag and hill climbing. Change-Id: I5d86c107cb7a0693c97a0cb404c2a89a172ce835 Reviewed-on: https://gerrit-int.chromium.org/42418 Reviewed-by: Simon Que <sque@google.com> Tested-by: Yuheng Long <yuhenglong@google.com> Commit-Queue: Yuheng Long <yuhenglong@google.com>
2013-08-07Add the best branching hill climbing algorithm.Yuheng Long
BUG=None TEST=unit testings for the pipeline stage, pipeline workers, generation, steering, task, flag and hill climbing. Change-Id: Ifd181f45c4b82f5fb77b0d4946757954aa806d33 Reviewed-on: https://gerrit-int.chromium.org/42284 Tested-by: Yuheng Long <yuhenglong@google.com> Reviewed-by: Simon Que <sque@google.com> Commit-Queue: Yuheng Long <yuhenglong@google.com>
2013-08-06Fix syntax error in suite_runner.py, to allow crosperf to run "pure"Caroline Tice
Telemetry tests. Change-Id: Ie6073c05fdc50cf22294816b6be37811652b6827 Reviewed-on: https://gerrit-int.chromium.org/42444 Reviewed-by: Yunlian Jiang <yunlian@google.com> Commit-Queue: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com>