[Findit] Modify tryjob pipelines to trigger try jobs for test failure.

appengine/findit/waterfall/try_job_result_format.md

Index: appengine/findit/waterfall/try_job_result_format.md
diff --git a/appengine/findit/waterfall/try_job_result_format.md b/appengine/findit/waterfall/try_job_result_format.md
new file mode 100644
index 0000000000000000000000000000000000000000..4d199bf42208740e6a999fbefe246c9c05bde423
--- /dev/null
+++ b/appengine/findit/waterfall/try_job_result_format.md
@@ -0,0 +1,108 @@
+# The format for final try-job result for compile failures is:
+{
+    'result': [
+        ['rev1', 'passed'],
+        ['rev2', 'failed']
+    ],
+    'url': 'url',
+    'try_job_id': '1',

[stgao, 2016/01/28 19:33:52]

This isn't included in the raw try job result, right?

Maybe we'd better make it more clear whether this is the recipe result from the
try job, or the result we save in datastore for a try job.

I would suggest adding the recipe result format here too, because the recipes
are in a different code repo and the returned format there is not obvious.

[chanli, 2016/01/28 22:40:12]

This is the final result saved in WfTryJob.compile_results or
WfTryJob.test_results. I'll add the recipe result format

+    'culprit': {
+        'revision': 'rev2',
+        'commit_position': '2',
+        'review_url': 'url_2'
+    }
+}
+
+# The format for final try-job result for test failures is:
+{
+    'result': {
+        'rev1': {
+            'a_test': {
+                'status': 'failed',
+                'valid': True,
+                'failures': ['a_test1']
+            },
+            'b_test': {
+                'status': 'failed',
+                'valid': True,
+                'failures': ['b_test1']
+            },
+            'c_test': {
+                'status': 'passed',
+                'valid': True
+            }
+        },
+        'rev2': {
+            'a_test': {
+                'status': 'failed',
+                'valid': True,
+                'failures': ['a_test1', 'a_test2']
+            },
+            'b_test': {
+                'status': 'passed',
+                'valid': True
+            },
+            'c_test': {
+                'status': 'failed',
+                'valid': True,
+                'failures': []
+            }
+        }
+    },
+    'url': 'url',
+    'try_job_id': '1',
+    'culprit': {
+        'a_test': {
+            'suspected_cls': {
+                'rev1': {
+                    'revision': 'rev1',
+                    'commit_position': '1',
+                    'review_url': 'url_1'
+                },
+                'rev2': {
+                    'revision': 'rev2',
+                    'commit_position': '2',
+                    'review_url': 'url_2'
+                }
+            },
+            'tests': {
+                'a_test1': {
+                  'revision': 'rev1',
+                  'commit_position': '1',
+                  'review_url': 'url_1'
+                },
+                'a_test2': {
+                  'revision': 'rev2',
+                  'commit_position': '2',
+                  'review_url': 'url_2'
+                }
+            }
+        },
+        'b_test': {
+            'suspected_cls': {
+                'rev1': {
+                    'revision': 'rev1',
+                    'commit_position': '1',
+                    'review_url': 'url_1'
+                }
+            },
+            'tests': {
+                'b_test1': {
+                  'revision': 'rev1',
+                  'commit_position': '1',
+                  'review_url': 'url_1'
+                }
+            }
+        },
+        'c_test': {
+            'suspected_cls': {
+                'rev2': {
+                  'revision': 'rev2',
+                  'commit_position': '2',
+                  'review_url': 'url_2',
+                }
+            },
+            'tests': {}
+        }
+    }
+}