Add the builds tab

endpoints/api/build.py

@@ -148,12 +148,17 @@ class RepositoryBuildList(RepositoryParamResource):
   @require_repo_read
   @parse_args
   @query_param('limit', 'The maximum number of builds to return', type=int, default=5)
+  @query_param('since', 'Returns all builds since the given unix timecode', type=int, default=None)
   @nickname('getRepoBuilds')
   def get(self, args, namespace, repository):
     """ Get the list of repository builds. """
-    limit = args['limit']
-    builds = list(model.list_repository_builds(namespace, repository, limit))
+    limit = args.get('limit', 5)
+    since = args.get('since', None)
 
+    if since is not None:
+      since = datetime.datetime.utcfromtimestamp(since)
+
+    builds = model.list_repository_builds(namespace, repository, limit, since=since)
     can_write = ModifyRepositoryPermission(namespace, repository).can()
     return {
       'builds': [build_status_view(build, can_write) for build in builds]

endpoints/callbacks.py

@@ -247,6 +247,7 @@ def github_oauth_attach():
 
 
 @callback.route('/github/callback/trigger/<path:repository>', methods=['GET'])
+@callback.route('/github/callback/trigger/<path:repository>/__new', methods=['GET'])
 @route_show_if(features.GITHUB_BUILD)
 @require_session_login
 @parse_repository_name
@@ -260,9 +261,18 @@ def attach_github_build_trigger(namespace, repository):
       abort(404, message=msg)
 
     trigger = model.create_build_trigger(repo, 'github', token, current_user.db_user())
+
+    # TODO(jschorr): Remove once the new layout is in place.
     admin_path = '%s/%s/%s' % (namespace, repository, 'admin')
     full_url = '%s%s%s' % (url_for('web.repository', path=admin_path), '?tab=trigger&new_trigger=',
                            trigger.uuid)
+
+    if '__new' in request.url:
+      repo_path = '%s/%s' % (namespace, repository)
+      full_url = '%s%s%s' % (url_for('web.repository', path=repo_path), '?tab=builds&newtrigger=',
+                             trigger.uuid)
+
+
     logger.debug('Redirecting to full url: %s' % full_url)
     return redirect(full_url)
 

endpoints/common.py

@@ -19,6 +19,7 @@ from app import app, oauth_apps, dockerfile_build_queue, LoginWrappedDBUser
 
 from auth.permissions import QuayDeferredPermissionUser
 from auth import scopes
+from auth.auth_context import get_authenticated_user
 from endpoints.api.discovery import swagger_route_data
 from werkzeug.routing import BaseConverter
 from functools import wraps
@@ -225,7 +226,8 @@ def start_build(repository, dockerfile_id, tags, build_name, subdir, manual,
     'registry': host,
     'build_subdir': subdir,
     'trigger_metadata': trigger_metadata or {},
-    'is_manual': manual
+    'is_manual': manual,
+    'manual_user': get_authenticated_user().username if get_authenticated_user() else None
   }
 
   with app.config['DB_TRANSACTION_FACTORY'](db):
@@ -250,7 +252,8 @@ def start_build(repository, dockerfile_id, tags, build_name, subdir, manual,
     'repo': repository.name,
     'namespace': repository.namespace_user.username,
     'fileid': dockerfile_id,
-    'manual': manual,
+    'is_manual': manual,
+    'manual_user': get_authenticated_user().username if get_authenticated_user() else None
   }
 
   if trigger:
@@ -267,7 +270,8 @@ def start_build(repository, dockerfile_id, tags, build_name, subdir, manual,
     'build_id': build_request.uuid,
     'build_name': build_name,
     'docker_tags': tags,
-    'is_manual': manual
+    'is_manual': manual,
+    'manual_user': get_authenticated_user().username if get_authenticated_user() else None
   }
 
   if trigger: