OLD | NEW |
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 """Utility functions to communicate with Rietveld.""" | 5 """Utility functions to communicate with Rietveld.""" |
6 | 6 |
7 import collections | 7 import collections |
8 import json | 8 import json |
9 import logging | 9 import logging |
10 import urllib2 | 10 import urllib2 |
(...skipping 23 matching lines...) Expand all Loading... |
34 url = _patchset_url(issue_number, patchset_number) | 34 url = _patchset_url(issue_number, patchset_number) |
35 else: | 35 else: |
36 url = _latest_patchset_url(issue_number, web) | 36 url = _latest_patchset_url(issue_number, web) |
37 patchset_data = _get_json(url, web) | 37 patchset_data = _get_json(url, web) |
38 except (urllib2.URLError, ValueError): | 38 except (urllib2.URLError, ValueError): |
39 return [] | 39 return [] |
40 jobs = [] | 40 jobs = [] |
41 for job in patchset_data['try_job_results']: | 41 for job in patchset_data['try_job_results']: |
42 if job['builder'] not in builder_names: | 42 if job['builder'] not in builder_names: |
43 continue | 43 continue |
| 44 # The master name may be prefixed with "master.", or possibly not; |
| 45 # We want to normalize master name by stripping this prefix. |
| 46 # See http://crbug.com/624545. |
| 47 master_name = job['master'] |
| 48 if master_name.startswith('master.'): |
| 49 master_name = master_name[len('master.'):] |
44 jobs.append(TryJob( | 50 jobs.append(TryJob( |
45 builder_name=job['builder'], | 51 builder_name=job['builder'], |
46 master_name=job['master'], | 52 master_name=master_name, |
47 build_number=job['buildnumber'])) | 53 build_number=job['buildnumber'])) |
48 return jobs | 54 return jobs |
49 | 55 |
50 | 56 |
51 def _latest_patchset_url(issue_number, web): | 57 def _latest_patchset_url(issue_number, web): |
52 issue_data = _get_json(_issue_url(issue_number), web) | 58 issue_data = _get_json(_issue_url(issue_number), web) |
53 latest_patchset_number = issue_data["patchsets"][-1] | 59 latest_patchset_number = issue_data["patchsets"][-1] |
54 return _patchset_url(issue_number, latest_patchset_number) | 60 return _patchset_url(issue_number, latest_patchset_number) |
55 | 61 |
56 | 62 |
(...skipping 24 matching lines...) Expand all Loading... |
81 return '%s/%s' % (_issue_url(issue_number), patchset_number) | 87 return '%s/%s' % (_issue_url(issue_number), patchset_number) |
82 | 88 |
83 | 89 |
84 def get_latest_try_job_results(issue_number, web): | 90 def get_latest_try_job_results(issue_number, web): |
85 url = _latest_patchset_url(issue_number, web) | 91 url = _latest_patchset_url(issue_number, web) |
86 patchset_data = _get_json(url, web) | 92 patchset_data = _get_json(url, web) |
87 results = {} | 93 results = {} |
88 for job in patchset_data['try_job_results']: | 94 for job in patchset_data['try_job_results']: |
89 results[job['builder']] = job['result'] | 95 results[job['builder']] = job['result'] |
90 return results | 96 return results |
OLD | NEW |