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

Side by Side Diff: scripts/slave/recipe_modules/auto_bisect/bisector.py

Issue 2275563003: Fix android recipe build upload names (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Rebase Created 4 years, 3 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
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/auto_bisect/example.expected/android_arm64_bisector.json » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 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 json 5 import json
6 import re 6 import re
7 import time 7 import time
8 import urllib 8 import urllib
9 9
10 from . import config_validation 10 from . import config_validation
(...skipping 835 matching lines...) Expand 10 before | Expand all | Expand 10 after
846 bot_name = self.get_perf_tester_name() 846 bot_name = self.get_perf_tester_name()
847 if 'win' in bot_name: 847 if 'win' in bot_name:
848 if any(b in bot_name for b in ['x64', 'gpu']): 848 if any(b in bot_name for b in ['x64', 'gpu']):
849 return 'gs://chrome-perf/Win x64 Builder/full-build-win32_' 849 return 'gs://chrome-perf/Win x64 Builder/full-build-win32_'
850 return 'gs://chrome-perf/Win Builder/full-build-win32_' 850 return 'gs://chrome-perf/Win Builder/full-build-win32_'
851 851
852 # TODO(prasadv): Refactor this code to remove hard coded values and use 852 # TODO(prasadv): Refactor this code to remove hard coded values and use
853 # target_bit from the bot config. crbug.com/640287 853 # target_bit from the bot config. crbug.com/640287
854 if 'android' in bot_name: 854 if 'android' in bot_name:
855 if any(b in bot_name for b in ['arm64', 'nexus9', 'nexus5X']): 855 if any(b in bot_name for b in ['arm64', 'nexus9', 'nexus5X']):
856 return 'gs://chrome-perf/android_perf_rel_arm64/full-build-linux_' 856 return 'gs://chrome-perf/Android arm64 Builder/full-build-linux_'
857 return 'gs://chrome-perf/android_perf_rel/full-build-linux_' 857 return 'gs://chrome-perf/Android Builder/full-build-linux_'
858 858
859 if 'mac' in bot_name: 859 if 'mac' in bot_name:
860 return 'gs://chrome-perf/Mac Builder/full-build-mac_' 860 return 'gs://chrome-perf/Mac Builder/full-build-mac_'
861 861
862 return 'gs://chrome-perf/Linux Builder/full-build-linux_' 862 return 'gs://chrome-perf/Linux Builder/full-build-linux_'
863 863
864 def ensure_sync_master_branch(self): 864 def ensure_sync_master_branch(self):
865 """Make sure the local master is in sync with the fetched origin/master. 865 """Make sure the local master is in sync with the fetched origin/master.
866 866
867 We have seen on several occasions that the local master branch gets reset 867 We have seen on several occasions that the local master branch gets reset
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
962 }) 962 })
963 return revision_rows 963 return revision_rows
964 964
965 def _get_build_url(self): 965 def _get_build_url(self):
966 properties = self.api.m.properties 966 properties = self.api.m.properties
967 bot_url = properties.get('buildbotURL', 967 bot_url = properties.get('buildbotURL',
968 'http://build.chromium.org/p/chromium/') 968 'http://build.chromium.org/p/chromium/')
969 builder_name = urllib.quote(properties.get('buildername', '')) 969 builder_name = urllib.quote(properties.get('buildername', ''))
970 builder_number = str(properties.get('buildnumber', '')) 970 builder_number = str(properties.get('buildnumber', ''))
971 return '%sbuilders/%s/builds/%s' % (bot_url, builder_name, builder_number) 971 return '%sbuilders/%s/builds/%s' % (bot_url, builder_name, builder_number)
OLDNEW
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/auto_bisect/example.expected/android_arm64_bisector.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698