| OLD | NEW |
| 1 # Copyright 2012 The Chromium Authors. All rights reserved. | 1 # Copyright 2012 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 """Presubmit script for changes affecting tools/perf/. | 5 """Presubmit script for changes affecting tools/perf/. |
| 6 | 6 |
| 7 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts | 7 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts |
| 8 for more details about the presubmit API built into depot_tools. | 8 for more details about the presubmit API built into depot_tools. |
| 9 """ | 9 """ |
| 10 | 10 |
| (...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 114 | 114 |
| 115 This hook adds extra try bots list to the CL description in order to run | 115 This hook adds extra try bots list to the CL description in order to run |
| 116 Telemetry benchmarks on Perf trybots in addition to CQ trybots if the CL | 116 Telemetry benchmarks on Perf trybots in addition to CQ trybots if the CL |
| 117 contains any changes to Telemetry benchmarks. | 117 contains any changes to Telemetry benchmarks. |
| 118 """ | 118 """ |
| 119 benchmarks_modified = _AreBenchmarksModified(change) | 119 benchmarks_modified = _AreBenchmarksModified(change) |
| 120 rietveld_obj = cl.RpcServer() | 120 rietveld_obj = cl.RpcServer() |
| 121 issue = cl.issue | 121 issue = cl.issue |
| 122 original_description = rietveld_obj.get_description(issue) | 122 original_description = rietveld_obj.get_description(issue) |
| 123 if not benchmarks_modified or re.search( | 123 if not benchmarks_modified or re.search( |
| 124 r'^CQ_EXTRA_TRYBOTS=.*', original_description, re.M | re.I): | 124 r'^CQ_INCLUDE_TRYBOTS=.*', original_description, re.M | re.I): |
| 125 return [] | 125 return [] |
| 126 | 126 |
| 127 results = [] | 127 results = [] |
| 128 bots = [ | 128 bots = [ |
| 129 'android_s5_perf_cq', | 129 'android_s5_perf_cq', |
| 130 'mac_retina_perf_cq', | 130 'mac_retina_perf_cq', |
| 131 # TODO(prasadv): Uncomment this once crbug.com/601699 is fixed. | 131 # TODO(prasadv): Uncomment this once crbug.com/601699 is fixed. |
| 132 # 'linux_perf_cq' | 132 # 'linux_perf_cq' |
| 133 ] | 133 ] |
| 134 bots = ['tryserver.chromium.perf:%s' % s for s in bots] | 134 bots = ['tryserver.chromium.perf:%s' % s for s in bots] |
| 135 bots_string = ';'.join(bots) | 135 bots_string = ';'.join(bots) |
| 136 description = original_description | 136 description = original_description |
| 137 description += '\nCQ_EXTRA_TRYBOTS=%s' % bots_string | 137 description += '\nCQ_INCLUDE_TRYBOTS=%s' % bots_string |
| 138 results.append(output_api.PresubmitNotifyResult( | 138 results.append(output_api.PresubmitNotifyResult( |
| 139 'Automatically added Perf trybots to run Telemetry benchmarks on CQ.')) | 139 'Automatically added Perf trybots to run Telemetry benchmarks on CQ.')) |
| 140 | 140 |
| 141 if description != original_description: | 141 if description != original_description: |
| 142 rietveld_obj.update_description(issue, description) | 142 rietveld_obj.update_description(issue, description) |
| 143 | 143 |
| 144 return results | 144 return results |
| OLD | NEW |