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

Side by Side Diff: scripts/slave/recipe_modules/auto_bisect/example.py

Issue 2282563004: Do not exclude the last revision in the range returned by gitiles. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: typo Created 4 years, 3 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 unified diff | Download patch
OLDNEW
1 # Copyright 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import json 5 import json
6 6
7 DEPS = [ 7 DEPS = [
8 'auto_bisect', 8 'auto_bisect',
9 'chromium_tests', 9 'chromium_tests',
10 'recipe_engine/json', 10 'recipe_engine/json',
(...skipping 363 matching lines...) Expand 10 before | Expand all | Expand 10 after
374 basic_test += api.properties(bisect_config=_get_config(extra_config)) 374 basic_test += api.properties(bisect_config=_get_config(extra_config))
375 basic_test += api.properties( 375 basic_test += api.properties(
376 buildbotURL= 'https://build.chromium.org/p/tryserver.chromium.perf') 376 buildbotURL= 'https://build.chromium.org/p/tryserver.chromium.perf')
377 return basic_test 377 return basic_test
378 378
379 379
380 def _get_revision_range_step_data(api, range_data): 380 def _get_revision_range_step_data(api, range_data):
381 """Adds canned output for fetch_intervening_revisions.py.""" 381 """Adds canned output for fetch_intervening_revisions.py."""
382 min_rev = range_data[0]['hash'] 382 min_rev = range_data[0]['hash']
383 max_rev = range_data[-1]['hash'] 383 max_rev = range_data[-1]['hash']
384 output = [[r['hash'], 'ignored'] for r in range_data[1:-1]] 384 output = [[r['hash'], 'ignored'] for r in range_data[1:]]
385 step_name = ('Expanding revision range.for revisions %s:%s' % 385 step_name = ('Expanding revision range.for revisions %s:%s' %
386 (min_rev, max_rev)) 386 (min_rev, max_rev))
387 return api.step_data(step_name, stdout=api.json.output(output)) 387 return api.step_data(step_name, stdout=api.json.output(output))
388 388
389 389
390 def _get_config(params=None): 390 def _get_config(params=None):
391 """Returns a sample bisect config dict with some fields overridden.""" 391 """Returns a sample bisect config dict with some fields overridden."""
392 example_config = { 392 example_config = {
393 'test_type': 'perf', 393 'test_type': 'perf',
394 'command': ( 394 'command': (
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
457 yield api.step_data(step_name, stdout=api.raw_io.output(file_hash)) 457 yield api.step_data(step_name, stdout=api.raw_io.output(file_hash))
458 step_name = 'Expanding revision range for revision %s on depot %s' 458 step_name = 'Expanding revision range for revision %s on depot %s'
459 step_name %= (interval[-1], depot_name) 459 step_name %= (interval[-1], depot_name)
460 stdout = api.json.output([(r, 0) for r in interval[:-1]]) 460 stdout = api.json.output([(r, 0) for r in interval[:-1]])
461 yield api.step_data(step_name, stdout=stdout) 461 yield api.step_data(step_name, stdout=stdout)
462 462
463 if 'cl_info' in revision_data: 463 if 'cl_info' in revision_data:
464 step_name = 'Reading culprit cl information.' 464 step_name = 'Reading culprit cl information.'
465 stdout = api.json.output(revision_data['cl_info']) 465 stdout = api.json.output(revision_data['cl_info'])
466 yield api.step_data(step_name, stdout=stdout) 466 yield api.step_data(step_name, stdout=stdout)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698