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

Side by Side Diff: infra/bots/recipe_modules/vars/api.py

Issue 2263323002: Apply gerrit ref if it is a Gerrit patch (Closed) Base URL: https://skia.googlesource.com/skia@master
Patch Set: Cleanup Created 4 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
OLDNEW
1 # Copyright 2016 The Chromium Authors. All rights reserved. 1 # Copyright 2016 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 5
6 # pylint: disable=W0201 6 # pylint: disable=W0201
7 7
8 8
9 from recipe_engine import recipe_api 9 from recipe_engine import recipe_api
10 10
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
103 self.configuration = CONFIG_RELEASE 103 self.configuration = CONFIG_RELEASE
104 else: 104 else:
105 self.configuration = self.builder_cfg.get('configuration', CONFIG_DEBUG) 105 self.configuration = self.builder_cfg.get('configuration', CONFIG_DEBUG)
106 arch = (self.builder_cfg.get('arch') or self.builder_cfg.get('target_arch')) 106 arch = (self.builder_cfg.get('arch') or self.builder_cfg.get('target_arch'))
107 if ('Win' in self.builder_cfg.get('os', '') and arch == 'x86_64'): 107 if ('Win' in self.builder_cfg.get('os', '') and arch == 'x86_64'):
108 self.configuration += '_x64' 108 self.configuration += '_x64'
109 109
110 self.default_env.update({'SKIA_OUT': self.skia_out, 110 self.default_env.update({'SKIA_OUT': self.skia_out,
111 'BUILDTYPE': self.configuration}) 111 'BUILDTYPE': self.configuration})
112 self.is_trybot = self.builder_cfg['is_trybot'] 112 self.is_trybot = self.builder_cfg['is_trybot']
113 self.patch_storage = self.m.properties.get('patch_storage', '')
113 self.issue = None 114 self.issue = None
114 self.patchset = None 115 self.patchset = None
115 self.rietveld = None
116 if self.is_trybot: 116 if self.is_trybot:
117 self.issue = self.m.properties['issue'] 117 if self.patch_storage == 'gerrit':
118 self.patchset = self.m.properties['patchset'] 118 self.issue = self.m.properties['event.change.number']
119 self.rietveld = self.m.properties['rietveld'] 119 self.patchset = self.m.properties['event.patchSet.ref'].split('/')[-1]
120 else:
121 self.issue = self.m.properties['issue']
122 self.patchset = self.m.properties['patchset']
120 self.dm_dir = self.m.path.join( 123 self.dm_dir = self.m.path.join(
121 self.swarming_out_dir, 'dm') 124 self.swarming_out_dir, 'dm')
122 self.perf_data_dir = self.m.path.join(self.swarming_out_dir, 125 self.perf_data_dir = self.m.path.join(self.swarming_out_dir,
123 'perfdata', self.builder_name, 'data') 126 'perfdata', self.builder_name, 'data')
124 127
125 @property 128 @property
126 def upload_dm_results(self): 129 def upload_dm_results(self):
127 # TODO(borenet): Move this into the swarm_test recipe. 130 # TODO(borenet): Move this into the swarm_test recipe.
128 skip_upload_bots = [ 131 skip_upload_bots = [
129 'ASAN', 132 'ASAN',
130 'Coverage', 133 'Coverage',
131 'MSAN', 134 'MSAN',
132 'TSAN', 135 'TSAN',
133 'UBSAN', 136 'UBSAN',
134 'Valgrind', 137 'Valgrind',
135 ] 138 ]
136 upload_dm_results = True 139 upload_dm_results = True
137 for s in skip_upload_bots: 140 for s in skip_upload_bots:
138 if s in self.m.properties['buildername']: 141 if s in self.m.properties['buildername']:
139 upload_dm_results = False 142 upload_dm_results = False
140 break 143 break
141 return upload_dm_results 144 return upload_dm_results
142 145
143 @property 146 @property
144 def upload_perf_results(self): 147 def upload_perf_results(self):
145 # TODO(borenet): Move this into the swarm_perf recipe. 148 # TODO(borenet): Move this into the swarm_perf recipe.
146 return ('Release' in self.m.properties['buildername']) 149 return ('Release' in self.m.properties['buildername'])
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698