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

Unified Diff: appengine/chromium_try_flakes/issue_tracker/issue_tracker_api.py

Issue 2134193002: Remove codesite bug tracker support from builder_alerts and chromium-try-flakes (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Update tests Created 4 years, 5 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
Index: appengine/chromium_try_flakes/issue_tracker/issue_tracker_api.py
diff --git a/appengine/chromium_try_flakes/issue_tracker/issue_tracker_api.py b/appengine/chromium_try_flakes/issue_tracker/issue_tracker_api.py
index 6182780a4bfbf764f831510b13de94e5d436a0e7..79f339aa5eb3b55429e48e0b2d90819bc2cf7d48 100644
--- a/appengine/chromium_try_flakes/issue_tracker/issue_tracker_api.py
+++ b/appengine/chromium_try_flakes/issue_tracker/issue_tracker_api.py
@@ -52,22 +52,14 @@ class IssueTrackerAPI(object): # pragma: no cover
CAN_ALL = 'all'
"""A wrapper around the issue tracker api."""
- def __init__(self, project_name, use_monorail):
+ def __init__(self, project_name):
self.project_name = project_name
- self.use_monorail = use_monorail
-
- if use_monorail:
- self.client = _buildClient(
- 'monorail', 'v1',
- _createHttpObject('https://www.googleapis.com/auth/userinfo.email'),
- 'https://monorail-prod.appspot.com/_ah/api/discovery/v1/apis/{api}/'
- '{apiVersion}/rest')
- else:
- self.client = _buildClient(
- 'projecthosting', 'v2',
- _createHttpObject('https://www.googleapis.com/auth/projecthosting'),
- 'https://www.googleapis.com/discovery/v1/apis/{api}/{apiVersion}/'
- 'rest')
+
+ self.client = _buildClient(
+ 'monorail', 'v1',
+ _createHttpObject('https://www.googleapis.com/auth/userinfo.email'),
+ 'https://monorail-prod.appspot.com/_ah/api/discovery/v1/apis/{api}/'
+ '{apiVersion}/rest')
def _retry_api_call(self, request, num_retries=5):
retries = 0
@@ -114,12 +106,7 @@ class IssueTrackerAPI(object): # pragma: no cover
if 'status' in issue.changed:
updates['status'] = issue.status
if 'owner' in issue.changed:
- # workaround for existing bug:
- # https://code.google.com/a/google.com/p/codesite/issues/detail?id=115
- if self.use_monorail:
- updates['owner'] = issue.owner
- else:
- updates['owner'] = issue.owner if issue.owner else '----'
+ updates['owner'] = issue.owner
if 'blocked_on' in issue.changed:
updates['blockedOn'] = issue.blocked_on
if issue.labels.isChanged():
« no previous file with comments | « appengine/chromium_try_flakes/handlers/test/flake_issues_test.py ('k') | infra/services/builder_alerts/__main__.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698