Commit e41dc5a5 authored by Owen Rodley's avatar Owen Rodley Committed by Commit Bot

Add tree closing configs for Linux builders.

Follows the current Gatekeeper config - tree closing enabled for all
builders, with a handful of builders that close the tree regardless of
which step fails.

Bug: 1094768
Change-Id: I03ddc4fbd7a02654146b93a9b1387b97a942c238
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2279267
Commit-Queue: Owen Rodley <orodley@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Reviewed-by: default avatarTakuto Ikuta <tikuta@chromium.org>
Auto-Submit: Owen Rodley <orodley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#785297}
parent 07fa7d8c
This diff is collapsed.
...@@ -805,12 +805,17 @@ def linux_builder( ...@@ -805,12 +805,17 @@ def linux_builder(
name, name,
goma_backend=builders.goma.backend.RBE_PROD, goma_backend=builders.goma.backend.RBE_PROD,
goma_jobs=builders.goma.jobs.MANY_JOBS_FOR_CI, goma_jobs=builders.goma.jobs.MANY_JOBS_FOR_CI,
tree_closing=True,
notifies=('chromium.linux',),
extra_notifies=None,
**kwargs): **kwargs):
return ci.builder( return ci.builder(
name = name, name = name,
goma_backend = goma_backend, goma_backend = goma_backend,
goma_jobs = goma_jobs, goma_jobs = goma_jobs,
mastername = 'chromium.linux', mastername = 'chromium.linux',
tree_closing = tree_closing,
notifies = list(notifies) + (extra_notifies or []),
**kwargs **kwargs
) )
......
...@@ -54,6 +54,11 @@ luci.tree_closer( ...@@ -54,6 +54,11 @@ luci.tree_closer(
failed_step_regexp = TREE_CLOSING_STEPS failed_step_regexp = TREE_CLOSING_STEPS
) )
luci.tree_closer(
name = 'close-on-any-step-failure',
tree_status_host = 'chromium-status.appspot.com',
)
def tree_closure_notifier(name, notify_emails): def tree_closure_notifier(name, notify_emails):
return luci.notifier( return luci.notifier(
name = name, name = name,
...@@ -174,3 +179,10 @@ luci.notifier( ...@@ -174,3 +179,10 @@ luci.notifier(
], ],
on_new_status = ['FAILURE'], on_new_status = ['FAILURE'],
) )
tree_closure_notifier(
name = 'chromium.linux',
notify_emails = [
'thomasanderson@chromium.org',
],
)
...@@ -839,7 +839,7 @@ ci.linux_builder( ...@@ -839,7 +839,7 @@ ci.linux_builder(
short_name = 'rel', short_name = 'rel',
), ),
main_console_view = settings.main_console_name, main_console_view = settings.main_console_name,
notifies = ['cr-fuchsia'], extra_notifies = ['cr-fuchsia'],
) )
ci.linux_builder( ci.linux_builder(
...@@ -849,7 +849,7 @@ ci.linux_builder( ...@@ -849,7 +849,7 @@ ci.linux_builder(
short_name = 'rel', short_name = 'rel',
), ),
main_console_view = settings.main_console_name, main_console_view = settings.main_console_name,
notifies = ['cr-fuchsia'], extra_notifies = ['cr-fuchsia'],
) )
ci.linux_builder( ci.linux_builder(
...@@ -898,7 +898,11 @@ ci.linux_builder( ...@@ -898,7 +898,11 @@ ci.linux_builder(
short_name = 'a64', short_name = 'a64',
), ),
main_console_view = settings.main_console_name, main_console_view = settings.main_console_name,
notifies = ['cr-fuchsia'], # Set tree_closing to false to disable the defaualt tree closer, which
# filters by step name, and instead enable tree closing for any step
# failure.
tree_closing = False,
extra_notifies = ['cr-fuchsia', 'close-on-any-step-failure'],
) )
ci.linux_builder( ci.linux_builder(
...@@ -908,7 +912,11 @@ ci.linux_builder( ...@@ -908,7 +912,11 @@ ci.linux_builder(
short_name = 'x64', short_name = 'x64',
), ),
main_console_view = settings.main_console_name, main_console_view = settings.main_console_name,
notifies = ['cr-fuchsia'], # Set tree_closing to false to disable the defaualt tree closer, which
# filters by step name, and instead enable tree closing for any step
# failure.
tree_closing = False,
extra_notifies = ['cr-fuchsia', 'close-on-any-step-failure'],
) )
ci.linux_builder( ci.linux_builder(
...@@ -918,7 +926,11 @@ ci.linux_builder( ...@@ -918,7 +926,11 @@ ci.linux_builder(
short_name = 'ozo', short_name = 'ozo',
), ),
main_console_view = settings.main_console_name, main_console_view = settings.main_console_name,
notifies = ['linux-ozone-rel'], # Set tree_closing to false to disable the defaualt tree closer, which
# filters by step name, and instead enable tree closing for any step
# failure.
tree_closing = False,
extra_notifies = ['linux-ozone-rel', 'close-on-any-step-failure'],
) )
ci.linux_builder( ci.linux_builder(
......
...@@ -2766,7 +2766,11 @@ ci.linux_builder( ...@@ -2766,7 +2766,11 @@ ci.linux_builder(
executable = 'recipe:swarming/deterministic_build', executable = 'recipe:swarming/deterministic_build',
execution_timeout = 6 * time.hour, execution_timeout = 6 * time.hour,
main_console_view = 'main', main_console_view = 'main',
notifies = ['Deterministic Linux'], # Set tree_closing to false to disable the defaualt tree closer, which
# filters by step name, and instead enable tree closing for any step
# failure.
tree_closing = False,
extra_notifies = ['Deterministic Linux', 'close-on-any-step-failure'],
) )
ci.linux_builder( ci.linux_builder(
...@@ -2788,6 +2792,8 @@ ci.linux_builder( ...@@ -2788,6 +2792,8 @@ ci.linux_builder(
category = 'linux', category = 'linux',
short_name = 'lk', short_name = 'lk',
), ),
notifies = [],
tree_closing = False,
) )
ci.linux_builder( ci.linux_builder(
...@@ -2815,7 +2821,7 @@ ci.linux_builder( ...@@ -2815,7 +2821,7 @@ ci.linux_builder(
short_name = 'dbg', short_name = 'dbg',
), ),
main_console_view = 'main', main_console_view = 'main',
notifies = ['cr-fuchsia'], extra_notifies = ['cr-fuchsia'],
) )
ci.linux_builder( ci.linux_builder(
...@@ -2854,7 +2860,7 @@ ci.linux_builder( ...@@ -2854,7 +2860,7 @@ ci.linux_builder(
schedule = '0 0,6,12,18 * * *', schedule = '0 0,6,12,18 * * *',
service_account = 'component-mapping-updater@chops-service-accounts.iam.gserviceaccount.com', service_account = 'component-mapping-updater@chops-service-accounts.iam.gserviceaccount.com',
triggered_by = [], triggered_by = [],
notifies = ['component-mapping'], extra_notifies = ['component-mapping'],
) )
......
...@@ -495,7 +495,7 @@ ci.linux_builder( ...@@ -495,7 +495,7 @@ ci.linux_builder(
console_view_entry = ci.console_view_entry( console_view_entry = ci.console_view_entry(
category = 'fuchsia|a64', category = 'fuchsia|a64',
), ),
notifies = ['cr-fuchsia'], extra_notifies = ['cr-fuchsia'],
) )
ci.linux_builder( ci.linux_builder(
...@@ -504,7 +504,7 @@ ci.linux_builder( ...@@ -504,7 +504,7 @@ ci.linux_builder(
category = 'fuchsia|x64', category = 'fuchsia|x64',
short_name = 'rel', short_name = 'rel',
), ),
notifies = ['cr-fuchsia'], extra_notifies = ['cr-fuchsia'],
) )
ci.linux_builder( ci.linux_builder(
...@@ -548,7 +548,7 @@ ci.linux_builder( ...@@ -548,7 +548,7 @@ ci.linux_builder(
category = 'fuchsia|cast', category = 'fuchsia|cast',
short_name = 'a64', short_name = 'a64',
), ),
notifies = ['cr-fuchsia'], extra_notifies = ['cr-fuchsia'],
) )
ci.linux_builder( ci.linux_builder(
...@@ -557,7 +557,7 @@ ci.linux_builder( ...@@ -557,7 +557,7 @@ ci.linux_builder(
category = 'fuchsia|cast', category = 'fuchsia|cast',
short_name = 'x64', short_name = 'x64',
), ),
notifies = ['cr-fuchsia'], extra_notifies = ['cr-fuchsia'],
) )
ci.linux_builder( ci.linux_builder(
......
...@@ -496,7 +496,7 @@ ci.linux_builder( ...@@ -496,7 +496,7 @@ ci.linux_builder(
category = 'fuchsia|a64', category = 'fuchsia|a64',
short_name = 'rel', short_name = 'rel',
), ),
notifies = ['cr-fuchsia'], extra_notifies = ['cr-fuchsia'],
) )
ci.linux_builder( ci.linux_builder(
...@@ -505,7 +505,7 @@ ci.linux_builder( ...@@ -505,7 +505,7 @@ ci.linux_builder(
category = 'fuchsia|x64', category = 'fuchsia|x64',
short_name = 'rel', short_name = 'rel',
), ),
notifies = ['cr-fuchsia'], extra_notifies = ['cr-fuchsia'],
) )
ci.linux_builder( ci.linux_builder(
...@@ -549,7 +549,7 @@ ci.linux_builder( ...@@ -549,7 +549,7 @@ ci.linux_builder(
category = 'fuchsia|cast', category = 'fuchsia|cast',
short_name = 'a64', short_name = 'a64',
), ),
notifies = ['cr-fuchsia'], extra_notifies = ['cr-fuchsia'],
) )
ci.linux_builder( ci.linux_builder(
...@@ -558,7 +558,7 @@ ci.linux_builder( ...@@ -558,7 +558,7 @@ ci.linux_builder(
category = 'fuchsia|cast', category = 'fuchsia|cast',
short_name = 'x64', short_name = 'x64',
), ),
notifies = ['cr-fuchsia'], extra_notifies = ['cr-fuchsia'],
) )
ci.linux_builder( ci.linux_builder(
......
...@@ -565,7 +565,7 @@ ci.linux_builder( ...@@ -565,7 +565,7 @@ ci.linux_builder(
category = 'fuchsia|a64', category = 'fuchsia|a64',
short_name = 'rel', short_name = 'rel',
), ),
notifies = ['cr-fuchsia'], extra_notifies = ['cr-fuchsia'],
) )
ci.linux_builder( ci.linux_builder(
...@@ -574,7 +574,7 @@ ci.linux_builder( ...@@ -574,7 +574,7 @@ ci.linux_builder(
category = 'fuchsia|x64', category = 'fuchsia|x64',
short_name = 'rel', short_name = 'rel',
), ),
notifies = ['cr-fuchsia'], extra_notifies = ['cr-fuchsia'],
) )
ci.linux_builder( ci.linux_builder(
...@@ -618,7 +618,7 @@ ci.linux_builder( ...@@ -618,7 +618,7 @@ ci.linux_builder(
category = 'fuchsia|cast', category = 'fuchsia|cast',
short_name = 'a64', short_name = 'a64',
), ),
notifies = ['cr-fuchsia'], extra_notifies = ['cr-fuchsia'],
) )
ci.linux_builder( ci.linux_builder(
...@@ -627,7 +627,7 @@ ci.linux_builder( ...@@ -627,7 +627,7 @@ ci.linux_builder(
category = 'fuchsia|cast', category = 'fuchsia|cast',
short_name = 'x64', short_name = 'x64',
), ),
notifies = ['cr-fuchsia'], extra_notifies = ['cr-fuchsia'],
) )
ci.linux_builder( ci.linux_builder(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment