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

Unified Diff: dashboard/dashboard/bug_details.py

Issue 2706813003: Add new endpoint to get bug details as JSON. (Closed)
Patch Set: addressed review comments Created 3 years, 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « dashboard/dashboard/benchmark_health_report.py ('k') | dashboard/dashboard/bug_details_test.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: dashboard/dashboard/bug_details.py
diff --git a/dashboard/dashboard/bug_details.py b/dashboard/dashboard/bug_details.py
new file mode 100644
index 0000000000000000000000000000000000000000..eebe8042a6649d675d767803ea5da8a16babc289
--- /dev/null
+++ b/dashboard/dashboard/bug_details.py
@@ -0,0 +1,81 @@
+# Copyright 2017 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+"""Provides an endpoint for getting details about a sheriffed bug."""
+
+import json
+import re
+
+from dashboard import oauth2_decorator
+from dashboard.common import request_handler
+from dashboard.models import try_job
+from dashboard.services import issue_tracker_service
+
+
+BUGDROID = 'bugdroid1@chromium.org'
+REVIEW_RE = r'(Review-Url|Reviewed-on): (https?:\/\/[\/\.\w\d]+)'
+
+
+class BugDetailsHandler(request_handler.RequestHandler):
+ """Gets details about a sheriffed bug."""
+
+ def post(self):
+ """POST is the same as GET for this endpoint."""
+ self.get()
+
+ @oauth2_decorator.DECORATOR.oauth_required
+ def get(self):
+ """Response handler to get details about a specific bug.
+
+ Request parameters:
+ bug_id: Bug ID number, as a string
+ """
+ bug_id = int(self.request.get('bug_id'), 0)
+ if bug_id <= 0:
+ self.ReportError('Invalid or no bug id specified.')
+ return
+
+ bug_details = self._GetDetailsFromMonorail(bug_id)
+ bug_details['review_urls'] = self._GetLinkedRevisions(
+ bug_details['comments'])
+ bug_details['bisects'] = self._GetBisectsForBug(bug_id)
+ self.response.out.write(json.dumps(bug_details))
+
+ def _GetDetailsFromMonorail(self, bug_id):
+ http = oauth2_decorator.DECORATOR.http()
+ issue_tracker = issue_tracker_service.IssueTrackerService(http)
+ bug_details = issue_tracker.GetIssue(bug_id)
+ if not bug_details:
+ return {'error': 'Failed to get bug details from monorail API'}
+ bug_details['comments'] = issue_tracker.GetIssueComments(bug_id)
+ owner = None
+ if bug_details.get('owner'):
+ owner = bug_details.get('owner').get('name')
+ return {
+ 'comments': bug_details['comments'],
+ 'owner': owner,
+ 'published': bug_details['published'],
+ 'state': bug_details['state'],
+ 'status': bug_details['status'],
+ 'summary': bug_details['summary'],
+ }
+
+ def _GetLinkedRevisions(self, comments):
+ """Parses the comments for commits linked by bugdroid."""
+ review_urls = []
+ bugdroid_comments = [c for c in comments if c['author'] == BUGDROID]
+ for comment in bugdroid_comments:
+ m = re.search(REVIEW_RE, comment['content'])
+ if m:
+ review_urls.append(m.group(2))
+ return review_urls
+
+ def _GetBisectsForBug(self, bug_id):
+ bisects = try_job.TryJob.query(try_job.TryJob.bug_id == bug_id).fetch()
+ return [{
+ 'status': b.status,
+ 'bot': b.bot,
+ 'buildbucket_link': '/buildbucket_job_status/%s' % b.buildbucket_job_id,
+ 'metric': b.results_data.get('metric'),
+ } for b in bisects]
« no previous file with comments | « dashboard/dashboard/benchmark_health_report.py ('k') | dashboard/dashboard/bug_details_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698