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

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: . 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 recipe result for compile failures is:
2
3 {
4 'report': {
5 'result': {
6 'rev1': 'passed',
7 'rev2': 'failed'
8 }
9 }
10 }
11
12
13 The format for final try-job result for compile failures saved in WfTryJob.compi le_results is:
14
15 [
16 {
17 'report': {
18 'result': {
19 'rev1': 'passed',
20 'rev2': 'failed'
21 }
22 },
23 'url': 'url',
24 'try_job_id': '1',
25 'culprit': {
26 'revision': 'rev2',
27 'commit_position': '2',
28 'review_url': 'url_2'
29 }
30 },
31 ...
32 ]
33
34
35 The format for recipe result for test failures is:
36 TODO(chanli): update the format after the changes on test recipe.
37
38 'result': {
39 'rev1': {
40 'a_test': {
41 'status': 'failed',
42 'valid': True,
43 'failures': ['a_test1']
44 },
45 'b_test': {
46 'status': 'failed',
47 'valid': True,
48 'failures': ['b_test1']
49 },
50 'c_test': {
51 'status': 'passed',
52 'valid': True
53 }
54 },
55 'rev2': {
56 'a_test': {
57 'status': 'failed',
58 'valid': True,
59 'failures': ['a_test1', 'a_test2']
60 },
61 'b_test': {
62 'status': 'passed',
63 'valid': True
64 },
65 'c_test': {
66 'status': 'failed',
67 'valid': True,
68 'failures': []
69 }
70 }
71 }
72
73
74 The format for final try-job result for test failures saved in WfTryJob.test_res ults is:
75
76 [
77 {
78 'result': {
79 'rev1': {
80 'a_test': {
81 'status': 'failed',
82 'valid': True,
83 'failures': ['a_test1']
84 },
85 'b_test': {
86 'status': 'failed',
87 'valid': True,
88 'failures': ['b_test1']
89 },
90 'c_test': {
91 'status': 'passed',
92 'valid': True
93 }
94 },
95 'rev2': {
96 'a_test': {
97 'status': 'failed',
98 'valid': True,
99 'failures': ['a_test1', 'a_test2']
100 },
101 'b_test': {
102 'status': 'passed',
103 'valid': True
104 },
105 'c_test': {
106 'status': 'failed',
107 'valid': True,
108 'failures': []
109 }
110 }
111 },
112 'url': 'url',
113 'try_job_id': '1',
114 'culprit': {
115 'a_test': {
116 'tests': {
117 'a_test1': {
118 'revision': 'rev1',
119 'commit_position': '1',
120 'review_url': 'url_1'
121 },
122 'a_test2': {
123 'revision': 'rev2',
124 'commit_position': '2',
125 'review_url': 'url_2'
126 }
127 }
128 },
129 'b_test': {
130 'tests': {
131 'b_test1': {
132 'revision': 'rev1',
133 'commit_position': '1',
134 'review_url': 'url_1'
135 }
136 }
137 },
138 'c_test': {
139 'revision': 'rev2',
140 'commit_position': '2',
141 'review_url': 'url_2',
142 'tests': {}
143 }
144 }
145 },
146 ...
147 }
OLDNEW
« no previous file with comments | « appengine/findit/waterfall/try_job_pipeline.py ('k') | appengine/findit/waterfall/try_job_type.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698