aboutsummaryrefslogtreecommitdiffhomepage
path: root/infra/bots/recipe_modules/core
Commit message (Collapse)AuthorAge
* Infra: Enable cross-repo tryjobsGravatar Eric Boren2017-04-21
| | | | | | | | Bug: skia: Change-Id: I560eb00b4b19643346b038822869e5bc77ce1454 Reviewed-on: https://skia-review.googlesource.com/13652 Reviewed-by: Ben Wagner <benjaminwagner@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Recipes: remove some pragma: no coverGravatar Eric Boren2017-04-19
| | | | | | | | | | | | | | | | These are the remaining instances, and they're harder to deal with: $ git grep pragma recipe_modules/swarming/api.py: except self.m.step.StepFailure as e: # pragma: no cover recipe_modules/swarming/api.py: if len(ids) > 1: # pragma: nocover recipes/ct_skps.py: if api.path.exists(version_file): # pragma: nocover recipes/ct_skps.py: if api.path.exists(blacklist_file): # pragma: nocover recipes/upload_nano_results.py: if len(results) != 1: # pragma: nocover Bug: skia:6473 Change-Id: If75e2ffcf39a944b513b9005bbc4bd9c4fb031f6 Reviewed-on: https://skia-review.googlesource.com/13867 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Eric Boren <borenet@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>
* 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>
* 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>
* 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>
* 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 DEPS againGravatar Eric Boren2017-02-21
| | | | | | | | | | | | This time with coverage bypass BUG=skia: Change-Id: I7f51dcad7bad53c7209ac512876a0d72c6181b8e Reviewed-on: https://skia-review.googlesource.com/8768 Reviewed-by: Eric Boren <borenet@google.com> Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Roll recipe DEPSGravatar Eric Boren2017-02-21
| | | | | | | | | | | Attempt 2. Lots of fixes due to upstream breaks. BUG=skia: Change-Id: Ie05b2da1790fbcce5c45ef09c4c832e09de6b5c5 Reviewed-on: https://skia-review.googlesource.com/8762 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Stream logs in symbolized dm/nanobenchGravatar Kevin Lubick2017-02-21
| | | | | | | | | BUG=skia: Change-Id: I693bec4daa819f3c7802aa54f452b47f228b370b Reviewed-on: https://skia-review.googlesource.com/8673 Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Kevin Lubick <kjlubick@google.com>
* Revert "Roll Recipe DEPS"Gravatar Eric Boren2017-02-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 310f4d02c1e2d01181c1db18acf05f2846f94cd4. Reason for revert: Broke uploads Original change's description: > Roll Recipe DEPS > > BUG=skia: > > Change-Id: I3e32207e5809e126d53ee8ec2158eb855c1a26d2 > Reviewed-on: https://skia-review.googlesource.com/8671 > Commit-Queue: Eric Boren <borenet@google.com> > Reviewed-by: Ravi Mistry <rmistry@google.com> > TBR=borenet@google.com,rmistry@google.com,reviews@skia.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Change-Id: If6eed7a72aa7178afd23d0a4a7786e23b27c1ac2 Reviewed-on: https://skia-review.googlesource.com/8681 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Roll Recipe DEPSGravatar Eric Boren2017-02-17
| | | | | | | | | BUG=skia: Change-Id: I3e32207e5809e126d53ee8ec2158eb855c1a26d2 Reviewed-on: https://skia-review.googlesource.com/8671 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Ravi Mistry <rmistry@google.com>
* Add Flutter compile botGravatar Ravi Mistry2017-02-17
| | | | | | | | | BUG=skia:6227 Change-Id: I997663e49c407e3ec64c8e67a2ca27700d5081b4 Reviewed-on: https://skia-review.googlesource.com/8448 Commit-Queue: Ravi Mistry <rmistry@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Symbolize any stacktraces in DM and nanobenchGravatar Kevin Lubick2017-02-15
| | | | | | | | | | | | As a soft rollout, this only affects the Linux NUCS (on Ubuntu16). BUG=skia:6206 Change-Id: Ic314e85159cd3c0d9e55bbdce412a8d61adebb33 Reviewed-on: https://skia-review.googlesource.com/8276 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Ravi Mistry <rmistry@google.com>
* Copy modules back in from skia-recipes repoGravatar Eric Boren2017-02-02
| | | | | | | | | BUG=skia:6070 Change-Id: I86b3f039c27bd8287516c1db8bd1cae62888171b Reviewed-on: https://skia-review.googlesource.com/7890 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Move recipe modules to shared repoGravatar Eric Boren2017-01-17
| | | | | | | | | | BUG=skia:6070 Change-Id: I6c487e77ddfddb6164b983981dff5ee8ae870376 Reviewed-on: https://skia-review.googlesource.com/6946 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Ravi Mistry <rmistry@google.com> Reviewed-by: Stephan Altmueller <stephana@google.com>
* Fix zero-padding in gerrit refGravatar borenet2016-11-07
| | | | | | | | BUG=skia: NOTRY=true GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2484763002 Review-Url: https://codereview.chromium.org/2484763002
* Make buildbotless trybots only support GerritGravatar skia.buildbots2016-11-04
| | | | | | | | | | | BUG=skia:5723 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4460 Change-Id: Ia5630de72f8b473551a5357fdf50eb3dd288b9a7 Reviewed-on: https://skia-review.googlesource.com/4460 Reviewed-by: Ben Wagner <benjaminwagner@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Change recipes to use bot_updateGravatar borenet2016-10-25
| | | | | | | | NOTREECHECKS=true BUG=skia:5588 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2448973002 Review-Url: https://codereview.chromium.org/2448973002
* Make the housekeeper upload doxygen to a newer bucketGravatar borenet2016-10-04
| | | | | | | | BUG=skia:2969 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2370153003 DOCS_PREVIEW= https://skia.org/?cl=2370153003 Review-Url: https://codereview.chromium.org/2370153003
* Make binary size analysis upload to skia-perf bucketGravatar borenet2016-10-04
| | | | | | | | | | It's currently pointed at chromium-skia-gm, but Perf results are now uploaded to skia-perf. BUG=skia:2969 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2371193002 Review-Url: https://codereview.chromium.org/2371193002
* Delete unused upload_*_results.pyGravatar borenet2016-09-27
| | | | | | | BUG=skia:5719 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2371883004 Review-Url: https://codereview.chromium.org/2371883004
* Attempt to fix nanobench uploadGravatar borenet2016-09-20
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2356813003 Review-Url: https://codereview.chromium.org/2356813003
* Clean up target_os=llvm now that it does nothing.Gravatar mtklein2016-09-08
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2317373004 Review-Url: https://codereview.chromium.org/2317373004
* Apply gerrit ref if it is a Gerrit patch.Gravatar rmistry2016-08-24
| | | | | | | | | | | Also: * Pass through a new property 'patch_storage' to DM/Nanobench/Coverage. This will be used by the different frameworks to figure out if it is Rietveld or Gerrit issue. * Calculate issue and patchset for Gerrit patches similar to Rietveld. BUG=skia:5627 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2263323002 Review-Url: https://codereview.chromium.org/2263323002
* [recipes] Increase test coverageGravatar borenet2016-08-05
| | | | | | | | | Remove a bunch of "pragma: no cover" by removing unused code or adding test bots to cover. BUG=skia:5578 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2215443003 Review-Url: https://codereview.chromium.org/2215443003
* [recipes] Stop pretending that buildbot_spec is a step, simplifyGravatar borenet2016-08-04
| | | | | | | | | | | | Since we no longer have to call out to obtain the buildbot_spec, merge the two functions in vars API. As a side effect, this applies default_env to the sync steps as well, which shouldn't have an appreciable effect on bot behavior. BUG=skia:5578 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2209343002 Review-Url: https://codereview.chromium.org/2209343002
* Move builder_spec, [dm|nanobench]_flags, builder_name_schema to recipesGravatar borenet2016-08-04
| | | | | | | | | | | | | | - builder_name_schema becomes its own recipe module. - builder_spec, dm, and nanobench flags move into vars module. - recipe expectation diffs include: - no more buildbot_spec.py step - "real" dm and nanobench flags, instead of --dummy-flags - some inconsequential stuff in visualbench, which is removed anyway. BUG=skia:5578 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2215803002 Review-Url: https://codereview.chromium.org/2215803002
* [Recipes] Move test and perf steps into test and perf recipesGravatar borenet2016-08-03
| | | | | | | | | This looks like another big change, but I really just moved blocks of code around. Again, no expectations diffs because no behavior is changed. BUG=skia:5578 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2198973002 Review-Url: https://codereview.chromium.org/2198973002
* Fix fake_specs.py, update recipe expectationGravatar borenet2016-08-03
| | | | | | | BUG=skia:5578 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2200763002 Review-Url: https://codereview.chromium.org/2200763002
* Re-organize Skia recipesGravatar borenet2016-08-03
Break Skia recipe module into: - skia_vars: defines and stores variables (eg. paths) - skia_step: utilities for running Skia steps - skia_flavor: flavor-specific stuff - skia: top-level setup, checkout, test/perf steps, etc etc This establishes a saner dependency structure for the recipes; skia_vars is at the bottom level, skia_step depends on it, skia_flavor depends on both of them, skia depends on all of the above, and the recipes themselves may depend on any or all of them. Next steps: - Merge buildbot_spec into skia_vars - Move test_steps and perf_steps from skia recipe_module into swarm_test and swarm_perf recipes - Cleaner checkout_steps process BUG=skia:5578 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2198173002 Review-Url: https://codereview.chromium.org/2198173002
generated by cgit v1.2.3 (git 2.39.1) at 2024-10-01 05:42:13 +0000