Skip to content

Commit 39c40e0

Browse files
author
bors-servo
authored
Auto merge of #124 - tomprince:limited-try-statuses, r=jdm
Allow a limited set of build statuses for try builds. Rust has started using try builds to get artifacts for running [cargobomb](https://github.com/brson/cargobomb]). But try builds only trigger travis and not appveyor, so it things they timeout when the in fact succeed. This allows configuring individual statuses to be skipped for try builds. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/homu/124) <!-- Reviewable:end -->
2 parents 62f0c9e + b8c5817 commit 39c40e0

File tree

2 files changed

+6
-0
lines changed

2 files changed

+6
-0
lines changed

cfg.sample.toml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,9 @@ secret = ""
112112
## String label set by status updates
113113
#context = ""
114114
#
115+
## Boolean which indicates whether the builder is included in try builds (defaults to true)
116+
#try = false
117+
#
115118
## Equivalent context to look for on the PR itself if checking whether the
116119
## build should be exempted. If omitted, looks for the same context. This is
117120
## only used if status_based_exemption is true.

homu/main.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1084,6 +1084,9 @@ def start_build(state, repo_cfgs, buildbot_slots, logger, db, git_cfg):
10841084
for key, value in repo_cfg['status'].items():
10851085
context = value.get('context')
10861086
if context is not None:
1087+
if state.try_ and not value.get('try', True):
1088+
# Skip this builder for tries.
1089+
continue
10871090
builders += ['status-' + key]
10881091
# We have an optional fast path if the Travis test passed
10891092
# for a given commit and master is unchanged, we can do

0 commit comments

Comments
 (0)