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

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 'report': {
4 'result': {
5 'rev1': 'passed',
6 'rev2': 'failed'
7 }
8 }
9 }
10
qyearsley 2016/02/03 18:55:14 It's very nice to have a document describing the f
chanli 2016/02/03 23:44:13 Yes. It does look much better now. Thank you : )
11
12 # The format for final try-job result for compile failures saved in WfTryJob.com pile_results is:
13 [
14 {
15 'report': {
16 'result': {
17 'rev1': 'passed',
18 'rev2': 'failed'
19 }
20 },
21 'url': 'url',
22 'try_job_id': '1',
23 'culprit': {
24 'revision': 'rev2',
25 'commit_position': '2',
26 'review_url': 'url_2'
27 }
28 },
29 ...
30 ]
31
32
33 # The format for recipe result for test failures is:
34 # TODO(chanli): update the format after the changes on test recipe.
35 'result': {
36 'rev1': {
37 'a_test': {
38 'status': 'failed',
39 'valid': True,
40 'failures': ['a_test1']
41 },
42 'b_test': {
43 'status': 'failed',
44 'valid': True,
45 'failures': ['b_test1']
46 },
47 'c_test': {
48 'status': 'passed',
49 'valid': True
50 }
51 },
52 'rev2': {
53 'a_test': {
54 'status': 'failed',
55 'valid': True,
56 'failures': ['a_test1', 'a_test2']
57 },
58 'b_test': {
59 'status': 'passed',
60 'valid': True
61 },
62 'c_test': {
63 'status': 'failed',
64 'valid': True,
65 'failures': []
66 }
67 }
68 }
69
70
71 # The format for final try-job result for test failures saved in WfTryJob.test_r esults is:
72 [
73 {
74 'result': {
75 'rev1': {
76 'a_test': {
77 'status': 'failed',
78 'valid': True,
79 'failures': ['a_test1']
80 },
81 'b_test': {
82 'status': 'failed',
83 'valid': True,
84 'failures': ['b_test1']
85 },
86 'c_test': {
87 'status': 'passed',
88 'valid': True
89 }
90 },
91 'rev2': {
92 'a_test': {
93 'status': 'failed',
94 'valid': True,
95 'failures': ['a_test1', 'a_test2']
96 },
97 'b_test': {
98 'status': 'passed',
99 'valid': True
100 },
101 'c_test': {
102 'status': 'failed',
103 'valid': True,
104 'failures': []
105 }
106 }
107 },
108 'url': 'url',
109 'try_job_id': '1',
110 'culprit': {
111 'a_test': {
112 'tests': {
113 'a_test1': {
114 'revision': 'rev1',
115 'commit_position': '1',
116 'review_url': 'url_1'
117 },
118 'a_test2': {
119 'revision': 'rev2',
120 'commit_position': '2',
121 'review_url': 'url_2'
122 }
123 }
124 },
125 'b_test': {
126 'tests': {
127 'b_test1': {
128 'revision': 'rev1',
129 'commit_position': '1',
130 'review_url': 'url_1'
131 }
132 }
133 },
134 'c_test': {
135 'revision': 'rev2',
136 'commit_position': '2',
137 'review_url': 'url_2',
138 'tests': {}
139 }
140 }
141 },
142 ...
143 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698