Age | Commit message (Collapse) | Author |
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
`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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|