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

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

Powered by Google App Engine
This is Rietveld 408576698