Skip to content

Commit

Permalink
backfill_revlist does not use 'dry_run' and 'times'
Browse files Browse the repository at this point in the history
  • Loading branch information
adusca committed Jun 5, 2015
1 parent f6bac2b commit f72135d
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 12 deletions.
2 changes: 1 addition & 1 deletion mozci/mozci.py
Expand Up @@ -468,7 +468,7 @@ def trigger(builder, revision, files=[], dry_run=False, extra_properties=None):
extra_properties)


def backfill_revlist(buildername, revisions, times=1, dry_run=False):
def backfill_revlist(buildername, revisions):
"""
Find the last known good job for that buildername iterating through the list of revisions.
Expand Down
15 changes: 4 additions & 11 deletions mozci/scripts/trigger.py
Expand Up @@ -126,8 +126,8 @@ def sanitize_buildernames(buildernames):
return ret_value


def determine_revlist(repo_url, buildername, rev, back_revisions, delta,
from_rev, backfill, skips, times, max_revisions, dry_run):
def determine_revlist(repo_url, buildername, rev, back_revisions,
delta, from_rev, backfill, skips, max_revisions):
"""Determine which revisions we need to trigger."""
if back_revisions:
push_info = query_revision_info(repo_url, rev)
Expand Down Expand Up @@ -160,12 +160,7 @@ def determine_revlist(repo_url, buildername, rev, back_revisions, delta,
start_id=start_id,
end_id=end_id)

revlist = backfill_revlist(
buildername,
revlist,
times,
dry_run
)
revlist = backfill_revlist(buildername, revlist)

else:
revlist = [rev]
Expand Down Expand Up @@ -202,9 +197,7 @@ def main():
from_rev=options.from_rev,
backfill=options.backfill,
skips=options.skips,
times=options.times,
max_revisions=options.max_revisions,
dry_run=options.dry_run)
max_revisions=options.max_revisions)

try:
trigger_range(
Expand Down

0 comments on commit f72135d

Please sign in to comment.