aboutsummaryrefslogtreecommitdiffhomepage
path: root/infra/bots/recipe_modules/flavor/api.py
Commit message (Collapse)AuthorAge
* [recipes] Provide tokenized extra_config and use it most places.Gravatar Ben Wagner2017-12-13
| | | | | | | Change-Id: I7baae5f90b2e510b66443cda449071c7c6ec9ec7 Reviewed-on: https://skia-review.googlesource.com/83520 Commit-Queue: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* [recipes] Fix *_VERSION step failure handlingGravatar Eric Boren2017-04-24
| | | | | | | | | | Bug: skia:6473 Change-Id: I2fa6f800f59f40f74b5c080c52cdec3a32329ef7 NOTRY=true Reviewed-on: https://skia-review.googlesource.com/14240 Commit-Queue: Stephan Altmueller <stephana@google.com> Reviewed-by: Kevin Lubick <kjlubick@google.com> Reviewed-by: Stephan Altmueller <stephana@google.com>
* Reland Recipes: Use strict coverage, part 2Gravatar Eric Boren2017-04-19
| | | | | | | | Bug: skia:6473 Change-Id: I90c1c3a3108ce583df60933150d97b426173a315 Reviewed-on: https://skia-review.googlesource.com/13840 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Revert "Recipes: Use strict coverage, part 2"Gravatar Eric Boren2017-04-19
| | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 4dbc9cead90eb31756a1c1bf61ea01f223c5fe96. Reason for revert: flaky? Original change's description: > Recipes: Use strict coverage, part 2 > > Bug: skia:6473 > Change-Id: Ia3a1587215a6a868c9577f2d864bc5c4e671a3fa > Reviewed-on: https://skia-review.googlesource.com/13812 > Reviewed-by: Ravi Mistry <rmistry@google.com> > Commit-Queue: Eric Boren <borenet@google.com> > TBR=borenet@google.com,rmistry@google.com,reviews@skia.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Change-Id: Id746de56609fbaf5ec8ecdd22b5e60bcb3c01fa2 Reviewed-on: https://skia-review.googlesource.com/13819 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Recipes: Use strict coverage, part 2Gravatar Eric Boren2017-04-19
| | | | | | | | Bug: skia:6473 Change-Id: Ia3a1587215a6a868c9577f2d864bc5c4e671a3fa Reviewed-on: https://skia-review.googlesource.com/13812 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Add ChromeOS Test/Perf jobs and recipe changesGravatar Kevin Lubick2017-04-07
| | | | | | | | | Minor cleanup of left over Tab3 tests. Bug: skia:6442 Change-Id: I03bb4a034101a2773619abd2be7a300a70f6ac3d Reviewed-on: https://skia-review.googlesource.com/11414 Reviewed-by: 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>
* 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>
* Add recipe to build for Asus Flip ChromebookGravatar Kevin Lubick2017-04-05
| | | | | | | | | | | | | | | | | | | | | | | | This build includes the GPU-specific code (EGL, GLES) The armhf_sysroot should be generic enough to support other ARM targets, like the Raspberry Pi. I split out the EGL dependencies into their own CIPD package (chromebook_c100p_lib) to facilitate the modularity of the approach. When we add another ARM chromebook, maybe one that has different libraries, we should be able to re-use armhf_sysroot and then a newly created CIPD asset for that specific GPU (if needed). Maybe this also can be used to build for the chromecasts (thus the TODO) Bug: skia: NOTRY=true Change-Id: Icc131025932dc8d41da5be39f3c5cad0aa95d848 Reviewed-on: https://skia-review.googlesource.com/11064 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Add Chromecast Perf jobsGravatar Kevin Lubick2017-03-21
| | | | | | | | | | | | It starts out with some skps and resource-based perf because all skps wouldn't fit and skimages are far too large BUG=skia: Change-Id: Icb07ffa84d39a85a8bc595e74ef934df921e749a Reviewed-on: https://skia-review.googlesource.com/9901 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: 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>
* Add jobs for building for ChromecastGravatar Kevin Lubick2017-03-08
| | | | | | | | | BUG=skia:6345 Change-Id: Iaf09eb7f57ae71687c6804221837a8cc8ef04931 Reviewed-on: https://skia-review.googlesource.com/9419 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Eric Boren <borenet@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>
* Restore ios_flavor.Gravatar Mike Klein2017-02-03
| | | | | | | | | | | Just like Android, there's enough special ceremony needed to move files to and from iOS devices that this reads most clearly as its own flavor. CQ_INCLUDE_TRYBOTS=skia.primary:Test-iOS-Clang-iPadMini4-GPU-GX6450-arm-Release Change-Id: I01c8e5494c364cc61e219568f022219f7c3f5fdc Reviewed-on: https://skia-review.googlesource.com/7988 Reviewed-by: Mike Klein <mtklein@chromium.org>
* Tear down old iOS bots, bring up one new one.Gravatar Mike Klein2017-02-02
| | | | | | | | | | | | | This replaces all GYP-based iOS bots with one new GN-based one. Ordinarily I'd do this more incrementally but we have only one iPad. This new bot will start very broken, but should be the quickest to cycle builds once I get it working. Change-Id: I41865e3ad431dea3cf023d0b33ccc88e175cf227 Reviewed-on: https://skia-review.googlesource.com/7941 Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Mike Klein <mtklein@chromium.org>
* 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>
* Add skpbench jobGravatar Kevin Lubick2016-11-14
| | | | | | | | | | | | | | | | | | | | | | | | This entails a few parts: 1. Isolating tools/skpbench/ to be used by the perf task. 2. Adding skpbench to BUILD_PRODUCTS_ISOLATE_WHITELIST 3. Add recipe commands to run skpbench. While it is conceptually similar to nanobench, it doesn't support images nor svgs, only skps, so we don't need to copy those to devices or bring them in via CIPD. 4. Add recipe commands to parse skpbench output and upload to Perf. 5. Update gen_tasks.go to handle skpbench jobs and tasks. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4551 Change-Id: I09bf0b078067279e138e95eca9316e55539ec735 Reviewed-on: https://skia-review.googlesource.com/4551 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Add GN iOS builder.Gravatar Mike Klein2016-11-07
| | | | | | | | | | | This gets up a bot to build for iOS via GN as much as we can right now. This is unlikely to be the long term structure of the iOS bots... by the time we add Test/Perf bots we'll likely need to have a gn_ios_flavor.py. But for now, this keeps the GN iOS build such as it is working. GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4457 Change-Id: Ideb10ae3f4ab5530ad153237a343dd5c0e7dd02f Reviewed-on: https://skia-review.googlesource.com/4457 Commit-Queue: Mike Klein <mtklein@chromium.org> Reviewed-by: Eric Boren <borenet@google.com>
* Some GN-related recipe cleanup.Gravatar Mike Klein2016-10-18
| | | | | | | | | | | | | | | | | | | All our bots but the iOS ones are on GN now. This cleans up a bunch of GYP and other obsolete stuff. - Nothing's using default_flavor.py any more except as a base class. - There are no -CMake, -Shared or -VisualBench bots anymore. - Only the iOS bots care about GYP_DEFINES. You'll see the PDFium bot's GYP_DEFINES change, but that doesn't matter... it's using PDFium's own (GN) build system, and it ignores GYP_DEFINES. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=3567 Change-Id: I699e10f013ea77df4dcaa1cb559c51c5bf55dfdb Reviewed-on: https://skia-review.googlesource.com/3567 Commit-Queue: Mike Klein <mtklein@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* GN: remove old Android recipe code.Gravatar mtklein2016-09-19
| | | | | | | | | | | All the Android builders, testers, and perfers are on gn_android_flavor now. Not as scary as it looks... all the big line count changes are deletes. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2352653002 Review-Url: https://codereview.chromium.org/2352653002
* Delete unused coverage recipe code.Gravatar mtklein2016-09-19
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2353483003 Review-Url: https://codereview.chromium.org/2353483003
* GN: add sanitize argGravatar mtklein2016-09-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Attempt to take over all *SAN builds. MSAN has a lot of coordination required between gn/BUILD.gn and gn_flavor.py. I'd like to follow up to move more of this into gn/BUILD.gn, to make it easier to use locally. The compile steps should be much faster now. We no longer build CMake and Clang for every run, instead using the clang_linux CIPD package. This removes the need for all the third_party/externals/llvm/... dependencies. Similarly, since we're using the clang_linux package, we no longer depend on Chrome's Clang, and thus no longer need to sync chromium on these bots. Instead of packaging up MSAN libraries and llvm-symbolizer in the compile output, I have the test / perf bots also depend on the clang_linux package. These do not vary from build to build. No more need for the xsan.blacklist -include hack: Clang, GN, and Ninja all track changes to xsan.blacklist without our help. This has the incidental effect of upgrading the compiler used by *SAN bots from Clang 3.8 to Clang 3.9. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2289343002 Review-Url: https://codereview.chromium.org/2289343002
* add gn_android_flavorGravatar mtklein2016-08-29
| | | | | | | | | | | | | | | | | | | | | | | This ought to support compiles for now. Am I picking up my CIPD ndk packages right? The main thing to note is that I'm passing the target_arch directly through as target_cpu. This means these bots will have a slightly different naming convention than we've been using, but it'll agree with what you must type yourself when using GN to build for Android: - Arm7 -> arm - Arm64 -> arm64 - Mips -> mipsel - Mips64 -> mips64el - x86 -> x86 (unchanged) - x86_64 -> x64 BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2292663002 Review-Url: https://codereview.chromium.org/2292663002
* Use SVGs CIPD package and use SVG as a DM source.Gravatar rmistry2016-08-11
| | | | | | | | | | Currently only turned on for one bot. May turn it on for more after more conversations. BUG=skia:5628 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2231943002 Review-Url: https://codereview.chromium.org/2231943002
* GN: take over some exisiting botsGravatar mtklein2016-08-11
| | | | | | | | | | | | | | | As an experiment, instead of replacing these with -GN twins, take them over in-place. This should take over: -FAST -SKFOO CQ_INCLUDE_TRYBOTS=master.client.skia:Perf-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Debug-SK_USE_DISCARDABLE_SCALEDIMAGECACHE-Trybot,Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Debug-SK_USE_DISCARDABLE_SCALEDIMAGECACHE-Trybot,Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot,Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-Fast-Trybot BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2229463002 Review-Url: https://codereview.chromium.org/2229463002
* [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] Remove build environment vars from default_envGravatar borenet2016-08-05
| | | | | | | BUG=skia:5578 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2209423002 Review-Url: https://codereview.chromium.org/2209423002
* [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
* 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