Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(183)

Unified Diff: master/skia_master_scripts/utils.py

Issue 12380071: Adding trybots for Periodic and PerCommit Housekeeper (Closed) Base URL: http://skia.googlecode.com/svn/buildbot/
Patch Set: Created 7 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « master/skia_master_scripts/housekeeping_periodic_factory.py ('k') | master/slaves.cfg » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: master/skia_master_scripts/utils.py
===================================================================
--- master/skia_master_scripts/utils.py (revision 7935)
+++ master/skia_master_scripts/utils.py (working copy)
@@ -25,6 +25,8 @@
from skia_master_scripts import android_factory
from skia_master_scripts import chromeos_factory
from skia_master_scripts import factory as skia_factory
+from skia_master_scripts import housekeeping_percommit_factory, \
+ housekeeping_periodic_factory
from skia_master_scripts import ios_factory
@@ -439,6 +441,37 @@
_MakeBuilderAndMaybeTrybotSet(factory_type=skia_factory.SkiaFactory, **kwargs)
+def MakeHousekeeperBuilderSet(helper, do_trybots, do_upload_results):
+ B = helper.Builder
+ F = helper.Factory
+
+ builder_factory_scheduler = [
+ # The Percommit housekeeper
+ ('Skia_PerCommit_House_Keeping',
+ housekeeping_percommit_factory.HouseKeepingPerCommitFactory,
+ 'skia_rel'),
+ # The Periodic housekeeper
+ ('Skia_Periodic_House_Keeping',
+ housekeeping_periodic_factory.HouseKeepingPeriodicFactory,
+ 'skia_periodic'),
+ ]
+ if do_trybots:
+ # Add the corresponding trybot builders to the above list.
+ builder_factory_scheduler.extend([
+ (builder + TRYBOT_NAME_SUFFIX, factory, 'skia_try')
+ for (builder, factory, _scheduler) in builder_factory_scheduler])
+
+ for (builder_name, factory, scheduler) in builder_factory_scheduler:
+ B(builder_name, 'f_%s' % builder_name, scheduler=scheduler)
+ F('f_%s' % builder_name,
+ factory(
+ do_upload_results=do_upload_results,
+ target_platform=skia_factory.TARGET_PLATFORM_LINUX,
+ builder_name=builder_name,
+ do_patch_step=(scheduler == 'skia_try'),
+ ).Build())
+
+
def MakeAndroidBuilderSet(extra_branches=None, **kwargs):
if not extra_branches:
extra_branches = []
« no previous file with comments | « master/skia_master_scripts/housekeeping_periodic_factory.py ('k') | master/slaves.cfg » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698