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

appengine/findit/common/waterfall/buildbucket_client.py

Index: appengine/findit/common/waterfall/buildbucket_client.py
diff --git a/appengine/findit/common/waterfall/buildbucket_client.py b/appengine/findit/common/waterfall/buildbucket_client.py
index 48468968a46666686bac2cf839ddcfa320de9516..c5ff9f304f170731ef6f52942029c7b3acd08819 100644
--- a/appengine/findit/common/waterfall/buildbucket_client.py
+++ b/appengine/findit/common/waterfall/buildbucket_client.py
@@ -34,7 +34,9 @@ def _GetBucketName(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.
@@ -45,6 +47,8 @@ class TryJob(collections.namedtuple(
         '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'] = [
           {