[Findit] Send targeted_tests to try-job in build_parameter instend of build_property (Findit side).

appengine/findit/common/waterfall/buildbucket_client.py

 # Copyright 2015 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.
 
 """"Serves as a client for selected APIs in Buildbucket."""
 
 import collections
 import json
 import logging
 
@@ skipping 16 matching lines @@
   name, while Findit uses shortened master name (tryserver.chromium.linux).
   """
   prefix = 'master.'
   if master_name.startswith(prefix):
     return master_name
   return '%s%s' % (prefix, master_name)
 
 
 class TryJob(collections.namedtuple(
     'TryJobNamedTuple',
-    ('master_name', 'builder_name', 'revision', 'properties', 'tags'))):
+    ('master_name', 'builder_name', 'revision', 'properties', 'tags',
+     'additional_parameters'))

[stgao, 2016/05/20 18:34:36]

nit: "build_parameters" to be more clear and to match with for build properties?

[chanli, 2016/05/20 20:52:55]

the entire 'parameters_json' represents actually build parameters(See
https://docs.google.com/document/d/10BH6oSq_UEtdSdhrguEuV4hx-JxY7jzFcgs1OCHcn...).
I can change it to additional_build_parameters if that's more clear.

+):
   """Represents a try-job to be triggered through Buildbucket.
 
   Tag for "user_agent" should not be set, as it will be added automatically.
   """
 
   def ToBuildbucketRequest(self):
     parameters_json = {
         'builder_name': self.builder_name,
         'properties': self.properties,
     }
+    if self.additional_parameters:
+      parameters_json['additional_parameters'] = self.additional_parameters
     if self.revision:
       parameters_json['changes'] = [
           {
               'author': {
                   'email': _ROLE_EMAIL,
               },
               'revision': self.revision,
           },
       ]
 
@@ skipping 126 matching lines @@
     else:
       error_content = {
           'error': {
               'reason': status_code,
               'message': content
           }
       }
       json_results.append(error_content)
 
   return _ConvertFuturesToResults(json_results)