Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(334)

Side by Side Diff: appengine/findit/waterfall/try_job_result_format.md

Issue 1591003002: [Findit] Modify tryjob pipelines to trigger try jobs for test failure. (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: rebase Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 # The format for final try-job result for compile failures is:
2 {
3 'result': [
4 ['rev1', 'passed'],
5 ['rev2', 'failed']
6 ],
7 'url': 'url',
8 'try_job_id': '1',
stgao 2016/01/28 19:33:52 This isn't included in the raw try job result, rig
chanli 2016/01/28 22:40:12 This is the final result saved in WfTryJob.compile
9 'culprit': {
10 'revision': 'rev2',
11 'commit_position': '2',
12 'review_url': 'url_2'
13 }
14 }
15
16 # The format for final try-job result for test failures is:
17 {
18 'result': {
19 'rev1': {
20 'a_test': {
21 'status': 'failed',
22 'valid': True,
23 'failures': ['a_test1']
24 },
25 'b_test': {
26 'status': 'failed',
27 'valid': True,
28 'failures': ['b_test1']
29 },
30 'c_test': {
31 'status': 'passed',
32 'valid': True
33 }
34 },
35 'rev2': {
36 'a_test': {
37 'status': 'failed',
38 'valid': True,
39 'failures': ['a_test1', 'a_test2']
40 },
41 'b_test': {
42 'status': 'passed',
43 'valid': True
44 },
45 'c_test': {
46 'status': 'failed',
47 'valid': True,
48 'failures': []
49 }
50 }
51 },
52 'url': 'url',
53 'try_job_id': '1',
54 'culprit': {
55 'a_test': {
56 'suspected_cls': {
57 'rev1': {
58 'revision': 'rev1',
59 'commit_position': '1',
60 'review_url': 'url_1'
61 },
62 'rev2': {
63 'revision': 'rev2',
64 'commit_position': '2',
65 'review_url': 'url_2'
66 }
67 },
68 'tests': {
69 'a_test1': {
70 'revision': 'rev1',
71 'commit_position': '1',
72 'review_url': 'url_1'
73 },
74 'a_test2': {
75 'revision': 'rev2',
76 'commit_position': '2',
77 'review_url': 'url_2'
78 }
79 }
80 },
81 'b_test': {
82 'suspected_cls': {
83 'rev1': {
84 'revision': 'rev1',
85 'commit_position': '1',
86 'review_url': 'url_1'
87 }
88 },
89 'tests': {
90 'b_test1': {
91 'revision': 'rev1',
92 'commit_position': '1',
93 'review_url': 'url_1'
94 }
95 }
96 },
97 'c_test': {
98 'suspected_cls': {
99 'rev2': {
100 'revision': 'rev2',
101 'commit_position': '2',
102 'review_url': 'url_2',
103 }
104 },
105 'tests': {}
106 }
107 }
108 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698