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

Side by Side Diff: scripts/slave/recipe_modules/v8/api.py

Issue 1403463002: V8 Buildbot: Pass isolate hashes from builder to tester. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 5 years, 2 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 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 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 argparse 5 import argparse
6 import collections 6 import collections
7 import datetime 7 import datetime
8 import math 8 import math
9 import re 9 import re
10 import urllib 10 import urllib
(...skipping 904 matching lines...) Expand 10 before | Expand all | Expand 10 after
915 ).stdout 915 ).stdout
916 916
917 def maybe_trigger(self, **additional_properties): 917 def maybe_trigger(self, **additional_properties):
918 triggers = self.bot_config.get('triggers') 918 triggers = self.bot_config.get('triggers')
919 if triggers: 919 if triggers:
920 properties = { 920 properties = {
921 'revision': self.revision, 921 'revision': self.revision,
922 'parent_got_revision': self.revision, 922 'parent_got_revision': self.revision,
923 'parent_got_revision_cp': self.revision_cp, 923 'parent_got_revision_cp': self.revision_cp,
924 } 924 }
925 isolated_tests = self.m.isolate.isolated_tests
926 if isolated_tests:
927 properties['isolated_tests'] = isolated_tests
925 properties.update(**additional_properties) 928 properties.update(**additional_properties)
926 self.m.trigger(*[{ 929 self.m.trigger(*[{
927 'builder_name': builder_name, 930 'builder_name': builder_name,
928 'properties': properties, 931 'properties': properties,
929 } for builder_name in triggers]) 932 } for builder_name in triggers])
930 933
931 def get_change_range(self): 934 def get_change_range(self):
932 url = '%sjson/builders/%s/builds/%s/source_stamp' % ( 935 url = '%sjson/builders/%s/builds/%s/source_stamp' % (
933 self.m.properties['buildbotURL'], 936 self.m.properties['buildbotURL'],
934 urllib.quote(self.m.properties['buildername']), 937 urllib.quote(self.m.properties['buildername']),
(...skipping 90 matching lines...) Expand 10 before | Expand all | Expand 10 after
1025 def report_culprits(self, culprit_range): 1028 def report_culprits(self, culprit_range):
1026 assert culprit_range 1029 assert culprit_range
1027 if len(culprit_range) > 1: 1030 if len(culprit_range) > 1:
1028 text = 'Suspecting multiple commits' 1031 text = 'Suspecting multiple commits'
1029 else: 1032 else:
1030 text = 'Suspecting %s' % culprit_range[0][:8] 1033 text = 'Suspecting %s' % culprit_range[0][:8]
1031 1034
1032 step_result = self.m.step(text, cmd=None) 1035 step_result = self.m.step(text, cmd=None)
1033 for culprit in culprit_range: 1036 for culprit in culprit_range:
1034 step_result.presentation.links[culprit[:8]] = COMMIT_TEMPLATE % culprit 1037 step_result.presentation.links[culprit[:8]] = COMMIT_TEMPLATE % culprit
OLDNEW
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/v8/builders.py » ('j') | scripts/slave/recipe_modules/v8/builders.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698