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

Side by Side Diff: infra/bots/recipes/upload_dm_results.py

Issue 2455523004: Fix recipes for Gerrit property rename (Closed)
Patch Set: Created 4 years, 1 month 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 # Recipe for uploading DM results. 6 # Recipe for uploading DM results.
7 7
8 8
9 DEPS = [ 9 DEPS = [
10 'build/file', 10 'build/file',
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
81 str(now.hour ).zfill(2), 81 str(now.hour ).zfill(2),
82 revision, 82 revision,
83 builder_name, 83 builder_name,
84 str(int(calendar.timegm(now.utctimetuple())))]) 84 str(int(calendar.timegm(now.utctimetuple())))])
85 85
86 # Trybot results are further siloed by issue/patchset. 86 # Trybot results are further siloed by issue/patchset.
87 issue = str(api.properties.get('issue', '')) 87 issue = str(api.properties.get('issue', ''))
88 patchset = str(api.properties.get('patchset', '')) 88 patchset = str(api.properties.get('patchset', ''))
89 if (api.properties.get('patch_storage', '') == 'gerrit' and 89 if (api.properties.get('patch_storage', '') == 'gerrit' and
90 api.properties.get('nobuildbot', '') != 'True'): 90 api.properties.get('nobuildbot', '') != 'True'):
91 issue = str(api.properties['event.change.number']) 91 issue = str(api.properties['patch_issue'])
92 patchset = str(api.properties['event.patchSet.ref']).split('/')[-1] 92 patchset = str(api.properties['patch_ref']).split('/')[-1]
93 if issue and patchset: 93 if issue and patchset:
94 summary_dest_path = '/'.join(( 94 summary_dest_path = '/'.join((
95 'trybot', summary_dest_path, issue, patchset)) 95 'trybot', summary_dest_path, issue, patchset))
96 96
97 summary_dest_path = '/'.join((GS_BUCKET, summary_dest_path)) 97 summary_dest_path = '/'.join((GS_BUCKET, summary_dest_path))
98 98
99 cp(api, 'JSON and logs', tmp_dir.join('*'), summary_dest_path, 99 cp(api, 'JSON and logs', tmp_dir.join('*'), summary_dest_path,
100 ['-z', 'json,log']) 100 ['-z', 'json,log'])
101 101
102 102
(...skipping 29 matching lines...) Expand all
132 builder = 'Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Debug-Trybot' 132 builder = 'Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Debug-Trybot'
133 yield ( 133 yield (
134 api.test('trybot') + 134 api.test('trybot') +
135 api.properties(buildername=builder, 135 api.properties(buildername=builder,
136 revision='abc123', 136 revision='abc123',
137 path_config='kitchen', 137 path_config='kitchen',
138 issue='12345', 138 issue='12345',
139 patchset='1002') 139 patchset='1002')
140 ) 140 )
141 141
142 gerrit_kwargs = {
143 'patch_storage': 'gerrit',
144 'repository': 'skia',
145 'event.patchSet.ref': 'refs/changes/00/2100/2',
146 'event.change.number': '2100',
147 }
148 yield ( 142 yield (
149 api.test('recipe_with_gerrit_patch') + 143 api.test('recipe_with_gerrit_patch') +
150 api.properties( 144 api.properties(
151 buildername=builder, 145 buildername=builder,
152 revision='abc123', 146 revision='abc123',
153 path_config='kitchen', 147 path_config='kitchen',
154 **gerrit_kwargs) 148 patch_storage='gerrit') +
149 api.properties.tryserver(
150 buildername=builder,
151 gerrit_project='skia',
152 gerrit_url='https://skia-review.googlesource.com/',
153 )
155 ) 154 )
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698