aboutsummaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2018-09-18Add OWNERS.android to toolchain-utils.android-o-mr1-iot-release-smart-display-r3android-o-mr1-iot-release-1.0.5oreo-mr1-1.2-iot-releasemaster-cuttlefish-testing-releaseStephen Hines
Change-Id: I3275f98b34eb4c155997b153854ae7eb4d6fea6d
2018-09-18Merge remote-tracking branch 'aosp/mirror-chromium-master' into update_utilsStephen Hines
Change-Id: Ia627b264f23935d8ae852e259c90c46377783abd
2018-09-17bisect tool: Symbolic link re-generated bad item to original oneZhizhou Yang
This patch modifies the way to genearte bad item while pass level bisection. Rather than directly overwrite the old object, we generate it to a tmp file and symblic link it to original one to satisfy building tool. BUG=chromium:878954 TEST=Ran test successfully with Android compiler wrapper. Change-Id: If152742e0a2088e9d8c2456bf046cede938d88d3 Reviewed-on: https://chromium-review.googlesource.com/1220016 Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com> Tested-by: Zhizhou Yang <zhizhouy@google.com> Reviewed-by: Caroline Tice <cmtice@chromium.org>
2018-09-16Remove lumpy and parrot which eliminated from lab.Luis A. Lozano
BUG=none TEST=none Change-Id: I5c84f765e7a958e5d7055f850d7da8fa7203b384 Reviewed-on: https://chromium-review.googlesource.com/1227060 Commit-Ready: Luis Lozano <llozano@chromium.org> Tested-by: Luis Lozano <llozano@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
2018-09-10Bisect tool: Pass/Transformation level bisection for bad itemZhizhou Yang
This patch provides pass and transformation level bisection support for existing bisection tool. Usage: When `--pass_bisect=script_path` is set, pass/transformation level bisection will be turned on. It only works when prune is set to False. Orignial bisection will end with the first bad item it find. This patch then will rebuild and bisect the bad item using bad compiler with pass/transformation level bisect limit. It will end up with the bad pass name/index and bad transformation index for that pass. Notice that this patch still need support from LLVM with this review: https://reviews.llvm.org/D50031, which allows debugcounter to print info we need. The patch eventally add a new binary search process, and two build function to saparately build with pass and transformation level limit. A new BinarySearcherForPass is also introduced for pass level bisect. The workflow is: 1) get bad item and command to build it with existing bisection. 2) rebuild bad item with pass level limit to -1 to get total pass count. 3) binary search pass index until we got bad pass. 4) rebuild bad item with pass limit we get and also set transformation level limit to -1 to get total number of transforamtion. 5) binary search transformation index until we got bad transformation. BUG=chromium:878954 TEST=Ran test successfully with Android compiler wrapper. Will add unittests once new patch in LLVM applied. Change-Id: I08f376f15d12eea232da5887981c44184ffb9568 Reviewed-on: https://chromium-review.googlesource.com/1208855 Commit-Ready: Zhizhou Yang <zhizhouy@google.com> Tested-by: Zhizhou Yang <zhizhouy@google.com> Reviewed-by: Caroline Tice <cmtice@chromium.org>
2018-09-04Bisect tool: Add pass level bisection command option helperZhizhou Yang
This patch introduces a new option for bisection tool which is `--pass_bisect`. When it is set to android/generate_cmd.sh, bisection tool will create a new script which extracts command line options to build the bad item (We now only support pass level bisecting for single bad item, so prune need to be turned off with this option on). Another python wrapper script will come soon and it will take the generated script as one argument, indicating how the compiler will build the bad item at certain limit. BUG=chromium:878954 TEST=Ran test successfully with Android compiler wrapper. Will add unittests once verified ChromeOS. Change-Id: Iaa972e9e1a364143b59a89e6518bbb2bba6e3522 Reviewed-on: https://chromium-review.googlesource.com/1195595 Commit-Ready: Zhizhou Yang <zhizhouy@google.com> Tested-by: Zhizhou Yang <zhizhouy@google.com> Reviewed-by: Caroline Tice <cmtice@chromium.org>
2018-08-21Revert "USE local lock for buildbot_test_toolchain."Ting-Yuan Huang
This reverts commit 4260aae2fe0abb937543c6cafc44f68e99de7410. Reason for revert: afe_lock_machine is fixed. Original change's description: > USE local lock for buildbot_test_toolchain. > > Currently the AFE lock machanism for our nightly test is broken, > we use local lock as a workaround for now. > > BUG=chromium:762589 > TEST=Generate the crosperf command in another simple python file > with the same code. Run the generated command line on > crotomation2 and it goes though the locking machine stage. > > Change-Id: Icd3132bc383b63aab6d6f5237a04348f11d8726d > Reviewed-on: https://chromium-review.googlesource.com/653213 > Commit-Ready: Yunlian Jiang <yunlian@chromium.org> > Tested-by: Yunlian Jiang <yunlian@chromium.org> > Reviewed-by: Caroline Tice <cmtice@chromium.org> Bug: chromium:762589 Change-Id: Ib9f787ff48953d384dd36c72811ebd0f20dd25db Reviewed-on: https://chromium-review.googlesource.com/1178897 Tested-by: Ting-Yuan Huang <laszio@chromium.org> Reviewed-by: Luis Lozano <llozano@chromium.org> Reviewed-by: Caroline Tice <cmtice@chromium.org> Commit-Queue: Ting-Yuan Huang <laszio@chromium.org>
2018-08-21crosperf: remove deprecated testsKuo-Hsin Yang
Remove the json grammar to parse the result for deprecated tests dromaeo.*, sunspider and and robohornet_pro. These tests are already removed from telemetry tests. BUG=chromium:867332 TEST=none Change-Id: I06e6e99b51181e30410000223aca1114641cace2 Reviewed-on: https://chromium-review.googlesource.com/1182715 Commit-Ready: Vovo Yang <vovoy@chromium.org> Tested-by: Vovo Yang <vovoy@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
2018-08-09pyrun: fix quotingMike Frysinger
BUG=None TEST=None Change-Id: Ieed7e7df9bed83dc6c54d687ae3f3ae657e22693 Reviewed-on: https://chromium-review.googlesource.com/1168457 Commit-Ready: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org> Reviewed-by: Luis Lozano <llozano@chromium.org>
2018-08-08Increase timeout for slower builders.Luis A. Lozano
Some of our slower builder (with latest_toolchain) are taking around 8 hours. This CL increases the timeout for such situations. BUG=chromium:870916 TESTS=None Change-Id: I8c15e4ecbd71f76bca65c6adcabd74df8da79660 Reviewed-on: https://chromium-review.googlesource.com/1167965 Commit-Ready: Luis Lozano <llozano@chromium.org> Tested-by: Luis Lozano <llozano@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
2018-08-07import toolchain_utils.shMike Frysinger
This is the only repo that uses these funcs anymore, so move them here so we can delete the source file entirely. BUG=None TEST=precq passes Change-Id: Ie3cfa2168783999f3a02df20957755ae4c6c46af Reviewed-on: https://chromium-review.googlesource.com/1163163 Commit-Ready: Mike Frysinger <vapier@chromium.org> Tested-by: Mike Frysinger <vapier@chromium.org> Reviewed-by: Caroline Tice <cmtice@chromium.org>
2018-08-02toolchain-utils: Add a tool to create llvm-extra ebuild.Manoj Gupta
llvm-extra ebuild is a slotted llvm compiler ebuild that can be installed in parallel to the regualr llvm compiler. Use of llvm-extra package is to provide an alternate clang in case the normal clang compiler binary can't be used. e.g. After a llvm upgrade, some package foo is miscompiled. To avoid the miscompilation, we can use llvm-extra ebuild to install the previous clang. To use the clang installed by llvm-extra, modify the CFLAGS and LDFLAGS to pass the patch of the clang binary installed by llvm-extra package. e.g. append-flags -Xclang-path=/usr/llvm-extra/version/clang append-ldflags -Xclang-path=/usr/llvm-extra/version/clang This CL only adds the tool that can generate the llvm-extra ebuilds from a given llvm ebuild. BUG=chromium:773887 TEST= ./create_llvm_extra.sh /path/to/llvm/llvm-7.0_pre331547_p20180529-r8.ebuild worked. A new llvm-extra ebuild was generated. Change-Id: I7e3e20a2740f5e7135dbcd252cc271945c6ddd34 Reviewed-on: https://chromium-review.googlesource.com/1161213 Commit-Ready: Manoj Gupta <manojgupta@chromium.org> Tested-by: Manoj Gupta <manojgupta@chromium.org> Reviewed-by: Caroline Tice <cmtice@chromium.org>
2018-07-06crosperf: remove smoothness.tough_webgl_casesTing-Yuan Huang
It is no longer available. TEST=none BUG=none Change-Id: I25437887ec2bd608ba90430a4753e4010ab8a2f5 Reviewed-on: https://chromium-review.googlesource.com/1128364 Commit-Ready: Ting-Yuan Huang <laszio@chromium.org> Tested-by: Ting-Yuan Huang <laszio@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org> Reviewed-by: Yunlian Jiang <yunlian@chromium.org>
2018-07-03toolchain-utils: update scripts for building and testing the Go toolchain.Rahul Chaudhry
BUG=None TEST=None Change-Id: If19fcb65915334c40970cbf456128dbc6ac7bcec Reviewed-on: https://chromium-review.googlesource.com/1125344 Commit-Ready: Rahul Chaudhry <rahulchaudhry@chromium.org> Tested-by: Rahul Chaudhry <rahulchaudhry@chromium.org> Reviewed-by: Ting-Yuan Huang <laszio@chromium.org>
2018-06-07Use internal chrome source tree for telemetry.Yunlian Jiang
Now sync_chrome works for mobiletc-prebuild, there is no need to have this workaround. BUG=none TEST=ebuild-caroline $(equery-caroline w chromeos-chrome) prepare works on mobiletc-prebuild account Change-Id: I36ca9967b8b8fe5f79ca3b4dedc69271484d633f Reviewed-on: https://chromium-review.googlesource.com/1089114 Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com> Tested-by: Yunlian Jiang <yunlian@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
2018-06-05update _estimated_stddev for speedometer2Yunlian Jiang
This updates _estimated_stddev for speedometer2. The value is collected by running speedometer2 on kevin and samus to get the maximum stddev/mean. BUG=chromium:849321 TEST=None Change-Id: If5efa962451a4f221eef4c33a408e88c7d3fee43 Reviewed-on: https://chromium-review.googlesource.com/1087347 Commit-Ready: Yunlian Jiang <yunlian@chromium.org> Tested-by: Yunlian Jiang <yunlian@chromium.org> Reviewed-by: Caroline Tice <cmtice@chromium.org>
2018-06-05add speedometer2 result formatYunlian Jiang
This adds the speedometer2 result format to default-telemetry-results.json. BUG=chromium:849321 TEST=run crosperf and it showed the result for speedometer2. Change-Id: I8b824b6c1739a032bb15a706cee96d1f45c444ab Reviewed-on: https://chromium-review.googlesource.com/1086292 Commit-Ready: Yunlian Jiang <yunlian@chromium.org> Tested-by: Yunlian Jiang <yunlian@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
2018-06-02crosperf: add speedometer2 to toolchain perf tests.Rahul Chaudhry
BUG=None TEST=None Change-Id: I73a8893e892b2941ba1170b1606e4a17744265c5 Reviewed-on: https://chromium-review.googlesource.com/1082954 Commit-Ready: Rahul Chaudhry <rahulchaudhry@chromium.org> Tested-by: Rahul Chaudhry <rahulchaudhry@chromium.org> Reviewed-by: Luis Lozano <llozano@chromium.org>
2018-05-25toolchain_utils: Use external chrome source tree for telemetry.Caroline Tice
When generating the nightly performance tests, specify an external Chrome source tree, since Chrome sync'ing is broken inside Chrome OS. Also remove unnecessary code that references a file that no longer exists. BUG=chromium:846508 TEST=Successfully ran nightly tests with these changes on Chrotomation2. Change-Id: I1cba94f82ad6ffb39c941a5dc3cb8f95cb833d20 Reviewed-on: https://chromium-review.googlesource.com/1072870 Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com> Tested-by: Caroline Tice <cmtice@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
2018-05-16toolchain-utils: Delete temporary chroot image after verification step.Gabriel Marin
When a local image is outside the chroot, it is temporarily copied into the chroot so that it can be mounted by chroot scripts. Changed the logic to delay the cleanup of the temporary copy until after the verification step. The image needs to be mounted once again during verification. Didn't catch this bug earlier, as the images were cached inside the chroot by previous failed runs. BUG=b:79376118 TEST=Tested with crosperf, two local images and a clean chroot without any cached images. Change-Id: I700f9aa0d9827aa349e75ef7b6b22618baf26fa9 Reviewed-on: https://chromium-review.googlesource.com/1062330 Commit-Ready: Gabriel Marin <gmx@chromium.org> Tested-by: Gabriel Marin <gmx@chromium.org> Reviewed-by: Gabriel Marin <gmx@chromium.org> Reviewed-by: Yunlian Jiang <yunlian@chromium.org>
2018-05-16toolchain-utils: Use the chromeos_image value computed initially.Gabriel Marin
When reimaging with a local image, use the chromeos_image value computed during the initial analysis, instead of attempting to recompute it at each use. The recomputation at the verification stage was also incorrect. BUG=b:79376118 TEST=Tested with crosperf and two local images. The script correctly switches between images and it doesn't fail during the reimaging step. Change-Id: Ie40ed52999ecbebb4e2f5dd3e0aa78a18482533a Reviewed-on: https://chromium-review.googlesource.com/1058181 Commit-Ready: Gabriel Marin <gmx@chromium.org> Tested-by: Gabriel Marin <gmx@chromium.org> Reviewed-by: Caroline Tice <cmtice@chromium.org>
2018-05-12debug_info_test: check that chrome was built with ICF enabledRahul Chaudhry
For both amd64 and arm, when ICF (Identical Code Folding) is enabled, there is >60,000 drop in the number of unique addresses of text symbols. We check here that the drop is at least 10,000. See https://crbug.com/813272#c18 for details. BUG=chromium:813272 TEST='./debug_info_test.py /build/chell/usr/lib/debug/opt/google/chrome/chrome.debug' TEST='./debug_info_test.py /build/elm/usr/lib/debug/opt/google/chrome/chrome.debug' Change-Id: I23d1b48e6547bf3042096bed1c566ec50c5b2d66 Reviewed-on: https://chromium-review.googlesource.com/1053388 Commit-Ready: Rahul Chaudhry <rahulchaudhry@chromium.org> Tested-by: Rahul Chaudhry <rahulchaudhry@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org> Reviewed-by: Yunlian Jiang <yunlian@chromium.org>
2018-05-06[toolchain-utils] Whitelist update for debug_info_testZhizhou Yang
This CL updates the whitelist for debug_info_test stage, when checking whether DebugInfo session exists or not. We do not care about the debug info in autotest, so whitelisted this directory. BUG=chromium:813271 TEST=debug_info_test /build/kip/usr/lib/debug with -nousepkg build Change-Id: I1ccd845462d07b0be58eb811550a00a2986a5086 Reviewed-on: https://chromium-review.googlesource.com/1044756 Commit-Ready: Zhizhou Yang <zhizhouy@chromium.org> Tested-by: Zhizhou Yang <zhizhouy@chromium.org> Reviewed-by: Yunlian Jiang <yunlian@chromium.org>
2018-05-04crosperf: Disable turbo only if enabled.Manoj Gupta
Write to no_turbo file only if the field is set to 0. BUG=chromium:839125 TEST=No more errors Change-Id: If5de67606557c21107843837afd035cf32545475 Reviewed-on: https://chromium-review.googlesource.com/1042866 Commit-Ready: Manoj Gupta <manojgupta@chromium.org> Tested-by: Manoj Gupta <manojgupta@chromium.org> Reviewed-by: Caroline Tice <cmtice@chromium.org>
2018-05-03crosperf: Ignore error when disabling turbo.Manoj Gupta
Some boards do not permit changing tubo config /sys/devices/system/cpu/intel_pstate/no_turbo Ignore the error for now so that tests can continue. Also update some formatting suggested by tc_format. BUG=chromium:839125 TEST=No more errors. Change-Id: I3a709ea2638d783dd3a3ebf70eba8a6dbc4df53a Reviewed-on: https://chromium-review.googlesource.com/1040773 Commit-Queue: Manoj Gupta <manojgupta@chromium.org> Tested-by: Manoj Gupta <manojgupta@chromium.org> Trybot-Ready: Manoj Gupta <manojgupta@chromium.org> Reviewed-by: Luis Lozano <llozano@chromium.org>
2018-05-02toolchain-utils: Update list of toolchain boards.Manoj Gupta
Add the new boards to the list of toolchain boards so that their status can be queried. BUG=chromium:837973 TEST=None Change-Id: I9565b2d38d3e2c95f1c6b6564b148315e9d182f2 Reviewed-on: https://chromium-review.googlesource.com/1038438 Tested-by: Manoj Gupta <manojgupta@chromium.org> Reviewed-by: Caroline Tice <cmtice@chromium.org>
2018-04-27debug_info_test: add a test to check DW_AT_producerZhizhou Yang
This patch introduces two tests, one to check whether debug info section exists in the elf file, the other to check whether DW_AT_producer exists in each compile unit of debug info. BUG=chromium:813271 TEST=debug_info_test /build/kip/usr/lib/debug Change-Id: I3c207abe05c55182ddcc1910073c15a40b0844bf Reviewed-on: https://chromium-review.googlesource.com/1007968 Commit-Ready: Zhizhou Yang <zhizhouy@chromium.org> Tested-by: Zhizhou Yang <zhizhouy@chromium.org> Reviewed-by: Ting-Yuan Huang <laszio@chromium.org>
2018-04-20[toolchain-utils] Update waterfall reports.Caroline Tice
This CL updates the waterfall reports to use the results of 'cros buildresult' and the new URLS. It removes the attempts to track individual test failures, as the new URL system no longer allows that. BUG=chromium:829648 TEST=launched reports from my directory. They worked fine. Tested buildbot_test_llvm.py changes with mobiletc-prebuild on chrotomation2. Change-Id: I12cf97c7960b032ec3caca636fe349f564369378 Reviewed-on: https://chromium-review.googlesource.com/1021990 Commit-Ready: Caroline Tice <cmtice@chromium.org> Tested-by: Caroline Tice <cmtice@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
2018-04-19toolchain-utils: Replace kahlee by grunt.Manoj Gupta
kahlee has been superseded by grunt. BUG=None TEST=None Change-Id: Iebdb0c4d75b1b4cf49605b3f71150245487894c0 Reviewed-on: https://chromium-review.googlesource.com/1017819 Commit-Ready: Manoj Gupta <manojgupta@chromium.org> Tested-by: Manoj Gupta <manojgupta@chromium.org> Reviewed-by: Yunlian Jiang <yunlian@chromium.org>
2018-04-18[toolchain-utils] Remove one last description reference.Caroline Tice
Previous CL, removing '--remote-description' flag, missed two uses which this CL deletes. BUG=chromium:833542 TEST=In progress. Change-Id: I37d06e12af815e063d2cd0bac71737356cdd15a5 Reviewed-on: https://chromium-review.googlesource.com/1017909 Commit-Ready: Caroline Tice <cmtice@chromium.org> Tested-by: Caroline Tice <cmtice@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
2018-04-16[toolchain-utils] Remove --remote-description flag.Caroline Tice
The cros tryjob command no longer recognizes this flag so we need to remove it from our scripts. BUG=chromium:833542 TEST=Tested with launching the rotating builders on chrotomation2. Change-Id: Ifcf62ae7073d84a9f58e12c29dcce5cdf9108e9f Reviewed-on: https://chromium-review.googlesource.com/1014369 Commit-Ready: Caroline Tice <cmtice@chromium.org> Tested-by: Caroline Tice <cmtice@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
2018-04-16[toolchain-utils] Add info to rotating builder logs.Caroline Tice
Due to recent changes in the tryjob launching system, the current rotating builder logs are empty. This CL causes the command and resulting buildbucket_id to be printed, and therefore go into the logs, so the sheriffs have some way of finding out what boards were built and what the buildbucket ids were for the builds. Also fix some format errors (forced by commit hook). BUG=chromium:833195 TEST=Tested for the last two nights on chrotomation2. Change-Id: Ic8e18baf1bc3f39af574d673962e43e830356210 Reviewed-on: https://chromium-review.googlesource.com/1013227 Commit-Ready: Caroline Tice <cmtice@chromium.org> Tested-by: Caroline Tice <cmtice@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
2018-04-11buildbot_utils: fix a typo.Yunlian Jiang
BUG=None TEST=None. Change-Id: Ib0ba6a3cf9bee432b93dc2b96f96e2c4c8a8944d Reviewed-on: https://chromium-review.googlesource.com/1007488 Commit-Ready: Yunlian Jiang <yunlian@chromium.org> Tested-by: Yunlian Jiang <yunlian@chromium.org> Reviewed-by: Yunlian Jiang <yunlian@chromium.org>
2018-04-10buildbot_utils: change method to get buildbucketidYunlian Jiang
The infrastructure team changed the output of cros tryjob, we need to change accordingly to get the buildbucketid from the output. BUG=none TEST=ParseTryjobBuildbucketId can get the buildbucketid from the new output. Change-Id: I389355cb4fae72b21859167e9e191cbcebfc64ae Reviewed-on: https://chromium-review.googlesource.com/1006187 Commit-Ready: Yunlian Jiang <yunlian@chromium.org> Tested-by: Yunlian Jiang <yunlian@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org> Reviewed-by: Caroline Tice <cmtice@chromium.org>
2018-04-10crosperf: filter result for jetstream and cros_ui_smoothnessYunlian Jiang
This CL removes most of data point for jetstream and cros_ui_smooothness benchmark from the final report of crosperf. It only keeps the summary numbers. BUG=none TEST=run crosperf, only summary data were shown for these two benchmarks. Change-Id: I4482e848700ee9555ad713a07095fe9db89f7747 Reviewed-on: https://chromium-review.googlesource.com/1003386 Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com> Tested-by: Yunlian Jiang <yunlian@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
2018-04-03crosperf: remove three benchmarks from test suiteYunlian Jiang
Three benchamrks were deleted in upstream, we need to remove them from our test suite. BUG=chromium:828463 TEST=uniitest passes. Change-Id: I5c92ca86217dc4bac79a75cba82dab4a793e0b3a Reviewed-on: https://chromium-review.googlesource.com/992879 Commit-Ready: Yunlian Jiang <yunlian@chromium.org> Tested-by: Yunlian Jiang <yunlian@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
2018-03-26buildbot_test_toolchains: update pattern to match artifact urlTing-Yuan Huang
The 'trybot-' prefix is removed in the artifact URL. See bug for more details. BUG=chromium:825997 TEST=Tested locally; The re pattern accepts the new URL. Change-Id: I46667c29c60aa6bde9710bbb0f699f4939a1a451 Reviewed-on: https://chromium-review.googlesource.com/981398 Commit-Ready: Ting-Yuan Huang <laszio@chromium.org> Tested-by: Ting-Yuan Huang <laszio@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org> Reviewed-by: Yunlian Jiang <yunlian@chromium.org>
2018-03-26[toolchain-utils] Fix DebugInfoTest stage for x30evb.Caroline Tice
Fix a new DebugInfoTest stage failure. BUG=824857 TEST=x30evb release tryjob passed the DebugInfoTest stage. Change-Id: I92b6e5513f7eb678efd1faff5a3c74e5e42a3187 Reviewed-on: https://chromium-review.googlesource.com/981074 Commit-Ready: Caroline Tice <cmtice@chromium.org> Tested-by: Caroline Tice <cmtice@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org> Reviewed-by: Yunlian Jiang <yunlian@chromium.org>
2018-03-23[toolchain-utils] Fix final DebugInfoTest corner cases.Caroline Tice
There are still a few release builders that are not passing the DebugInfoTest stage. This CL should fix all of those. BUG=chromium:824857 TEST=DebugInfoTest stage now passes, in the following tryjobs: capri-release, cobblepot-release, guado-macrophage-release, lakitu-nc-release, lakitu-st-release, lasilla-ground-release, lassen-release, macchiato-release. Change-Id: Ice7c051f67598df94b0d0ee6f6329bc139f9baef Reviewed-on: https://chromium-review.googlesource.com/976863 Commit-Ready: Caroline Tice <cmtice@chromium.org> Tested-by: Caroline Tice <cmtice@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
2018-03-18[toolchain-utils] More whitelist updates.Caroline Tice
We found a few more packages/files that need to be built with GCC and are adding them to our debug_info_test whitelists. BUG=chromium:813268 TEST=Tested with remote tryjobs: lakitu-release, gale-release, rainier-release, beaglebone-relase, fizz-release, cave-release. All passed the DebugInfoTest stage. Change-Id: I012d1620a85180f946c2ad2c8bbb175f5ef23e49 Reviewed-on: https://chromium-review.googlesource.com/967455 Commit-Ready: Caroline Tice <cmtice@chromium.org> Tested-by: Caroline Tice <cmtice@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
2018-03-15[toolchain-utils] Update whitelists for debug_info_test stage.Caroline Tice
The debug_info_test stage verifies that everything that is supposed to be built with LLVM is actually being built with LLVM. We have to whitelist a few files/packages that must continue to be built with GCC. This CL updates the whitelists to make this stage pass on AMD64. BUG=chromium:817648 TEST= debug_info_test.py passed on my local terra build, with these changes. DebugInfoTest stage passed on cros tryjob fizz-llvm-toolchain-tryjob. Change-Id: Ia9c79b5f13663d1f8a0e2b948f03a532b46d270b Reviewed-on: https://chromium-review.googlesource.com/964590 Commit-Ready: Caroline Tice <cmtice@chromium.org> Tested-by: Caroline Tice <cmtice@chromium.org> Reviewed-by: Ting-Yuan Huang <laszio@chromium.org>
2018-03-15toolchain testing: Switch to different boards with better availability.Manoj Gupta
Replace some testing boards with other boards in same platform that have better availability in pool:suites. BUG=chromium:815357 TEST=None Change-Id: Icf89f00462479b5b34695442995fe5f3e3776ebd Reviewed-on: https://chromium-review.googlesource.com/964702 Commit-Queue: Manoj Gupta <manojgupta@chromium.org> Tested-by: Manoj Gupta <manojgupta@chromium.org> Trybot-Ready: Manoj Gupta <manojgupta@chromium.org> Reviewed-by: Rahul Chaudhry <rahulchaudhry@chromium.org>
2018-03-13cros_utils/buildbot_utils: GetTrybotImage should return a partial URLTing-Yuan Huang
It should return a partial URL which is appended to xbuddy://remote/ in crosperf's exp file. BUG=chromium:807029 TEST=cros_utils/buildbot_utils_unittest.py buildbot_test_toolchains.py Change-Id: I1297b4ce586173d8ad97f6c30f989f826050e72b Reviewed-on: https://chromium-review.googlesource.com/959361 Commit-Ready: Ting-Yuan Huang <laszio@chromium.org> Tested-by: Ting-Yuan Huang <laszio@chromium.org> Reviewed-by: Caroline Tice <cmtice@chromium.org>
2018-03-12toolchain-utils: Make changes go through CQ.Manoj Gupta
dev-util/toolchain-utils host package depends on toolchain-utils repo. So add it to CQ but ignore any HW/VM test fails. BUG=None TEST=pre-cq passes. Change-Id: I797278ece9c460c022dd2fda1f02be7b5ed0df22 Reviewed-on: https://chromium-review.googlesource.com/959156 Commit-Ready: Manoj Gupta <manojgupta@chromium.org> Tested-by: Manoj Gupta <manojgupta@chromium.org> Reviewed-by: Ting-Yuan Huang <laszio@chromium.org>
2018-03-09cros_utils/buildbot_utils: leverage `cros buildresult`Ting-Yuan Huang
Originally it sends a http request to some magic URL and parse the contents. Now there is a `cros buildresult` available for use. BUG=chromium:807029 TEST=cros_utils/buildbot_utils_unittest.py buildbot_test_toolchains.py Change-Id: I33441859afd675c1a6b70d3d1f4e2348d54cbd77 Reviewed-on: https://chromium-review.googlesource.com/954052 Commit-Ready: Ting-Yuan Huang <laszio@chromium.org> Tested-by: Ting-Yuan Huang <laszio@chromium.org> Reviewed-by: Caroline Tice <cmtice@chromium.org>
2018-03-08[crosperf] Fix mount and image-finding issues.Caroline Tice
Update MountImage to only call mount_gpt_image.sh from inside the chroot; also update to allow extra commands to be executed on the mounted image. Add FindChromeOSImage to take an external path to a ChromeOS image and convert it to an internal (inside chroot) path. BUG=chromium:820185 TEST=Ran crosperf with these changes on chrotomation2, and it worked. Change-Id: I797b290987e04975fdd62f926838e06e833f18ae Reviewed-on: https://chromium-review.googlesource.com/956238 Commit-Ready: Caroline Tice <cmtice@chromium.org> Tested-by: Caroline Tice <cmtice@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
2018-03-08Remove lumpy from rotating builders.Caroline Tice
The buildbot configs have removed lumpy completely, so we should no longer try to run it in our rotating builders. BUG=chromium:820195 TEST=None Change-Id: I72191588cd53dfc515186c26b433b2a51984fda1 Reviewed-on: https://chromium-review.googlesource.com/955901 Commit-Ready: Caroline Tice <cmtice@chromium.org> Tested-by: Caroline Tice <cmtice@chromium.org> Reviewed-by: Yunlian Jiang <yunlian@chromium.org>
2018-03-08Replacing top_25_smooth with cros_ui_smoothness in crosperfKuo-Hsin Yang
top_25_smooth was replaced in CL:903246. BUG=chromium:818100 TEST=None Change-Id: I5bfe5659b06192237b0a985abb03b99139928e17 Reviewed-on: https://chromium-review.googlesource.com/954809 Commit-Ready: Vovo Yang <vovoy@chromium.org> Tested-by: Vovo Yang <vovoy@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org> Reviewed-by: Chung-yih Wang <cywang@google.com>
2018-03-07crosperf: restart ui instead of reboot machine.Yunlian Jiang
We restart machine to try to get a clean state for testing. But the restart beeping is anoying, as a workaround, we restart ui instead of reboot machine. BUG=None TEST=unittest passes. Change-Id: Idec234e86af661213206f954eda1fb0967c74024 Reviewed-on: https://chromium-review.googlesource.com/953864 Commit-Ready: Yunlian Jiang <yunlian@chromium.org> Tested-by: Yunlian Jiang <yunlian@chromium.org> Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
2018-03-05debug_info_test: test !gcc rather than clang.Ting-Yuan Huang
There are other producers: gnu as, go, etc. Also update the whitelist. BUG=chromium:817648 TEST=debug_info_test Change-Id: Idc1ded503ed926fc85fbd55a97543583584468bd Reviewed-on: https://chromium-review.googlesource.com/947883 Commit-Ready: Ting-Yuan Huang <laszio@chromium.org> Tested-by: Ting-Yuan Huang <laszio@chromium.org> Reviewed-by: Yunlian Jiang <yunlian@chromium.org>