[GN]: Fix bug in roller script with try servers The API endpoint being used appears to be broken. There is a direct URL to the try_server_results data, so use that. BUG=none Review URL: https://codereview.chromium.org/1365393002 Cr-Original-Commit-Position: refs/heads/master@{#350906} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: e1bbf3218766ea727b229398aaae0df17fe3856b
diff --git a/tools/gn/bin/roll_gn.py b/tools/gn/bin/roll_gn.py index f02ba8b..a479bdc 100644 --- a/tools/gn/bin/roll_gn.py +++ b/tools/gn/bin/roll_gn.py
@@ -206,12 +206,11 @@ patchset = int(props['patchsets'][-1]) try: - patchset_data = json.loads(rpc_server.Send('/api/%d/%d' % - (issue, patchset))) + try_job_results = json.loads(rpc_server.Send( + '/api/%d/%d/try_job_results' % (issue, patchset))) except Exception as _e: raise - try_job_results = patchset_data['try_job_results'] if not try_job_results: print('No try jobs found on most recent patchset') return {} @@ -375,7 +374,7 @@ gn_changes = self.GetGNChanges() return ( - 'Roll DEPS %s..%s\n' + 'Roll buildtools %s..%s\n' '\n' ' In order to roll GN %s..%s (r%s:r%s) and pick up\n' ' the following changes:\n'