aboutsummaryrefslogtreecommitdiff
path: root/cros_utils
AgeCommit message (Collapse)Author
2016-10-27[crosperf] Fix buildbot_utils for rotating tester waterfall.Caroline Tice
The rotating testers trybot waterfall location has moved. This CL updates the script to look for their build logs in the correct builder location. It also makes sure we don't crash/die because their build status is non-zero (ie. a HWTest failed). This also fixes a tiny bug in the results reporting. BUG=chromium:660085 TEST=Tested in the role account. Change-Id: I396ec0ad588e54e88bc44dcc0e72eda45d077173 Reviewed-on: https://chrome-internal-review.googlesource.com/300598 Commit-Ready: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com> Reviewed-by: Yunlian Jiang <yunlian@google.com>
2016-10-18[toolchain-utils] Fix rotating builders to work with no patches.Caroline Tice
Currently if no patch is passed to cbuildbot, it hangs; similarly if no patch is given to the rotating builder script, it dies. This CL fixes both of those. It also temporarily stops the rotating builders from sending emails until we can figure out how to fix their reports. BUG=chromium:657060 TEST=Tested changes in role account;seemed to work. Change-Id: I4ba70b562f9cb44d396361cd6461230d7080c6e9 Reviewed-on: https://chrome-internal-review.googlesource.com/297584 Commit-Ready: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com> Reviewed-by: Caroline Tice <cmtice@google.com> Reviewed-by: Ting-Yuan Huang <laszio@google.com>
2016-10-06[toolchain-utils] Update rotating testers to use new configs.Caroline Tice
This CL updates the rotating testers to use the new chromes_configs for building trybots & running HW tests. BUG=chromium:653296 TEST=Ran scripts in my directory; it worked. Change-Id: Ia60af8b921a966c86b335ef51db5372bf740ed95 Reviewed-on: https://chrome-internal-review.googlesource.com/293775 Commit-Ready: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com> Reviewed-by: Yunlian Jiang <yunlian@google.com>
2016-09-22cros-utils: fix an out of bound error.Yunlian Jiang
The input rgb2short can reach to 255. When the value exceeds 250, it causes out of bound error. This CL fixes that. BUG=chromium:649374 TEST=The out of bound error is fixed. Change-Id: Iee31337e82907e1803a84b5837542733bb224e8f Reviewed-on: https://chrome-internal-review.googlesource.com/289536 Commit-Ready: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com> Reviewed-by: Caroline Tice <cmtice@google.com>
2016-09-21toolchain-utils: remove colortrans.pyYunlian Jiang
This removes colortrans.py from cros_utils. We add a single function inside misc.py instead. BUG=chromium:647308 TEST=crosperf still works, the color text is shown on terminal. Change-Id: I1871bd2c57608f59ded6a8cf720c52cb2c07c3ff Reviewed-on: https://chrome-internal-review.googlesource.com/287538 Commit-Ready: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com> Reviewed-by: Luis Lozano <llozano@chromium.org>
2016-09-13crosperf+cros_utils: Swap to #!/usr/bin/env.George Burgess IV
`cros lint` was recently updated to complain about using #!/usr/bin/python{,2,3} directly. Instead, it prefers /usr/bin/env python{2,3} now. BUG=None TEST=./run_tests.sh passes; linter seems much happier with crosperf. Change-Id: I9b88a7af1f8e03b7c870781c3d8dc1b872a3dfc3 Reviewed-on: https://chrome-internal-review.googlesource.com/286341 Commit-Ready: Luis Lozano <llozano@chromium.org> Tested-by: Luis Lozano <llozano@chromium.org> Reviewed-by: Luis Lozano <llozano@chromium.org>
2016-09-02cros_utils: increast timeout for trybot.Yunlian Jiang
This increase time out for trybot jobs to 8 hours. BUG=None TEST=None Change-Id: Ieed04fa7082e4d71ee1ed9cee62e6eb33e6763a9 Reviewed-on: https://chrome-internal-review.googlesource.com/283936 Commit-Ready: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com> Reviewed-by: Ting-Yuan Huang <laszio@google.com>
2016-08-24crosperf: fix telemetry results parsing.Yunlian Jiang
New telemetry results provide multiple values for a single key. This CL uses the average value to make crosperf work. BUG=chromium:639896 TEST=run crosperf with 2 iterations of speedometer. Change-Id: I32e7a00a57dbbe995915af0c32918f1d680ae484 Reviewed-on: https://chrome-internal-review.googlesource.com/279057 Commit-Ready: Yunlian Jiang <yunlian@google.com> Tested-by: Yunlian Jiang <yunlian@google.com> Reviewed-by: Caroline Tice <cmtice@google.com>
2016-08-17[crosperf] Fix telemetry results parsing.Caroline Tice
Many telemetry results fields now contain multiple values. This CL fixes crosperf to accomodate that. BUG=None TEST=Ran crosperf with change and it worked. Change-Id: Iccb8926223c8447b72d51f413756f5dbc09f89de Reviewed-on: https://chrome-internal-review.googlesource.com/276817 Commit-Ready: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com> Reviewed-by: Luis Lozano <llozano@chromium.org>
2016-07-27[crosperf] Clean up exceptions.Caroline Tice
Found all instances of 'raise Exception' and changed them to raise something slightly more specific. Changed a few 'except Exception' statements appropriately as well. BUG=chromium:566255 TEST=None Change-Id: If2666bd55838342bc71431e5e6ab2157c2c9e125 Reviewed-on: https://chrome-internal-review.googlesource.com/270731 Commit-Ready: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com> Reviewed-by: Yunlian Jiang <yunlian@google.com>
2016-07-25[crosperf] Change flags to cbuildbot.Caroline Tice
The command for cbuildbot it buildbot_utils had hardcoded the '--notests' options; this was right for the normal nightly jobs, but not for the rotating tests. This CL makes that option a variable, and also allows for users to be able to add other custom flags to their cbuildbot invocations. BUG=chromium:631149 TEST=none Change-Id: Ifd2abd42808d99a28c3f7860f67a2eb2c9cd0118 Reviewed-on: https://chrome-internal-review.googlesource.com/271005 Commit-Ready: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com> Reviewed-by: Yunlian Jiang <yunlian@google.com>
2016-07-25[toolchain-utils] Finish switching utils/ to cros_utils/.Caroline Tice
This CL finishes switching the subdirectory from 'utils' to 'cros_utils'. It changes all the remaining import statements to use 'cros_utils'; it removes the 'cros_utils' symlink, and it renames the 'utils' subdirectory to 'cros_utils'. BUG=chromium:568195 TEST=ran crosperf & binary search tool unittests. Change-Id: I7427f8bfb2ddac3a4b6108e46782039059684382 Reviewed-on: https://chrome-internal-review.googlesource.com/270396 Commit-Ready: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com> Reviewed-by: Cassidy Burden <cburden@google.com> Reviewed-by: Luis Lozano <llozano@chromium.org>
2015-12-09crosperf: use cros_utils instead of utils.Yunlian Jiang
We got some lint warnings about some module is not in utils. So we change the name. This CL creates a symbolic link cros_utils from utils. Once we are sure that we clean all the code, we can rename the utils to cros_utils directly. BUG=chromium:567921 chromium:568195 TEST=the lint warning is gone. Change-Id: I776e45c7bdd75d1f065ea5797f3bcff457203ca2 Reviewed-on: https://chrome-internal-review.googlesource.com/241457 Commit-Ready: Caroline Tice <cmtice@google.com> Tested-by: Caroline Tice <cmtice@google.com> Reviewed-by: Caroline Tice <cmtice@google.com>