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

Side by Side Diff: tools/run-bisect-perf-regression.py

Issue 457793003: Fix performance trybots. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: typo Created 6 years, 4 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright (c) 2013 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """Run Performance Test Bisect Tool 6 """Run Performance Test Bisect Tool
7 7
8 This script is used by a trybot to run the src/tools/bisect-perf-regression.py 8 This script is used by a trybot to run the src/tools/bisect-perf-regression.py
9 script with the parameters specified in run-bisect-perf-regression.cfg. It will 9 script with the parameters specified in run-bisect-perf-regression.cfg. It will
10 check out a copy of the depot in a subdirectory 'bisect' of the working 10 check out a copy of the depot in a subdirectory 'bisect' of the working
(...skipping 252 matching lines...) Expand 10 before | Expand all | Expand 10 after
263 263
264 results_with_patch = b.RunPerformanceTestAndParseResults( 264 results_with_patch = b.RunPerformanceTestAndParseResults(
265 opts.command, opts.metric, reset_on_first_run=True, results_label='Patch') 265 opts.command, opts.metric, reset_on_first_run=True, results_label='Patch')
266 266
267 if results_with_patch[1]: 267 if results_with_patch[1]:
268 raise RuntimeError('Patched version failed to run performance test.') 268 raise RuntimeError('Patched version failed to run performance test.')
269 269
270 bisect_utils.OutputAnnotationStepClosed() 270 bisect_utils.OutputAnnotationStepClosed()
271 271
272 bisect_utils.OutputAnnotationStepStart('Reverting Patch') 272 bisect_utils.OutputAnnotationStepStart('Reverting Patch')
273 if bisect_utils.RunGClient(['revert']): 273 # TODO: When this is re-written to recipes, this should use bot_update's
274 raise RuntimeError('Failed to run gclient runhooks') 274 # revert mechanism to fully revert the client. But for now, since we know that
275 # the perf trybot currently only supports src/ and src/third_party/WebKit, we
276 # simply reset those two directories.
277 bisect_utils.CheckRunGit(['reset', '--hard'])
278 bisect_utils.CheckRunGit(['reset', '--hard'],
279 os.path.join('third_party', 'WebKit'))
275 bisect_utils.OutputAnnotationStepClosed() 280 bisect_utils.OutputAnnotationStepClosed()
276 281
277 bisect_utils.OutputAnnotationStepStart('Building Without Patch') 282 bisect_utils.OutputAnnotationStepStart('Building Without Patch')
278 283
279 if bisect_utils.RunGClient(['runhooks']): 284 if bisect_utils.RunGClient(['runhooks']):
280 raise RuntimeError('Failed to run gclient runhooks') 285 raise RuntimeError('Failed to run gclient runhooks')
281 286
282 if not b.BuildCurrentRevision('chromium'): 287 if not b.BuildCurrentRevision('chromium'):
283 raise RuntimeError('Unpatched version failed to build.') 288 raise RuntimeError('Unpatched version failed to build.')
284 289
(...skipping 266 matching lines...) Expand 10 before | Expand all | Expand 10 after
551 config, current_dir, opts.path_to_goma) 556 config, current_dir, opts.path_to_goma)
552 557
553 print ('Error: Could not load config file. Double check your changes to ' 558 print ('Error: Could not load config file. Double check your changes to '
554 'run-bisect-perf-regression.cfg or run-perf-test.cfg for syntax ' 559 'run-bisect-perf-regression.cfg or run-perf-test.cfg for syntax '
555 'errors.\n') 560 'errors.\n')
556 return 1 561 return 1
557 562
558 563
559 if __name__ == '__main__': 564 if __name__ == '__main__':
560 sys.exit(main()) 565 sys.exit(main())
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698