aboutsummaryrefslogtreecommitdiffhomepage
path: root/infra/bots/recipes
Commit message (Collapse)AuthorAge
* Fix tasks.json and the infra botGravatar Eric Boren2017-04-19
| | | | | | | | | | | | | | Diffs caused by a collision between two CLs: https://skia-review.googlesource.com/c/13597/ https://skia-review.googlesource.com/c/13770/ Bug: skia: Change-Id: I1173560148e54a273df2ce16bcd8365557938873 Reviewed-on: https://skia-review.googlesource.com/13808 Reviewed-by: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Matt Sarett <msarett@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Update blacklist for Iris 540 Vulkan to exclude new testGravatar Brian Osman2017-04-19
| | | | | | | | | | | This test is very similar to the existing WritePixels_Gpu test, which was blacklisted for failing in the same way. Bug: skia: Change-Id: Id8e4253e030e01e215f161f81969764bc8364352 Reviewed-on: https://skia-review.googlesource.com/13809 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
* Recipes: Use strict coverage, part 1Gravatar Eric Boren2017-04-19
| | | | | | | | | | Shouldn't change any behavior. Bug: skia: Change-Id: I953e0da886a6d61096eba7915101f245a6cec741 Reviewed-on: https://skia-review.googlesource.com/13765 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Revert "Roll recipe dependencies (nontrivial)."Gravatar Eric Boren2017-04-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 6bc873f0be2c3351d34d92e85910ed7fab7a6bce. Reason for revert: Broke CT SKPs bots Original change's description: > Roll recipe dependencies (nontrivial). > > This is an automated CL created by the recipe roller. This CL rolls recipe > changes from upstream projects (e.g. depot_tools) into downstream projects > (e.g. tools/build). > > > Please review the expectation changes, and LGTM+CQ. > > > More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug. > build: > https://crrev.com/06e6b3accc66f6c31053055c8e0efcd978f18b03 Implement a generic postprocess/merge hook for swarming tasks. (RELAND 4) (jbudorick@chromium.org) > > > Change-Id: Ib62301f7336873f022cd5a7efe8df577c1414689 > Recipe-Tryjob-Bypass-Reason: Autoroller > Bugdroid-Send-Email: False > Reviewed-on: https://skia-review.googlesource.com/13775 > Commit-Queue: Eric Boren <borenet@google.com> > Reviewed-by: Eric Boren <borenet@google.com> > TBR=borenet@google.com,rmistry@google.com,recipe-roller@chromium.org,reviews@skia.org,iannucci@google.com,jbudorick@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Change-Id: I534dfb1d9adbb57ccafa1c1306b852106a7f8c31 Reviewed-on: https://skia-review.googlesource.com/13778 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Roll recipe dependencies (nontrivial).Gravatar recipe-roller2017-04-19
| | | | | | | | | | | | | | | | | | | | | | This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). Please review the expectation changes, and LGTM+CQ. More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug. build: https://crrev.com/06e6b3accc66f6c31053055c8e0efcd978f18b03 Implement a generic postprocess/merge hook for swarming tasks. (RELAND 4) (jbudorick@chromium.org) Change-Id: Ib62301f7336873f022cd5a7efe8df577c1414689 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Reviewed-on: https://skia-review.googlesource.com/13775 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Use new recipes.py test commandGravatar Eric Boren2017-04-18
| | | | | | | | Bug: skia: Change-Id: I669e96bb62ce7c91ef40f48d01bca55128db5016 Reviewed-on: https://skia-review.googlesource.com/13688 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* RecreateSKPs: Use api.run instead of api.stepGravatar Eric Boren2017-04-17
| | | | | | | | | | This gets us the default env, which includes the missing depot_tools. NOTRY=true Bug: skia: Change-Id: Idc6f9c426eb88c36d8c5d9c9bf9568a9414d61a6 Reviewed-on: https://skia-review.googlesource.com/13587 Reviewed-by: Ravi Mistry <rmistry@google.com>
* skirt std::chrono on MSAN buildsGravatar Mike Klein2017-04-14
| | | | | | | | | | | | | | | I don't know why, but only std::chrono's calls to clock_gettime() seem to be affected by this MSAN bug. Other calls into libc++ that call libc, like std::to_string(int) calling snprintf, work fine. CQ_INCLUDE_TRYBOTS=skia.primary:Test-Ubuntu-Clang-GCE-CPU-AVX2-x86_64-Debug-MSAN,Perf-Ubuntu-Clang-GCE-CPU-AVX2-x86_64-Debug-MSAN BUG=skia:6504 Change-Id: I73fbe8793d2b5b5cca46ed68fb078a77d8748127 Reviewed-on: https://skia-review.googlesource.com/13502 Reviewed-by: Ben Wagner <bungeman@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* Do non-deterministic FP opt testing on all botsGravatar Brian Osman2017-04-14
| | | | | | | | Bug: skia:6395 Change-Id: Ic5065bfc91cac83495836cd6d2a7cc059423c96e Reviewed-on: https://skia-review.googlesource.com/13412 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
* jumper, explicitly tag functions as functionsGravatar Mike Klein2017-04-13
| | | | | | | | | | | | | | | | | | This helps the linker decide to rewrite bl -> blx when linking Thumb2 SkJumper.o code with ARM SkJumper_generated.o. The reason Android wasn't failing is because it somehow figured out to do this without these .type directives. We use a different toolchain for ChromeOS builds that I guess needs more handholding. BUG=skia:6471 CQ_INCLUDE_TRYBOTS=skia.primary:Test-ChromeOS-Clang-Chromebook_C100p-GPU-MaliT764-arm-Release,Test-ChromeOS-Clang-Chromebook_C100p-GPU-MaliT764-arm-Debug Change-Id: I4a5c50b6ab7683512776c70aec6e9a75a0999787 Reviewed-on: https://skia-review.googlesource.com/13464 Reviewed-by: Herb Derby <herb@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* c100p bots: put a bandaid on broken ThumbGravatar Mike Klein2017-04-13
| | | | | | | | | | | BUG=skia:6471 CQ_INCLUDE_TRYBOTS=skia.primary:Test-ChromeOS-Clang-Chromebook_C100p-GPU-MaliT764-arm-Release,Test-ChromeOS-Clang-Chromebook_C100p-GPU-MaliT764-arm-Debug Change-Id: I3faa28c69510fbb33d40abbfb59bf3a7ba5ddb77 Reviewed-on: https://skia-review.googlesource.com/13407 Reviewed-by: Kevin Lubick <kjlubick@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* Add infra/bots/Makefile to simplify test/trainGravatar Eric Boren2017-04-13
| | | | | | | | | | | | | | | eg. $ make test or $ make train Update docs accordingly. Bug: skia: Change-Id: I069fa686833f0a78f398bd85eb8bb44025cae0d5 Reviewed-on: https://skia-review.googlesource.com/13327 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* re-disable lld on MSAN botsGravatar Mike Klein2017-04-13
| | | | | | | | | | | | | When using lld, calls to clock_gettime() end up in the real libc clock_gettime(). When using ld, calls to clock_gettime() end up in __interceptor_clock_gettime(). Don't know why yet. Change-Id: I41f54218eb961e07ed2ef2c37b864e9aeeb49662 Reviewed-on: https://skia-review.googlesource.com/13403 Reviewed-by: Kevin Lubick <kjlubick@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* Change infra recipe module to use the Go assetGravatar Eric Boren2017-04-13
| | | | | | | | Bug: skia: Change-Id: Ib2fa0b971f201ea2f16afd0467d7882b66b4047c Reviewed-on: https://skia-review.googlesource.com/13331 Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Set skia_enable_gpu=false on MSAN builds.Gravatar Mike Klein2017-04-13
| | | | | | | | | | | | This should get us past the crash at startup. The next problem I'd expect to see, if any, should involve HumanizeMS() and chrono::steady_clock::now(). Change-Id: I14dbe6bedd6939c1a355db8dba74ab0853db7706 Reviewed-on: https://skia-review.googlesource.com/13401 Commit-Queue: Mike Klein <mtklein@chromium.org> Reviewed-by: Kevin Lubick <kjlubick@google.com>
* try -BGravatar Mike Klein2017-04-13
| | | | | | Change-Id: I4e03d668197d76feb71113f47bae24fdd2fc6b6f Reviewed-on: https://skia-review.googlesource.com/13334 Reviewed-by: Kevin Lubick <kjlubick@google.com>
* Update clang_linux to version 4Gravatar Kevin Lubick2017-04-13
| | | | | | | | Bug: skia: Change-Id: Iab4fcd7b8f17fa7bf478e879a325759e0c622b1a Reviewed-on: https://skia-review.googlesource.com/13120 Commit-Queue: Mike Klein <mtklein@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
* Add jobs for Samsung Chromebook PlusGravatar Kevin Lubick2017-04-12
| | | | | | | | | | | Also refactor the cipd package used to compile these, as it can be reused. Bug: skia: NOTRY=true Change-Id: I53cf5b7fdfcb8cafd50791a0beaaf557fafa65c3 Reviewed-on: https://skia-review.googlesource.com/13252 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Add/edit README files for infra/recipesGravatar Eric Boren2017-04-12
| | | | | | | | | | | Bug: skia:6473 Change-Id: I050df47a5a3f633263f0ca6258e83e96b78d7957 Reviewed-on: https://skia-review.googlesource.com/13123 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Stephan Altmueller <stephana@google.com> Reviewed-by: Mike Klein <mtklein@chromium.org> Reviewed-by: Kevin Lubick <kjlubick@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com>
* Re-align tests in recipes with the names of realityGravatar Kevin Lubick2017-04-12
| | | | | | | | | For example, renaming GN_Android to Android Bug: skia: Change-Id: Ib71001b3cbecc3d5a0f3bb1bf0cf1f48dc23cfe4 Reviewed-on: https://skia-review.googlesource.com/13182 Reviewed-by: Eric Boren <borenet@google.com>
* Blacklist DeferredTextureImage on Win Vulkan IntelIris540Gravatar Matt Sarett2017-04-11
| | | | | | | | | | CQ_INCLUDE_TRYBOTS=skia.primary:Test-Win10-MSVC-NUC6i5SYK-GPU-IntelIris540-x86_64-Release-Vulkan Bug: skia: Change-Id: I2e465a4668956d4337ecde4f65081bf22b3b0e7a Reviewed-on: https://skia-review.googlesource.com/13183 Commit-Queue: Mike Klein <mtklein@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
* Fix symbolization of ASAN crashesGravatar Kevin Lubick2017-04-11
| | | | | | | | Bug: skia: Change-Id: Id127a3c4a1a40bc6ddb0641c9d72f1a5960bbc6e Reviewed-on: https://skia-review.googlesource.com/13125 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Kevin Lubick <kjlubick@google.com>
* Add MIPS Ci20 jobsGravatar Kevin Lubick2017-04-10
| | | | | | | | | Bug: skia:6423 NOTRY=true Change-Id: I2193c49151359a174ffa071650e1dbd366c91b28 Reviewed-on: https://skia-review.googlesource.com/11783 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Mike Klein <mtklein@chromium.org>
* Remove GeForce 320M botsGravatar Eric Boren2017-04-10
| | | | | | | | Bug: skia: Change-Id: I3ae75d4edde1b3c8cc44413db74e5ab9789a9020 Reviewed-on: https://skia-review.googlesource.com/13065 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Recipes: Remove depot_tools path hackGravatar Eric Boren2017-04-10
| | | | | | | | Bug: skia:6473 Change-Id: I722059844bb0bc9d58347b4f6002ee62043b9e20 Reviewed-on: https://skia-review.googlesource.com/13006 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Clean up no-longer-used recipes stuffGravatar Eric Boren2017-04-10
| | | | | | | | | | | | - Remove buildbot logic - Remove non-Gerrit logic - Remove master, slave, and build number properties Bug: skia:6473 Change-Id: I203d523f3a0e03f933c0c48d29c7aabac503b2b9 Reviewed-on: https://skia-review.googlesource.com/12625 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Move recipe content from modules back into recipesGravatar Eric Boren2017-04-10
| | | | | | | | | | | | At one point I moved the contents of the recipes into modules so that they could be shared between repos. It turns out that we don't need that, and it adds complexity. Bug: skia:6473 Change-Id: I75a920b6a8474dcdd8b37ee9edd52aac801d1ab0 Reviewed-on: https://skia-review.googlesource.com/12622 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Rename recipes: swarm_[name] -> [name]Gravatar Eric Boren2017-04-10
| | | | | | | | Bug: skia:6473 Change-Id: Id78f0b5cb50ae2243b9a5aed8e552f7ed8202228 Reviewed-on: https://skia-review.googlesource.com/12060 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Ravi Mistry <rmistry@google.com>
* Reland: Use bundled recipes for all test/perf tasksGravatar Eric Boren2017-04-07
| | | | | | | | | | | It won't save as much time for faster machines, but it should still shave a few seconds, and it'll be nice to be consistent. Bug: skia: Change-Id: I0b2aa010579b527c66325cb5b691bfb549b31316 Reviewed-on: https://skia-review.googlesource.com/11418 Reviewed-by: Kevin Lubick <kjlubick@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Commit the patch before bundling recipesGravatar Eric Boren2017-04-06
| | | | | | | | | | | This was part of https://skia-review.googlesource.com/c/11340/ but split out before relanding. Bug: skia:5813 Change-Id: I5cc97a588300a8656989a8d73e1231bdd606ef39 Reviewed-on: https://skia-review.googlesource.com/11483 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Revert "Use bundled recipes on all Test/Perf tasks"Gravatar Eric Boren2017-04-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 43b9c6bbf66b0927a99062c68dff9ea8358f82db. Reason for revert: Windows failing Original change's description: > Use bundled recipes on all Test/Perf tasks > > It won't save as much time for faster machines, but it should still > shave a few seconds, and it'll be nice to be consistent. > > Bug: skia:5813 > Change-Id: I5724e7aae83851edff3129265a9ffee5c3f95825 > Reviewed-on: https://skia-review.googlesource.com/11340 > Commit-Queue: Eric Boren <borenet@google.com> > Reviewed-by: Kevin Lubick <kjlubick@google.com> > TBR=borenet@google.com,kjlubick@google.com,reviews@skia.org,iannucci@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Change-Id: I2a9cc1c12079755969879e37da5cae9f93921ddd Reviewed-on: https://skia-review.googlesource.com/11416 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Use bundled recipes on all Test/Perf tasksGravatar Eric Boren2017-04-06
| | | | | | | | | | | It won't save as much time for faster machines, but it should still shave a few seconds, and it'll be nice to be consistent. Bug: skia:5813 Change-Id: I5724e7aae83851edff3129265a9ffee5c3f95825 Reviewed-on: https://skia-review.googlesource.com/11340 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Kevin Lubick <kjlubick@google.com>
* Move iOS app packaging to the cloudGravatar Stephan Altmueller2017-04-05
| | | | | | | | | | | | | | | | | This moves the signing and packaging of ios apps from the host that drives the device to the cloud machine where the app is built. This is in preparation of moving to Raspberry Pi hosts. BUG=skia:5331 Change-Id: I4918272a6c734f5344351f0726b7721fde042234 Reviewed-on: https://skia-review.googlesource.com/9906 Commit-Queue: Stephan Altmueller <stephana@google.com> Reviewed-by: Eric Boren <borenet@google.com> Reviewed-by: Mike Klein <mtklein@chromium.org>
* Bundle recipes for use on RPiGravatar Eric Boren2017-04-04
| | | | | | | | | | This saves ~1:30 of overhead when the isolate cache is warm. Bug: skia:5813 Change-Id: I5053d66e865ca6500b82567f83be8042568e22cf Reviewed-on: https://skia-review.googlesource.com/11063 Reviewed-by: Kevin Lubick <kjlubick@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Manually roll recipesGravatar Robert Iannucci2017-03-27
| | | | | | | | | | | build: crrev.com/232ecb4e8e9a3175db1c14449dca0230ae01b80c Switch to an explicit swarming.summary placeholder instead of json.output. Bug: skia: Change-Id: I7c0dd6ce6225d48541c574ff6f38ea70e1823f1f Reviewed-on: https://skia-review.googlesource.com/10126 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Manually roll recipesGravatar Robert Iannucci2017-03-27
| | | | | | | | | | | | | | | | | | | Removes unused recipe expectation. build: https://crrev.com/e0f3d95ef043b98fcceb1f34b03f9b8fac091eb9 Roll recipe dependencies (trivial). depot_tools: https://crrev.com/0bbe6fa00ef873080c4f6d145ea7a00497eeed3f Roll recipe dependencies (trivial). recipe_engine: https://crrev.com/e300f3a3b8193b84b88599faa30c0301e155c906 Handle unused recipe expectations in new 'test' command Bug: skia: Change-Id: I3f00821de5b7f5158c6da2ff1206f2e1ec133e0f Reviewed-on: https://skia-review.googlesource.com/10124 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Followup fixes for recipe rollGravatar Eric Boren2017-03-21
| | | | | | | | | | | | | | Simplifying some environment madness. Nobody gets to modify default_env. Always apply it in run.__call__(). Add depot_tools path to PATH in default_env. Result is simpler but now default_env is applied in many more places, which should be harmless. BUG=skia: Change-Id: I8299d50bde2bc3e2d47568a267d1d962a3fc3d56 Reviewed-on: https://skia-review.googlesource.com/9916 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Roll Recipe DEPSGravatar Eric Boren2017-03-21
| | | | | | | | | BUG=skia: Change-Id: Ic89d1b49aba842fb41c82ad6fc8f5ebe787893bd Reviewed-on: https://skia-review.googlesource.com/9884 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Ravi Mistry <rmistry@google.com>
* Roll Recipe DEPSGravatar Eric Boren2017-03-20
| | | | | | | | | | | Backing up from https://skia-review.googlesource.com/c/9884 due to failures. BUG=skia: Change-Id: I12182044191f5df56518f355262bbf83f8c5c1f8 Reviewed-on: https://skia-review.googlesource.com/9895 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Switch 16 sample gpu configs to 8 samplesGravatar Brian Salomon2017-03-20
| | | | | | | | | | BUG=skia: Change-Id: I053cce7c05c74860cbace00cd94ea4b96e9a482c Reviewed-on: https://skia-review.googlesource.com/9889 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com>
* Widen nondetmerministic testing of processors to all Ubuntu botsGravatar Brian Osman2017-03-20
| | | | | | | | | BUG=skia:6395 Change-Id: Ieca04f9e5ce534dc7e2be9c867907d5fa0b70a15 Reviewed-on: https://skia-review.googlesource.com/9890 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
* Revert "Revert "Add color spin test for SkColorSpaceXformCanvas""Gravatar Mike Klein2017-03-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 73e21af21390c2806eb1350253233903808edd6b. Reason for revert: I will fix the broken bot next week. Original change's description: > Revert "Add color spin test for SkColorSpaceXformCanvas" > > This reverts commit cb01aec63bcb3dee52afcf3605bcd64166b873c0. > > Reason for revert: Breaks Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Debug-SK_USE_DISCARDABLE_SCALEDIMAGECACHE > > Original change's description: > > Add color spin test for SkColorSpaceXformCanvas > > > > Also changes behavior to treat nullptr srcs as sRGB. > > > > Testing locally, it looks like 353 gms have no diffs from 8888. > > There are 269 diffs - some are fine (gms that do color space stuff) > > and some are bugs. > > > > BUG=skia: > > > > Change-Id: I55c2825f4f4b857e0b0a0ec050c6db82ac881492 > > Reviewed-on: https://skia-review.googlesource.com/9738 > > Reviewed-by: Brian Osman <brianosman@google.com> > > Commit-Queue: Matt Sarett <msarett@google.com> > > > > TBR=mtklein@google.com,msarett@google.com,brianosman@google.com,reviews@skia.org > # Not skipping CQ checks because original CL landed > 1 day ago. > BUG=skia: > > Change-Id: I70bb69f747b863d267494e37a60888a51ab0184c > Reviewed-on: https://skia-review.googlesource.com/9823 > Reviewed-by: Eric Boren <borenet@google.com> > Commit-Queue: Eric Boren <borenet@google.com> > TBR=borenet@google.com,mtklein@google.com,msarett@google.com,reviews@skia.org,brianosman@google.com # Not skipping CQ checks because original CL landed > 1 day ago. BUG=skia: Change-Id: I766382e6655f614042cded84f547f9fd5b109fca Reviewed-on: https://skia-review.googlesource.com/9879 Reviewed-by: Mike Klein <mtklein@google.com> Commit-Queue: Mike Klein <mtklein@google.com>
* Run 4 sample msaa configs for perf on iOSGravatar Brian Salomon2017-03-17
| | | | | | | | | | Also remove seemingly incorrect DM suppression for msaa16 on iOS. Change-Id: I3e86dc9c3c625a1b802a6edfbb9db3b708ecba4f Reviewed-on: https://skia-review.googlesource.com/9863 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com>
* Be explicit on test and perf bots about GL vs GLESGravatar Brian Salomon2017-03-17
| | | | | | | | | | This changes the names of several shorthand names for gpu configs to be prefixed either with "gl" or "gles" and makes the bots only use such configs. It adds some missing named configs. Change-Id: Iea4e0e2ddafe0ac08e623111a15be7335156957c Reviewed-on: https://skia-review.googlesource.com/9833 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com>
* Revert "Add color spin test for SkColorSpaceXformCanvas"Gravatar Eric Boren2017-03-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit cb01aec63bcb3dee52afcf3605bcd64166b873c0. Reason for revert: Breaks Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Debug-SK_USE_DISCARDABLE_SCALEDIMAGECACHE Original change's description: > Add color spin test for SkColorSpaceXformCanvas > > Also changes behavior to treat nullptr srcs as sRGB. > > Testing locally, it looks like 353 gms have no diffs from 8888. > There are 269 diffs - some are fine (gms that do color space stuff) > and some are bugs. > > BUG=skia: > > Change-Id: I55c2825f4f4b857e0b0a0ec050c6db82ac881492 > Reviewed-on: https://skia-review.googlesource.com/9738 > Reviewed-by: Brian Osman <brianosman@google.com> > Commit-Queue: Matt Sarett <msarett@google.com> > TBR=mtklein@google.com,msarett@google.com,brianosman@google.com,reviews@skia.org # Not skipping CQ checks because original CL landed > 1 day ago. BUG=skia: Change-Id: I70bb69f747b863d267494e37a60888a51ab0184c Reviewed-on: https://skia-review.googlesource.com/9823 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Reboot PixelCs before every taskGravatar Kevin Lubick2017-03-16
| | | | | | | | | | | | | | | Just like the NexusPlayers, swarming can't reboot the PixelCs. This may be causing the strange state we see in tasks like: https://chromium-swarm.appspot.com/task?id=34ed0f1bdcd5a610&refresh=10 BUG=skia: Change-Id: Icb0b4a5564d34093abb463ca8ef5da43d066db4a Reviewed-on: https://skia-review.googlesource.com/9818 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Kevin Lubick <kjlubick@google.com>
* Add color spin test for SkColorSpaceXformCanvasGravatar Matt Sarett2017-03-15
| | | | | | | | | | | | | | | Also changes behavior to treat nullptr srcs as sRGB. Testing locally, it looks like 353 gms have no diffs from 8888. There are 269 diffs - some are fine (gms that do color space stuff) and some are bugs. BUG=skia: Change-Id: I55c2825f4f4b857e0b0a0ec050c6db82ac881492 Reviewed-on: https://skia-review.googlesource.com/9738 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Matt Sarett <msarett@google.com>
* Roll Recipe DEPSGravatar Eric Boren2017-03-06
| | | | | | | | | | | | Just one commit: 97ab90f Reland "Make Ubuntu-14.04 the default version of Linux used when triggering Swarming tasks" BUG=skia: Change-Id: Ie9f37a1d1bb8326cf34123765618221b1460d6ea Reviewed-on: https://skia-review.googlesource.com/9290 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Roll Recipe DEPSGravatar Eric Boren2017-03-06
| | | | | | | | | BUG=skia: Change-Id: Id93e801d1ad9df5a3ce82a4b6b2ec52e3fc2981b Reviewed-on: https://skia-review.googlesource.com/9288 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Remove extract_comments in housekeeper bot.Gravatar Joe Gregorio2017-03-03
| | | | | | | | | BUG=skia: Change-Id: I6bd745a03c9f8c06ed711839a4bda1db0693cf5a Reviewed-on: https://skia-review.googlesource.com/9220 Reviewed-by: Cary Clark <caryclark@google.com> Commit-Queue: Joe Gregorio <jcgregorio@google.com>