Merge branch master into bees

endpoints/api/trigger.py

@@ -15,7 +15,7 @@ from endpoints.api.build import (build_status_view, trigger_view, RepositoryBuil
 from endpoints.common import start_build
 from endpoints.trigger import (BuildTrigger as BuildTriggerBase, TriggerDeactivationException,
                                TriggerActivationException, EmptyRepositoryException,
-                               RepositoryReadException)
+                               RepositoryReadException, TriggerStartException)
 from data import model
 from auth.permissions import UserAdminPermission, AdministerOrganizationPermission, ReadRepositoryPermission
 from util.names import parse_robot_username
@@ -212,7 +212,7 @@ class BuildTriggerActivate(RepositoryParamResource):
                                           'write')
 
       try:
-        repository_path = '%s/%s' % (trigger.repository.namespace,
+        repository_path = '%s/%s' % (trigger.repository.namespace_user.username,
                                      trigger.repository.name)
         path = url_for('webhooks.build_trigger_webhook',
                        repository=repository_path, trigger_uuid=trigger.uuid)
@@ -385,9 +385,24 @@ class BuildTriggerAnalyze(RepositoryParamResource):
 @path_param('trigger_uuid', 'The UUID of the build trigger')
 class ActivateBuildTrigger(RepositoryParamResource):
   """ Custom verb to manually activate a build trigger. """
+  schemas = {
+    'RunParameters': {
+      'id': 'RunParameters',
+      'type': 'object',
+      'description': 'Optional run parameters for activating the build trigger',
+      'additional_properties': False,
+      'properties': {
+        'branch_name': {
+          'type': 'string',
+          'description': '(GitHub Only) If specified, the name of the GitHub branch to build.'
+        }
+      }
+    }
+  }
 
   @require_repo_admin
   @nickname('manuallyStartBuildTrigger')
+  @validate_json_request('RunParameters')
   def post(self, namespace, repository, trigger_uuid):
     """ Manually start a build from the specified trigger. """
     try:
@@ -400,14 +415,18 @@ class ActivateBuildTrigger(RepositoryParamResource):
     if not handler.is_active(config_dict):
       raise InvalidRequest('Trigger is not active.')
 
-    specs = handler.manual_start(trigger.auth_token, config_dict)
-    dockerfile_id, tags, name, subdir = specs
+    try:
+      run_parameters = request.get_json()
+      specs = handler.manual_start(trigger.auth_token, config_dict, run_parameters=run_parameters)
+      dockerfile_id, tags, name, subdir = specs
 
-    repo = model.get_repository(namespace, repository)
-    pull_robot_name = model.get_pull_robot_name(trigger)
+      repo = model.get_repository(namespace, repository)
+      pull_robot_name = model.get_pull_robot_name(trigger)
 
-    build_request = start_build(repo, dockerfile_id, tags, name, subdir, True,
-                                pull_robot_name=pull_robot_name)
+      build_request = start_build(repo, dockerfile_id, tags, name, subdir, True,
+                                  pull_robot_name=pull_robot_name)
+    except TriggerStartException as tse:
+      raise InvalidRequest(tse.message)
 
     resp = build_status_view(build_request, True)
     repo_string = '%s/%s' % (namespace, repository)
@@ -437,6 +456,36 @@ class TriggerBuildList(RepositoryParamResource):
     }
 
 
+
+@resource('/v1/repository/<repopath:repository>/trigger/<trigger_uuid>/fields/<field_name>')
+@internal_only
+class BuildTriggerFieldValues(RepositoryParamResource):
+  """ Custom verb to fetch a values list for a particular field name. """
+  @require_repo_admin
+  @nickname('listTriggerFieldValues')
+  def get(self, namespace, repository, trigger_uuid, field_name):
+    """ List the field values for a custom run field. """
+    try:
+      trigger = model.get_build_trigger(namespace, repository, trigger_uuid)
+    except model.InvalidBuildTriggerException:
+      raise NotFound()
+
+    user_permission = UserAdminPermission(trigger.connected_user.username)
+    if user_permission.can():
+      trigger_handler = BuildTriggerBase.get_trigger_for_service(trigger.service.name)
+      values = trigger_handler.list_field_values(trigger.auth_token, json.loads(trigger.config),
+                                                 field_name)
+
+      if values is None:
+        raise NotFound()
+        
+      return {
+        'values': values
+      }
+    else:
+      raise Unauthorized()
+
+
 @resource('/v1/repository/<repopath:repository>/trigger/<trigger_uuid>/sources')
 @path_param('repository', 'The full path of the repository. e.g. namespace/name')
 @path_param('trigger_uuid', 'The UUID of the build trigger')