blob: b585feabefebfe6a186cb63964931bf5f18f923e [file] [log] [blame]
James E. Blairb0fcae42012-07-17 11:12:10 -07001#!/usr/bin/env python
2
3# Copyright 2012 Hewlett-Packard Development Company, L.P.
4#
5# Licensed under the Apache License, Version 2.0 (the "License"); you may
6# not use this file except in compliance with the License. You may obtain
7# a copy of the License at
8#
9# http://www.apache.org/licenses/LICENSE-2.0
10#
11# Unless required by applicable law or agreed to in writing, software
12# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
13# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
14# License for the specific language governing permissions and limitations
15# under the License.
16
James E. Blairb0fcae42012-07-17 11:12:10 -070017import json
Monty Taylorbc758832013-06-17 17:22:42 -040018import logging
19import os
James E. Blairb0fcae42012-07-17 11:12:10 -070020import re
James E. Blair4886cc12012-07-18 15:39:41 -070021import shutil
Monty Taylorbc758832013-06-17 17:22:42 -040022import time
James E. Blair1843a552013-07-03 14:19:52 -070023import urllib
Monty Taylorbc758832013-06-17 17:22:42 -040024import urllib2
Maru Newby3fe5f852015-01-13 04:22:14 +000025import yaml
Monty Taylorbc758832013-06-17 17:22:42 -040026
James E. Blair4886cc12012-07-18 15:39:41 -070027import git
Monty Taylorbc758832013-06-17 17:22:42 -040028import testtools
James E. Blairb0fcae42012-07-17 11:12:10 -070029
Maru Newby3fe5f852015-01-13 04:22:14 +000030import zuul.change_matcher
James E. Blairb0fcae42012-07-17 11:12:10 -070031import zuul.scheduler
James E. Blairad28e912013-11-27 10:43:22 -080032import zuul.rpcclient
Joshua Hesketh1879cf72013-08-19 14:13:15 +100033import zuul.reporter.gerrit
Joshua Hesketh5fea8672013-08-19 17:32:01 +100034import zuul.reporter.smtp
James E. Blairb0fcae42012-07-17 11:12:10 -070035
Maru Newby3fe5f852015-01-13 04:22:14 +000036from tests.base import (
37 BaseTestCase,
38 ZuulTestCase,
39 repack_repo,
40)
James E. Blairb0fcae42012-07-17 11:12:10 -070041
James E. Blair1f4c2bb2013-04-26 08:40:46 -070042logging.basicConfig(level=logging.DEBUG,
43 format='%(asctime)s %(name)-32s '
44 '%(levelname)-8s %(message)s')
James E. Blairb0fcae42012-07-17 11:12:10 -070045
46
Maru Newby3fe5f852015-01-13 04:22:14 +000047class TestSchedulerConfigParsing(BaseTestCase):
48
49 def test_parse_skip_if(self):
50 job_yaml = """
51jobs:
52 - name: job_name
53 skip-if:
54 - project: ^project_name$
55 branch: ^stable/icehouse$
56 all-files-match-any:
57 - ^filename$
58 - project: ^project2_name$
59 all-files-match-any:
60 - ^filename2$
61 """.strip()
62 data = yaml.load(job_yaml)
63 config_job = data.get('jobs')[0]
Joshua Hesketh352264b2015-08-11 23:42:08 +100064 sched = zuul.scheduler.Scheduler({})
Maru Newby3fe5f852015-01-13 04:22:14 +000065 cm = zuul.change_matcher
66 expected = cm.MatchAny([
67 cm.MatchAll([
68 cm.ProjectMatcher('^project_name$'),
69 cm.BranchMatcher('^stable/icehouse$'),
70 cm.MatchAllFiles([cm.FileMatcher('^filename$')]),
71 ]),
72 cm.MatchAll([
73 cm.ProjectMatcher('^project2_name$'),
74 cm.MatchAllFiles([cm.FileMatcher('^filename2$')]),
75 ]),
76 ])
77 matcher = sched._parseSkipIf(config_job)
78 self.assertEqual(expected, matcher)
79
80
Clark Boylanb640e052014-04-03 16:41:46 -070081class TestScheduler(ZuulTestCase):
Antoine Mussobd86a312014-01-08 14:51:33 +010082
James E. Blairb0fcae42012-07-17 11:12:10 -070083 def test_jobs_launched(self):
84 "Test that jobs are launched and a change is merged"
James E. Blair1f4c2bb2013-04-26 08:40:46 -070085
James E. Blairb0fcae42012-07-17 11:12:10 -070086 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
James E. Blair8c803f82012-07-31 16:25:42 -070087 A.addApproval('CRVW', 2)
James E. Blairb0fcae42012-07-17 11:12:10 -070088 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
89 self.waitUntilSettled()
Monty Taylor98f0f3e2013-07-06 16:02:31 -040090 self.assertEqual(self.getJobFromHistory('project-merge').result,
91 'SUCCESS')
92 self.assertEqual(self.getJobFromHistory('project-test1').result,
93 'SUCCESS')
94 self.assertEqual(self.getJobFromHistory('project-test2').result,
95 'SUCCESS')
96 self.assertEqual(A.data['status'], 'MERGED')
97 self.assertEqual(A.reported, 2)
James E. Blairb0fcae42012-07-17 11:12:10 -070098
James E. Blair66eeebf2013-07-27 17:44:32 -070099 self.assertReportedStat('gerrit.event.comment-added', value='1|c')
100 self.assertReportedStat('zuul.pipeline.gate.current_changes',
101 value='1|g')
102 self.assertReportedStat('zuul.pipeline.gate.job.project-merge.SUCCESS',
103 kind='ms')
104 self.assertReportedStat('zuul.pipeline.gate.job.project-merge.SUCCESS',
105 value='1|c')
106 self.assertReportedStat('zuul.pipeline.gate.resident_time', kind='ms')
107 self.assertReportedStat('zuul.pipeline.gate.total_changes',
108 value='1|c')
James E. Blair412e5582013-04-22 15:50:12 -0700109 self.assertReportedStat(
James E. Blair66eeebf2013-07-27 17:44:32 -0700110 'zuul.pipeline.gate.org.project.resident_time', kind='ms')
James E. Blair412e5582013-04-22 15:50:12 -0700111 self.assertReportedStat(
James E. Blair66eeebf2013-07-27 17:44:32 -0700112 'zuul.pipeline.gate.org.project.total_changes', value='1|c')
James E. Blair412e5582013-04-22 15:50:12 -0700113
James E. Blair5821bd92015-09-16 08:48:15 -0700114 for build in self.builds:
115 self.assertEqual(build.parameters['ZUUL_VOTING'], '1')
116
James E. Blair3cb10702013-08-24 08:56:03 -0700117 def test_initial_pipeline_gauges(self):
118 "Test that each pipeline reported its length on start"
119 pipeline_names = self.sched.layout.pipelines.keys()
120 self.assertNotEqual(len(pipeline_names), 0)
121 for name in pipeline_names:
122 self.assertReportedStat('zuul.pipeline.%s.current_changes' % name,
123 value='0|g')
124
James E. Blair42f74822013-05-14 15:18:03 -0700125 def test_duplicate_pipelines(self):
126 "Test that a change matching multiple pipelines works"
James E. Blair1b4d9722013-05-21 10:32:04 -0700127
James E. Blair42f74822013-05-14 15:18:03 -0700128 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
129 self.fake_gerrit.addEvent(A.getChangeRestoredEvent())
130 self.waitUntilSettled()
James E. Blair42f74822013-05-14 15:18:03 -0700131
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400132 self.assertEqual(len(self.history), 2)
Monty Taylor6bef8ef2013-06-02 08:17:12 -0400133 self.history[0].name == 'project-test1'
134 self.history[1].name == 'project-test1'
James E. Blair42f74822013-05-14 15:18:03 -0700135
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400136 self.assertEqual(len(A.messages), 2)
James E. Blair42f74822013-05-14 15:18:03 -0700137 if 'dup1/project-test1' in A.messages[0]:
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400138 self.assertIn('dup1/project-test1', A.messages[0])
139 self.assertNotIn('dup2/project-test1', A.messages[0])
140 self.assertNotIn('dup1/project-test1', A.messages[1])
141 self.assertIn('dup2/project-test1', A.messages[1])
James E. Blair42f74822013-05-14 15:18:03 -0700142 else:
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400143 self.assertIn('dup1/project-test1', A.messages[1])
144 self.assertNotIn('dup2/project-test1', A.messages[1])
145 self.assertNotIn('dup1/project-test1', A.messages[0])
146 self.assertIn('dup2/project-test1', A.messages[0])
James E. Blair42f74822013-05-14 15:18:03 -0700147
James E. Blairb0fcae42012-07-17 11:12:10 -0700148 def test_parallel_changes(self):
149 "Test that changes are tested in parallel and merged in series"
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700150
151 self.worker.hold_jobs_in_build = True
James E. Blairb0fcae42012-07-17 11:12:10 -0700152 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
153 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
154 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
James E. Blair8c803f82012-07-31 16:25:42 -0700155 A.addApproval('CRVW', 2)
156 B.addApproval('CRVW', 2)
157 C.addApproval('CRVW', 2)
James E. Blairb0fcae42012-07-17 11:12:10 -0700158
159 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
160 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
161 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
162
163 self.waitUntilSettled()
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400164 self.assertEqual(len(self.builds), 1)
165 self.assertEqual(self.builds[0].name, 'project-merge')
166 self.assertTrue(self.job_has_changes(self.builds[0], A))
James E. Blairb0fcae42012-07-17 11:12:10 -0700167
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700168 self.worker.release('.*-merge')
James E. Blairb0fcae42012-07-17 11:12:10 -0700169 self.waitUntilSettled()
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400170 self.assertEqual(len(self.builds), 3)
171 self.assertEqual(self.builds[0].name, 'project-test1')
172 self.assertTrue(self.job_has_changes(self.builds[0], A))
173 self.assertEqual(self.builds[1].name, 'project-test2')
174 self.assertTrue(self.job_has_changes(self.builds[1], A))
175 self.assertEqual(self.builds[2].name, 'project-merge')
176 self.assertTrue(self.job_has_changes(self.builds[2], A, B))
James E. Blairb0fcae42012-07-17 11:12:10 -0700177
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700178 self.worker.release('.*-merge')
James E. Blairb0fcae42012-07-17 11:12:10 -0700179 self.waitUntilSettled()
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400180 self.assertEqual(len(self.builds), 5)
181 self.assertEqual(self.builds[0].name, 'project-test1')
182 self.assertTrue(self.job_has_changes(self.builds[0], A))
183 self.assertEqual(self.builds[1].name, 'project-test2')
184 self.assertTrue(self.job_has_changes(self.builds[1], A))
James E. Blairb0fcae42012-07-17 11:12:10 -0700185
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400186 self.assertEqual(self.builds[2].name, 'project-test1')
187 self.assertTrue(self.job_has_changes(self.builds[2], A, B))
188 self.assertEqual(self.builds[3].name, 'project-test2')
189 self.assertTrue(self.job_has_changes(self.builds[3], A, B))
James E. Blairb0fcae42012-07-17 11:12:10 -0700190
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400191 self.assertEqual(self.builds[4].name, 'project-merge')
192 self.assertTrue(self.job_has_changes(self.builds[4], A, B, C))
James E. Blairb0fcae42012-07-17 11:12:10 -0700193
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700194 self.worker.release('.*-merge')
James E. Blairb0fcae42012-07-17 11:12:10 -0700195 self.waitUntilSettled()
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400196 self.assertEqual(len(self.builds), 6)
197 self.assertEqual(self.builds[0].name, 'project-test1')
198 self.assertTrue(self.job_has_changes(self.builds[0], A))
199 self.assertEqual(self.builds[1].name, 'project-test2')
200 self.assertTrue(self.job_has_changes(self.builds[1], A))
James E. Blairb0fcae42012-07-17 11:12:10 -0700201
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400202 self.assertEqual(self.builds[2].name, 'project-test1')
203 self.assertTrue(self.job_has_changes(self.builds[2], A, B))
204 self.assertEqual(self.builds[3].name, 'project-test2')
205 self.assertTrue(self.job_has_changes(self.builds[3], A, B))
James E. Blairb0fcae42012-07-17 11:12:10 -0700206
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400207 self.assertEqual(self.builds[4].name, 'project-test1')
208 self.assertTrue(self.job_has_changes(self.builds[4], A, B, C))
209 self.assertEqual(self.builds[5].name, 'project-test2')
210 self.assertTrue(self.job_has_changes(self.builds[5], A, B, C))
James E. Blairb0fcae42012-07-17 11:12:10 -0700211
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700212 self.worker.hold_jobs_in_build = False
213 self.worker.release()
James E. Blairb0fcae42012-07-17 11:12:10 -0700214 self.waitUntilSettled()
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400215 self.assertEqual(len(self.builds), 0)
James E. Blairb0fcae42012-07-17 11:12:10 -0700216
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400217 self.assertEqual(len(self.history), 9)
218 self.assertEqual(A.data['status'], 'MERGED')
219 self.assertEqual(B.data['status'], 'MERGED')
220 self.assertEqual(C.data['status'], 'MERGED')
221 self.assertEqual(A.reported, 2)
222 self.assertEqual(B.reported, 2)
223 self.assertEqual(C.reported, 2)
James E. Blairb02a3bb2012-07-30 17:49:55 -0700224
225 def test_failed_changes(self):
226 "Test that a change behind a failed change is retested"
James E. Blaire2819012013-06-28 17:17:26 -0400227 self.worker.hold_jobs_in_build = True
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700228
James E. Blairb02a3bb2012-07-30 17:49:55 -0700229 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
230 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
James E. Blair8c803f82012-07-31 16:25:42 -0700231 A.addApproval('CRVW', 2)
232 B.addApproval('CRVW', 2)
James E. Blairb02a3bb2012-07-30 17:49:55 -0700233
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700234 self.worker.addFailTest('project-test1', A)
James E. Blairb02a3bb2012-07-30 17:49:55 -0700235
James E. Blaire2819012013-06-28 17:17:26 -0400236 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
237 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
James E. Blairb02a3bb2012-07-30 17:49:55 -0700238 self.waitUntilSettled()
James E. Blaire2819012013-06-28 17:17:26 -0400239
240 self.worker.release('.*-merge')
241 self.waitUntilSettled()
242
243 self.worker.hold_jobs_in_build = False
244 self.worker.release()
245
246 self.waitUntilSettled()
247 # It's certain that the merge job for change 2 will run, but
248 # the test1 and test2 jobs may or may not run.
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400249 self.assertTrue(len(self.history) > 6)
250 self.assertEqual(A.data['status'], 'NEW')
251 self.assertEqual(B.data['status'], 'MERGED')
252 self.assertEqual(A.reported, 2)
253 self.assertEqual(B.reported, 2)
James E. Blairb02a3bb2012-07-30 17:49:55 -0700254
255 def test_independent_queues(self):
256 "Test that changes end up in the right queues"
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700257
258 self.worker.hold_jobs_in_build = True
Zhongyue Luo5d556072012-09-21 02:00:47 +0900259 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
James E. Blairb02a3bb2012-07-30 17:49:55 -0700260 B = self.fake_gerrit.addFakeChange('org/project1', 'master', 'B')
261 C = self.fake_gerrit.addFakeChange('org/project2', 'master', 'C')
James E. Blair8c803f82012-07-31 16:25:42 -0700262 A.addApproval('CRVW', 2)
263 B.addApproval('CRVW', 2)
264 C.addApproval('CRVW', 2)
James E. Blairb02a3bb2012-07-30 17:49:55 -0700265
266 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
267 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
268 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
269
James E. Blairb02a3bb2012-07-30 17:49:55 -0700270 self.waitUntilSettled()
271
272 # There should be one merge job at the head of each queue running
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400273 self.assertEqual(len(self.builds), 2)
274 self.assertEqual(self.builds[0].name, 'project-merge')
275 self.assertTrue(self.job_has_changes(self.builds[0], A))
276 self.assertEqual(self.builds[1].name, 'project1-merge')
277 self.assertTrue(self.job_has_changes(self.builds[1], B))
James E. Blairb02a3bb2012-07-30 17:49:55 -0700278
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700279 # Release the current merge builds
280 self.worker.release('.*-merge')
James E. Blairb02a3bb2012-07-30 17:49:55 -0700281 self.waitUntilSettled()
282 # Release the merge job for project2 which is behind project1
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700283 self.worker.release('.*-merge')
James E. Blairb02a3bb2012-07-30 17:49:55 -0700284 self.waitUntilSettled()
285
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700286 # All the test builds should be running:
James E. Blairb02a3bb2012-07-30 17:49:55 -0700287 # project1 (3) + project2 (3) + project (2) = 8
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400288 self.assertEqual(len(self.builds), 8)
James E. Blairb02a3bb2012-07-30 17:49:55 -0700289
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700290 self.worker.release()
James E. Blairb02a3bb2012-07-30 17:49:55 -0700291 self.waitUntilSettled()
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400292 self.assertEqual(len(self.builds), 0)
James E. Blairb02a3bb2012-07-30 17:49:55 -0700293
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400294 self.assertEqual(len(self.history), 11)
295 self.assertEqual(A.data['status'], 'MERGED')
296 self.assertEqual(B.data['status'], 'MERGED')
297 self.assertEqual(C.data['status'], 'MERGED')
298 self.assertEqual(A.reported, 2)
299 self.assertEqual(B.reported, 2)
300 self.assertEqual(C.reported, 2)
James E. Blaird466dc42012-07-31 10:42:56 -0700301
302 def test_failed_change_at_head(self):
303 "Test that if a change at the head fails, jobs behind it are canceled"
James E. Blaird466dc42012-07-31 10:42:56 -0700304
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700305 self.worker.hold_jobs_in_build = True
James E. Blaird466dc42012-07-31 10:42:56 -0700306 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
307 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
308 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
James E. Blair8c803f82012-07-31 16:25:42 -0700309 A.addApproval('CRVW', 2)
310 B.addApproval('CRVW', 2)
311 C.addApproval('CRVW', 2)
James E. Blaird466dc42012-07-31 10:42:56 -0700312
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700313 self.worker.addFailTest('project-test1', A)
James E. Blaird466dc42012-07-31 10:42:56 -0700314
315 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
316 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
317 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
318
319 self.waitUntilSettled()
James E. Blaird466dc42012-07-31 10:42:56 -0700320
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400321 self.assertEqual(len(self.builds), 1)
322 self.assertEqual(self.builds[0].name, 'project-merge')
323 self.assertTrue(self.job_has_changes(self.builds[0], A))
James E. Blaird466dc42012-07-31 10:42:56 -0700324
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700325 self.worker.release('.*-merge')
James E. Blaird466dc42012-07-31 10:42:56 -0700326 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700327 self.worker.release('.*-merge')
James E. Blaird466dc42012-07-31 10:42:56 -0700328 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700329 self.worker.release('.*-merge')
James E. Blaird466dc42012-07-31 10:42:56 -0700330 self.waitUntilSettled()
331
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400332 self.assertEqual(len(self.builds), 6)
333 self.assertEqual(self.builds[0].name, 'project-test1')
334 self.assertEqual(self.builds[1].name, 'project-test2')
335 self.assertEqual(self.builds[2].name, 'project-test1')
336 self.assertEqual(self.builds[3].name, 'project-test2')
337 self.assertEqual(self.builds[4].name, 'project-test1')
338 self.assertEqual(self.builds[5].name, 'project-test2')
James E. Blaird466dc42012-07-31 10:42:56 -0700339
Monty Taylor6bef8ef2013-06-02 08:17:12 -0400340 self.release(self.builds[0])
James E. Blaird466dc42012-07-31 10:42:56 -0700341 self.waitUntilSettled()
342
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400343 # project-test2, project-merge for B
344 self.assertEqual(len(self.builds), 2)
345 self.assertEqual(self.countJobResults(self.history, 'ABORTED'), 4)
James E. Blaird466dc42012-07-31 10:42:56 -0700346
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700347 self.worker.hold_jobs_in_build = False
348 self.worker.release()
James E. Blaird466dc42012-07-31 10:42:56 -0700349 self.waitUntilSettled()
350
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400351 self.assertEqual(len(self.builds), 0)
352 self.assertEqual(len(self.history), 15)
353 self.assertEqual(A.data['status'], 'NEW')
354 self.assertEqual(B.data['status'], 'MERGED')
355 self.assertEqual(C.data['status'], 'MERGED')
356 self.assertEqual(A.reported, 2)
357 self.assertEqual(B.reported, 2)
358 self.assertEqual(C.reported, 2)
James E. Blaird466dc42012-07-31 10:42:56 -0700359
James E. Blair0aac4872013-08-23 14:02:38 -0700360 def test_failed_change_in_middle(self):
361 "Test a failed change in the middle of the queue"
362
363 self.worker.hold_jobs_in_build = True
364 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
365 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
366 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
367 A.addApproval('CRVW', 2)
368 B.addApproval('CRVW', 2)
369 C.addApproval('CRVW', 2)
370
371 self.worker.addFailTest('project-test1', B)
372
373 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
374 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
375 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
376
377 self.waitUntilSettled()
378
379 self.worker.release('.*-merge')
380 self.waitUntilSettled()
381 self.worker.release('.*-merge')
382 self.waitUntilSettled()
383 self.worker.release('.*-merge')
384 self.waitUntilSettled()
385
386 self.assertEqual(len(self.builds), 6)
387 self.assertEqual(self.builds[0].name, 'project-test1')
388 self.assertEqual(self.builds[1].name, 'project-test2')
389 self.assertEqual(self.builds[2].name, 'project-test1')
390 self.assertEqual(self.builds[3].name, 'project-test2')
391 self.assertEqual(self.builds[4].name, 'project-test1')
392 self.assertEqual(self.builds[5].name, 'project-test2')
393
394 self.release(self.builds[2])
395 self.waitUntilSettled()
396
James E. Blair972e3c72013-08-29 12:04:55 -0700397 # project-test1 and project-test2 for A
398 # project-test2 for B
399 # project-merge for C (without B)
400 self.assertEqual(len(self.builds), 4)
James E. Blair0aac4872013-08-23 14:02:38 -0700401 self.assertEqual(self.countJobResults(self.history, 'ABORTED'), 2)
402
James E. Blair972e3c72013-08-29 12:04:55 -0700403 self.worker.release('.*-merge')
404 self.waitUntilSettled()
405
406 # project-test1 and project-test2 for A
407 # project-test2 for B
408 # project-test1 and project-test2 for C
409 self.assertEqual(len(self.builds), 5)
410
James E. Blair0aac4872013-08-23 14:02:38 -0700411 items = self.sched.layout.pipelines['gate'].getAllItems()
412 builds = items[0].current_build_set.getBuilds()
413 self.assertEqual(self.countJobResults(builds, 'SUCCESS'), 1)
414 self.assertEqual(self.countJobResults(builds, None), 2)
415 builds = items[1].current_build_set.getBuilds()
416 self.assertEqual(self.countJobResults(builds, 'SUCCESS'), 1)
417 self.assertEqual(self.countJobResults(builds, 'FAILURE'), 1)
418 self.assertEqual(self.countJobResults(builds, None), 1)
419 builds = items[2].current_build_set.getBuilds()
420 self.assertEqual(self.countJobResults(builds, 'SUCCESS'), 1)
James E. Blair972e3c72013-08-29 12:04:55 -0700421 self.assertEqual(self.countJobResults(builds, None), 2)
James E. Blair0aac4872013-08-23 14:02:38 -0700422
423 self.worker.hold_jobs_in_build = False
424 self.worker.release()
425 self.waitUntilSettled()
426
427 self.assertEqual(len(self.builds), 0)
428 self.assertEqual(len(self.history), 12)
429 self.assertEqual(A.data['status'], 'MERGED')
430 self.assertEqual(B.data['status'], 'NEW')
431 self.assertEqual(C.data['status'], 'MERGED')
432 self.assertEqual(A.reported, 2)
433 self.assertEqual(B.reported, 2)
434 self.assertEqual(C.reported, 2)
435
James E. Blaird466dc42012-07-31 10:42:56 -0700436 def test_failed_change_at_head_with_queue(self):
437 "Test that if a change at the head fails, queued jobs are canceled"
James E. Blaird466dc42012-07-31 10:42:56 -0700438
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700439 self.gearman_server.hold_jobs_in_queue = True
James E. Blaird466dc42012-07-31 10:42:56 -0700440 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
441 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
442 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
James E. Blair8c803f82012-07-31 16:25:42 -0700443 A.addApproval('CRVW', 2)
444 B.addApproval('CRVW', 2)
445 C.addApproval('CRVW', 2)
James E. Blaird466dc42012-07-31 10:42:56 -0700446
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700447 self.worker.addFailTest('project-test1', A)
James E. Blaird466dc42012-07-31 10:42:56 -0700448
449 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
450 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
451 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
452
453 self.waitUntilSettled()
James E. Blair701c5b42013-06-06 09:34:59 -0700454 queue = self.gearman_server.getQueue()
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400455 self.assertEqual(len(self.builds), 0)
456 self.assertEqual(len(queue), 1)
457 self.assertEqual(queue[0].name, 'build:project-merge')
458 self.assertTrue(self.job_has_changes(queue[0], A))
James E. Blaird466dc42012-07-31 10:42:56 -0700459
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700460 self.gearman_server.release('.*-merge')
James E. Blaird466dc42012-07-31 10:42:56 -0700461 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700462 self.gearman_server.release('.*-merge')
James E. Blaird466dc42012-07-31 10:42:56 -0700463 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700464 self.gearman_server.release('.*-merge')
James E. Blaird466dc42012-07-31 10:42:56 -0700465 self.waitUntilSettled()
James E. Blair701c5b42013-06-06 09:34:59 -0700466 queue = self.gearman_server.getQueue()
James E. Blaird466dc42012-07-31 10:42:56 -0700467
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400468 self.assertEqual(len(self.builds), 0)
469 self.assertEqual(len(queue), 6)
470 self.assertEqual(queue[0].name, 'build:project-test1')
471 self.assertEqual(queue[1].name, 'build:project-test2')
472 self.assertEqual(queue[2].name, 'build:project-test1')
473 self.assertEqual(queue[3].name, 'build:project-test2')
474 self.assertEqual(queue[4].name, 'build:project-test1')
475 self.assertEqual(queue[5].name, 'build:project-test2')
James E. Blaird466dc42012-07-31 10:42:56 -0700476
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700477 self.release(queue[0])
James E. Blaird466dc42012-07-31 10:42:56 -0700478 self.waitUntilSettled()
479
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400480 self.assertEqual(len(self.builds), 0)
James E. Blair701c5b42013-06-06 09:34:59 -0700481 queue = self.gearman_server.getQueue()
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400482 self.assertEqual(len(queue), 2) # project-test2, project-merge for B
483 self.assertEqual(self.countJobResults(self.history, 'ABORTED'), 0)
James E. Blaird466dc42012-07-31 10:42:56 -0700484
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700485 self.gearman_server.hold_jobs_in_queue = False
486 self.gearman_server.release()
James E. Blaird466dc42012-07-31 10:42:56 -0700487 self.waitUntilSettled()
488
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400489 self.assertEqual(len(self.builds), 0)
490 self.assertEqual(len(self.history), 11)
491 self.assertEqual(A.data['status'], 'NEW')
492 self.assertEqual(B.data['status'], 'MERGED')
493 self.assertEqual(C.data['status'], 'MERGED')
494 self.assertEqual(A.reported, 2)
495 self.assertEqual(B.reported, 2)
496 self.assertEqual(C.reported, 2)
James E. Blair8c803f82012-07-31 16:25:42 -0700497
James E. Blairfef71632013-09-23 11:15:47 -0700498 def test_two_failed_changes_at_head(self):
499 "Test that changes are reparented correctly if 2 fail at head"
500
501 self.worker.hold_jobs_in_build = True
502 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
503 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
504 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
505 A.addApproval('CRVW', 2)
506 B.addApproval('CRVW', 2)
507 C.addApproval('CRVW', 2)
508
509 self.worker.addFailTest('project-test1', A)
510 self.worker.addFailTest('project-test1', B)
511
512 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
513 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
514 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
515 self.waitUntilSettled()
516
517 self.worker.release('.*-merge')
518 self.waitUntilSettled()
519 self.worker.release('.*-merge')
520 self.waitUntilSettled()
521 self.worker.release('.*-merge')
522 self.waitUntilSettled()
523
524 self.assertEqual(len(self.builds), 6)
525 self.assertEqual(self.builds[0].name, 'project-test1')
526 self.assertEqual(self.builds[1].name, 'project-test2')
527 self.assertEqual(self.builds[2].name, 'project-test1')
528 self.assertEqual(self.builds[3].name, 'project-test2')
529 self.assertEqual(self.builds[4].name, 'project-test1')
530 self.assertEqual(self.builds[5].name, 'project-test2')
531
532 self.assertTrue(self.job_has_changes(self.builds[0], A))
533 self.assertTrue(self.job_has_changes(self.builds[2], A))
534 self.assertTrue(self.job_has_changes(self.builds[2], B))
535 self.assertTrue(self.job_has_changes(self.builds[4], A))
536 self.assertTrue(self.job_has_changes(self.builds[4], B))
537 self.assertTrue(self.job_has_changes(self.builds[4], C))
538
539 # Fail change B first
540 self.release(self.builds[2])
541 self.waitUntilSettled()
542
543 # restart of C after B failure
544 self.worker.release('.*-merge')
545 self.waitUntilSettled()
546
547 self.assertEqual(len(self.builds), 5)
548 self.assertEqual(self.builds[0].name, 'project-test1')
549 self.assertEqual(self.builds[1].name, 'project-test2')
550 self.assertEqual(self.builds[2].name, 'project-test2')
551 self.assertEqual(self.builds[3].name, 'project-test1')
552 self.assertEqual(self.builds[4].name, 'project-test2')
553
554 self.assertTrue(self.job_has_changes(self.builds[1], A))
555 self.assertTrue(self.job_has_changes(self.builds[2], A))
556 self.assertTrue(self.job_has_changes(self.builds[2], B))
557 self.assertTrue(self.job_has_changes(self.builds[4], A))
558 self.assertFalse(self.job_has_changes(self.builds[4], B))
559 self.assertTrue(self.job_has_changes(self.builds[4], C))
560
561 # Finish running all passing jobs for change A
562 self.release(self.builds[1])
563 self.waitUntilSettled()
564 # Fail and report change A
565 self.release(self.builds[0])
566 self.waitUntilSettled()
567
568 # restart of B,C after A failure
569 self.worker.release('.*-merge')
570 self.waitUntilSettled()
571 self.worker.release('.*-merge')
572 self.waitUntilSettled()
573
574 self.assertEqual(len(self.builds), 4)
575 self.assertEqual(self.builds[0].name, 'project-test1') # B
576 self.assertEqual(self.builds[1].name, 'project-test2') # B
577 self.assertEqual(self.builds[2].name, 'project-test1') # C
578 self.assertEqual(self.builds[3].name, 'project-test2') # C
579
580 self.assertFalse(self.job_has_changes(self.builds[1], A))
581 self.assertTrue(self.job_has_changes(self.builds[1], B))
582 self.assertFalse(self.job_has_changes(self.builds[1], C))
583
584 self.assertFalse(self.job_has_changes(self.builds[2], A))
585 # After A failed and B and C restarted, B should be back in
586 # C's tests because it has not failed yet.
587 self.assertTrue(self.job_has_changes(self.builds[2], B))
588 self.assertTrue(self.job_has_changes(self.builds[2], C))
589
590 self.worker.hold_jobs_in_build = False
591 self.worker.release()
592 self.waitUntilSettled()
593
594 self.assertEqual(len(self.builds), 0)
595 self.assertEqual(len(self.history), 21)
596 self.assertEqual(A.data['status'], 'NEW')
597 self.assertEqual(B.data['status'], 'NEW')
598 self.assertEqual(C.data['status'], 'MERGED')
599 self.assertEqual(A.reported, 2)
600 self.assertEqual(B.reported, 2)
601 self.assertEqual(C.reported, 2)
602
James E. Blair8c803f82012-07-31 16:25:42 -0700603 def test_patch_order(self):
604 "Test that dependent patches are tested in the right order"
605 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
606 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
607 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
608 A.addApproval('CRVW', 2)
609 B.addApproval('CRVW', 2)
610 C.addApproval('CRVW', 2)
611
612 M2 = self.fake_gerrit.addFakeChange('org/project', 'master', 'M2')
613 M1 = self.fake_gerrit.addFakeChange('org/project', 'master', 'M1')
614 M2.setMerged()
615 M1.setMerged()
616
617 # C -> B -> A -> M1 -> M2
618 # M2 is here to make sure it is never queried. If it is, it
619 # means zuul is walking down the entire history of merged
620 # changes.
621
622 C.setDependsOn(B, 1)
623 B.setDependsOn(A, 1)
624 A.setDependsOn(M1, 1)
625 M1.setDependsOn(M2, 1)
626
627 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
628
629 self.waitUntilSettled()
630
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400631 self.assertEqual(A.data['status'], 'NEW')
632 self.assertEqual(B.data['status'], 'NEW')
633 self.assertEqual(C.data['status'], 'NEW')
James E. Blair8c803f82012-07-31 16:25:42 -0700634
635 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
636 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
637
638 self.waitUntilSettled()
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400639 self.assertEqual(M2.queried, 0)
640 self.assertEqual(A.data['status'], 'MERGED')
641 self.assertEqual(B.data['status'], 'MERGED')
642 self.assertEqual(C.data['status'], 'MERGED')
643 self.assertEqual(A.reported, 2)
644 self.assertEqual(B.reported, 2)
645 self.assertEqual(C.reported, 2)
James E. Blair8c803f82012-07-31 16:25:42 -0700646
James E. Blair063672f2015-01-29 13:09:12 -0800647 def test_needed_changes_enqueue(self):
648 "Test that a needed change is enqueued ahead"
649 # A Given a git tree like this, if we enqueue
650 # / \ change C, we should walk up and down the tree
651 # B G and enqueue changes in the order ABCDEFG.
652 # /|\ This is also the order that you would get if
653 # *C E F you enqueued changes in the order ABCDEFG, so
654 # / the ordering is stable across re-enqueue events.
655 # D
656
657 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
658 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
659 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
660 D = self.fake_gerrit.addFakeChange('org/project', 'master', 'D')
661 E = self.fake_gerrit.addFakeChange('org/project', 'master', 'E')
662 F = self.fake_gerrit.addFakeChange('org/project', 'master', 'F')
663 G = self.fake_gerrit.addFakeChange('org/project', 'master', 'G')
664 B.setDependsOn(A, 1)
665 C.setDependsOn(B, 1)
666 D.setDependsOn(C, 1)
667 E.setDependsOn(B, 1)
668 F.setDependsOn(B, 1)
669 G.setDependsOn(A, 1)
670
671 A.addApproval('CRVW', 2)
672 B.addApproval('CRVW', 2)
673 C.addApproval('CRVW', 2)
674 D.addApproval('CRVW', 2)
675 E.addApproval('CRVW', 2)
676 F.addApproval('CRVW', 2)
677 G.addApproval('CRVW', 2)
678 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
679
680 self.waitUntilSettled()
681
682 self.assertEqual(A.data['status'], 'NEW')
683 self.assertEqual(B.data['status'], 'NEW')
684 self.assertEqual(C.data['status'], 'NEW')
685 self.assertEqual(D.data['status'], 'NEW')
686 self.assertEqual(E.data['status'], 'NEW')
687 self.assertEqual(F.data['status'], 'NEW')
688 self.assertEqual(G.data['status'], 'NEW')
689
690 # We're about to add approvals to changes without adding the
691 # triggering events to Zuul, so that we can be sure that it is
692 # enqueing the changes based on dependencies, not because of
693 # triggering events. Since it will have the changes cached
694 # already (without approvals), we need to clear the cache
695 # first.
Joshua Hesketh4bd7da32016-02-17 20:58:47 +1100696 for connection in self.connections.values():
697 connection.maintainCache([])
James E. Blair063672f2015-01-29 13:09:12 -0800698
699 self.worker.hold_jobs_in_build = True
700 A.addApproval('APRV', 1)
701 B.addApproval('APRV', 1)
702 D.addApproval('APRV', 1)
703 E.addApproval('APRV', 1)
704 F.addApproval('APRV', 1)
705 G.addApproval('APRV', 1)
706 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
707
708 for x in range(8):
709 self.worker.release('.*-merge')
710 self.waitUntilSettled()
711 self.worker.hold_jobs_in_build = False
712 self.worker.release()
713 self.waitUntilSettled()
714
715 self.assertEqual(A.data['status'], 'MERGED')
716 self.assertEqual(B.data['status'], 'MERGED')
717 self.assertEqual(C.data['status'], 'MERGED')
718 self.assertEqual(D.data['status'], 'MERGED')
719 self.assertEqual(E.data['status'], 'MERGED')
720 self.assertEqual(F.data['status'], 'MERGED')
721 self.assertEqual(G.data['status'], 'MERGED')
722 self.assertEqual(A.reported, 2)
723 self.assertEqual(B.reported, 2)
724 self.assertEqual(C.reported, 2)
725 self.assertEqual(D.reported, 2)
726 self.assertEqual(E.reported, 2)
727 self.assertEqual(F.reported, 2)
728 self.assertEqual(G.reported, 2)
729 self.assertEqual(self.history[6].changes,
730 '1,1 2,1 3,1 4,1 5,1 6,1 7,1')
731
Joshua Hesketh850ccb62014-11-27 11:31:02 +1100732 def test_source_cache(self):
733 "Test that the source cache operates correctly"
James E. Blair0e933c52013-07-11 10:18:52 -0700734 self.worker.hold_jobs_in_build = True
735
736 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
737 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
738 X = self.fake_gerrit.addFakeChange('org/project', 'master', 'X')
739 A.addApproval('CRVW', 2)
740 B.addApproval('CRVW', 2)
741
742 M1 = self.fake_gerrit.addFakeChange('org/project', 'master', 'M1')
743 M1.setMerged()
744
745 B.setDependsOn(A, 1)
746 A.setDependsOn(M1, 1)
747
748 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
749 self.fake_gerrit.addEvent(X.getPatchsetCreatedEvent(1))
750
751 self.waitUntilSettled()
752
753 for build in self.builds:
754 if build.parameters['ZUUL_PIPELINE'] == 'check':
755 build.release()
756 self.waitUntilSettled()
757 for build in self.builds:
758 if build.parameters['ZUUL_PIPELINE'] == 'check':
759 build.release()
760 self.waitUntilSettled()
761
762 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
763 self.waitUntilSettled()
764
Joshua Hesketh352264b2015-08-11 23:42:08 +1000765 self.log.debug("len %s" % self.fake_gerrit._change_cache.keys())
James E. Blair0e933c52013-07-11 10:18:52 -0700766 # there should still be changes in the cache
Joshua Hesketh352264b2015-08-11 23:42:08 +1000767 self.assertNotEqual(len(self.fake_gerrit._change_cache.keys()), 0)
James E. Blair0e933c52013-07-11 10:18:52 -0700768
769 self.worker.hold_jobs_in_build = False
770 self.worker.release()
771 self.waitUntilSettled()
772
773 self.assertEqual(A.data['status'], 'MERGED')
774 self.assertEqual(B.data['status'], 'MERGED')
775 self.assertEqual(A.queried, 2) # Initial and isMerged
776 self.assertEqual(B.queried, 3) # Initial A, refresh from B, isMerged
777
James E. Blair8c803f82012-07-31 16:25:42 -0700778 def test_can_merge(self):
James E. Blair4886cc12012-07-18 15:39:41 -0700779 "Test whether a change is ready to merge"
James E. Blair8c803f82012-07-31 16:25:42 -0700780 # TODO: move to test_gerrit (this is a unit test!)
781 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
James E. Blairc0dedf82014-08-06 09:37:52 -0700782 source = self.sched.layout.pipelines['gate'].source
783 a = source._getChange(1, 2)
James E. Blaireff88162013-07-01 12:44:14 -0400784 mgr = self.sched.layout.pipelines['gate'].manager
James E. Blairc0dedf82014-08-06 09:37:52 -0700785 self.assertFalse(source.canMerge(a, mgr.getSubmitAllowNeeds()))
James E. Blair8c803f82012-07-31 16:25:42 -0700786
787 A.addApproval('CRVW', 2)
James E. Blairc0dedf82014-08-06 09:37:52 -0700788 a = source._getChange(1, 2, refresh=True)
789 self.assertFalse(source.canMerge(a, mgr.getSubmitAllowNeeds()))
James E. Blair8c803f82012-07-31 16:25:42 -0700790
791 A.addApproval('APRV', 1)
James E. Blairc0dedf82014-08-06 09:37:52 -0700792 a = source._getChange(1, 2, refresh=True)
793 self.assertTrue(source.canMerge(a, mgr.getSubmitAllowNeeds()))
James E. Blair4886cc12012-07-18 15:39:41 -0700794
795 def test_build_configuration(self):
796 "Test that zuul merges the right commits for testing"
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700797
798 self.gearman_server.hold_jobs_in_queue = True
James E. Blair4886cc12012-07-18 15:39:41 -0700799 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
800 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
801 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
802 A.addApproval('CRVW', 2)
803 B.addApproval('CRVW', 2)
804 C.addApproval('CRVW', 2)
805 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
806 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
807 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
808 self.waitUntilSettled()
809
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700810 self.gearman_server.release('.*-merge')
James E. Blair4886cc12012-07-18 15:39:41 -0700811 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700812 self.gearman_server.release('.*-merge')
James E. Blair4886cc12012-07-18 15:39:41 -0700813 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700814 self.gearman_server.release('.*-merge')
James E. Blair4886cc12012-07-18 15:39:41 -0700815 self.waitUntilSettled()
James E. Blair701c5b42013-06-06 09:34:59 -0700816 queue = self.gearman_server.getQueue()
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700817 ref = self.getParameter(queue[-1], 'ZUUL_REF')
818 self.gearman_server.hold_jobs_in_queue = False
819 self.gearman_server.release()
James E. Blair973721f2012-08-15 10:19:43 -0700820 self.waitUntilSettled()
James E. Blair4886cc12012-07-18 15:39:41 -0700821
Monty Taylorbc758832013-06-17 17:22:42 -0400822 path = os.path.join(self.git_root, "org/project")
James E. Blair4886cc12012-07-18 15:39:41 -0700823 repo = git.Repo(path)
824 repo_messages = [c.message.strip() for c in repo.iter_commits(ref)]
825 repo_messages.reverse()
James E. Blair4886cc12012-07-18 15:39:41 -0700826 correct_messages = ['initial commit', 'A-1', 'B-1', 'C-1']
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400827 self.assertEqual(repo_messages, correct_messages)
James E. Blair973721f2012-08-15 10:19:43 -0700828
829 def test_build_configuration_conflict(self):
830 "Test that merge conflicts are handled"
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700831
832 self.gearman_server.hold_jobs_in_queue = True
James E. Blair6736beb2013-07-11 15:18:15 -0700833 A = self.fake_gerrit.addFakeChange('org/conflict-project',
834 'master', 'A')
James E. Blair973721f2012-08-15 10:19:43 -0700835 A.addPatchset(['conflict'])
James E. Blair6736beb2013-07-11 15:18:15 -0700836 B = self.fake_gerrit.addFakeChange('org/conflict-project',
837 'master', 'B')
James E. Blair973721f2012-08-15 10:19:43 -0700838 B.addPatchset(['conflict'])
James E. Blair6736beb2013-07-11 15:18:15 -0700839 C = self.fake_gerrit.addFakeChange('org/conflict-project',
840 'master', 'C')
James E. Blair973721f2012-08-15 10:19:43 -0700841 A.addApproval('CRVW', 2)
842 B.addApproval('CRVW', 2)
843 C.addApproval('CRVW', 2)
844 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
845 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
846 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
847 self.waitUntilSettled()
848
James E. Blair6736beb2013-07-11 15:18:15 -0700849 self.assertEqual(A.reported, 1)
850 self.assertEqual(B.reported, 1)
851 self.assertEqual(C.reported, 1)
852
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700853 self.gearman_server.release('.*-merge')
James E. Blair973721f2012-08-15 10:19:43 -0700854 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700855 self.gearman_server.release('.*-merge')
James E. Blair973721f2012-08-15 10:19:43 -0700856 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700857 self.gearman_server.release('.*-merge')
James E. Blair973721f2012-08-15 10:19:43 -0700858 self.waitUntilSettled()
James E. Blair972e3c72013-08-29 12:04:55 -0700859
860 self.assertEqual(len(self.history), 2) # A and C merge jobs
861
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700862 self.gearman_server.hold_jobs_in_queue = False
863 self.gearman_server.release()
James E. Blair973721f2012-08-15 10:19:43 -0700864 self.waitUntilSettled()
865
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400866 self.assertEqual(A.data['status'], 'MERGED')
867 self.assertEqual(B.data['status'], 'NEW')
868 self.assertEqual(C.data['status'], 'MERGED')
869 self.assertEqual(A.reported, 2)
870 self.assertEqual(B.reported, 2)
871 self.assertEqual(C.reported, 2)
James E. Blair972e3c72013-08-29 12:04:55 -0700872 self.assertEqual(len(self.history), 6)
James E. Blair6736beb2013-07-11 15:18:15 -0700873
James E. Blairdaabed22012-08-15 15:38:57 -0700874 def test_post(self):
875 "Test that post jobs run"
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700876
Zhongyue Luo5d556072012-09-21 02:00:47 +0900877 e = {
878 "type": "ref-updated",
879 "submitter": {
880 "name": "User Name",
881 },
882 "refUpdate": {
883 "oldRev": "90f173846e3af9154517b88543ffbd1691f31366",
884 "newRev": "d479a0bfcb34da57a31adb2a595c0cf687812543",
885 "refName": "master",
886 "project": "org/project",
887 }
888 }
James E. Blairdaabed22012-08-15 15:38:57 -0700889 self.fake_gerrit.addEvent(e)
890 self.waitUntilSettled()
891
Monty Taylor6bef8ef2013-06-02 08:17:12 -0400892 job_names = [x.name for x in self.history]
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400893 self.assertEqual(len(self.history), 1)
894 self.assertIn('project-post', job_names)
James E. Blairc6294a52012-08-17 10:19:48 -0700895
K Jonathan Harkerf95e7232015-04-29 13:33:16 -0700896 def test_post_ignore_deletes(self):
897 "Test that deleting refs does not trigger post jobs"
898
899 e = {
900 "type": "ref-updated",
901 "submitter": {
902 "name": "User Name",
903 },
904 "refUpdate": {
905 "oldRev": "90f173846e3af9154517b88543ffbd1691f31366",
906 "newRev": "0000000000000000000000000000000000000000",
907 "refName": "master",
908 "project": "org/project",
909 }
910 }
911 self.fake_gerrit.addEvent(e)
912 self.waitUntilSettled()
913
914 job_names = [x.name for x in self.history]
915 self.assertEqual(len(self.history), 0)
916 self.assertNotIn('project-post', job_names)
917
918 def test_post_ignore_deletes_negative(self):
919 "Test that deleting refs does trigger post jobs"
920
921 self.config.set('zuul', 'layout_config',
922 'tests/fixtures/layout-dont-ignore-deletes.yaml')
923 self.sched.reconfigure(self.config)
924
925 e = {
926 "type": "ref-updated",
927 "submitter": {
928 "name": "User Name",
929 },
930 "refUpdate": {
931 "oldRev": "90f173846e3af9154517b88543ffbd1691f31366",
932 "newRev": "0000000000000000000000000000000000000000",
933 "refName": "master",
934 "project": "org/project",
935 }
936 }
937 self.fake_gerrit.addEvent(e)
938 self.waitUntilSettled()
939
940 job_names = [x.name for x in self.history]
941 self.assertEqual(len(self.history), 1)
942 self.assertIn('project-post', job_names)
943
James E. Blairc6294a52012-08-17 10:19:48 -0700944 def test_build_configuration_branch(self):
945 "Test that the right commits are on alternate branches"
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700946
947 self.gearman_server.hold_jobs_in_queue = True
James E. Blairc6294a52012-08-17 10:19:48 -0700948 A = self.fake_gerrit.addFakeChange('org/project', 'mp', 'A')
949 B = self.fake_gerrit.addFakeChange('org/project', 'mp', 'B')
950 C = self.fake_gerrit.addFakeChange('org/project', 'mp', 'C')
951 A.addApproval('CRVW', 2)
952 B.addApproval('CRVW', 2)
953 C.addApproval('CRVW', 2)
954 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
955 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
956 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
957 self.waitUntilSettled()
958
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700959 self.gearman_server.release('.*-merge')
James E. Blairc6294a52012-08-17 10:19:48 -0700960 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700961 self.gearman_server.release('.*-merge')
James E. Blairc6294a52012-08-17 10:19:48 -0700962 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700963 self.gearman_server.release('.*-merge')
James E. Blairc6294a52012-08-17 10:19:48 -0700964 self.waitUntilSettled()
James E. Blair701c5b42013-06-06 09:34:59 -0700965 queue = self.gearman_server.getQueue()
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700966 ref = self.getParameter(queue[-1], 'ZUUL_REF')
967 self.gearman_server.hold_jobs_in_queue = False
968 self.gearman_server.release()
James E. Blairc6294a52012-08-17 10:19:48 -0700969 self.waitUntilSettled()
970
Monty Taylorbc758832013-06-17 17:22:42 -0400971 path = os.path.join(self.git_root, "org/project")
James E. Blairc6294a52012-08-17 10:19:48 -0700972 repo = git.Repo(path)
973 repo_messages = [c.message.strip() for c in repo.iter_commits(ref)]
974 repo_messages.reverse()
James E. Blairc6294a52012-08-17 10:19:48 -0700975 correct_messages = ['initial commit', 'mp commit', 'A-1', 'B-1', 'C-1']
Monty Taylor98f0f3e2013-07-06 16:02:31 -0400976 self.assertEqual(repo_messages, correct_messages)
James E. Blairc6294a52012-08-17 10:19:48 -0700977
978 def test_build_configuration_branch_interaction(self):
979 "Test that switching between branches works"
980 self.test_build_configuration()
981 self.test_build_configuration_branch()
982 # C has been merged, undo that
Monty Taylorbc758832013-06-17 17:22:42 -0400983 path = os.path.join(self.upstream_root, "org/project")
James E. Blairc6294a52012-08-17 10:19:48 -0700984 repo = git.Repo(path)
985 repo.heads.master.commit = repo.commit('init')
986 self.test_build_configuration()
987
988 def test_build_configuration_multi_branch(self):
989 "Test that dependent changes on multiple branches are merged"
James E. Blair1f4c2bb2013-04-26 08:40:46 -0700990
991 self.gearman_server.hold_jobs_in_queue = True
James E. Blairc6294a52012-08-17 10:19:48 -0700992 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
993 B = self.fake_gerrit.addFakeChange('org/project', 'mp', 'B')
994 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
995 A.addApproval('CRVW', 2)
996 B.addApproval('CRVW', 2)
997 C.addApproval('CRVW', 2)
998 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
999 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
1000 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
1001 self.waitUntilSettled()
James E. Blairbb1fe502014-03-04 10:15:06 -08001002 queue = self.gearman_server.getQueue()
1003 job_A = None
1004 for job in queue:
1005 if 'project-merge' in job.name:
1006 job_A = job
1007 ref_A = self.getParameter(job_A, 'ZUUL_REF')
1008 commit_A = self.getParameter(job_A, 'ZUUL_COMMIT')
1009 self.log.debug("Got Zuul ref for change A: %s" % ref_A)
1010 self.log.debug("Got Zuul commit for change A: %s" % commit_A)
James E. Blairc6294a52012-08-17 10:19:48 -07001011
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001012 self.gearman_server.release('.*-merge')
James E. Blairc6294a52012-08-17 10:19:48 -07001013 self.waitUntilSettled()
James E. Blair701c5b42013-06-06 09:34:59 -07001014 queue = self.gearman_server.getQueue()
James E. Blaird320d7e2013-07-30 16:36:20 -07001015 job_B = None
1016 for job in queue:
1017 if 'project-merge' in job.name:
1018 job_B = job
1019 ref_B = self.getParameter(job_B, 'ZUUL_REF')
James E. Blairbb1fe502014-03-04 10:15:06 -08001020 commit_B = self.getParameter(job_B, 'ZUUL_COMMIT')
James E. Blairf750aa02013-07-15 14:11:24 -07001021 self.log.debug("Got Zuul ref for change B: %s" % ref_B)
James E. Blairbb1fe502014-03-04 10:15:06 -08001022 self.log.debug("Got Zuul commit for change B: %s" % commit_B)
1023
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001024 self.gearman_server.release('.*-merge')
James E. Blairc6294a52012-08-17 10:19:48 -07001025 self.waitUntilSettled()
James E. Blair701c5b42013-06-06 09:34:59 -07001026 queue = self.gearman_server.getQueue()
James E. Blaird320d7e2013-07-30 16:36:20 -07001027 for job in queue:
1028 if 'project-merge' in job.name:
1029 job_C = job
1030 ref_C = self.getParameter(job_C, 'ZUUL_REF')
James E. Blairbb1fe502014-03-04 10:15:06 -08001031 commit_C = self.getParameter(job_C, 'ZUUL_COMMIT')
James E. Blairf750aa02013-07-15 14:11:24 -07001032 self.log.debug("Got Zuul ref for change C: %s" % ref_C)
James E. Blairbb1fe502014-03-04 10:15:06 -08001033 self.log.debug("Got Zuul commit for change C: %s" % commit_C)
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001034 self.gearman_server.hold_jobs_in_queue = False
1035 self.gearman_server.release()
James E. Blairc6294a52012-08-17 10:19:48 -07001036 self.waitUntilSettled()
1037
Monty Taylorbc758832013-06-17 17:22:42 -04001038 path = os.path.join(self.git_root, "org/project")
James E. Blairc6294a52012-08-17 10:19:48 -07001039 repo = git.Repo(path)
1040
1041 repo_messages = [c.message.strip()
James E. Blairf750aa02013-07-15 14:11:24 -07001042 for c in repo.iter_commits(ref_C)]
James E. Blairbb1fe502014-03-04 10:15:06 -08001043 repo_shas = [c.hexsha for c in repo.iter_commits(ref_C)]
James E. Blairc6294a52012-08-17 10:19:48 -07001044 repo_messages.reverse()
James E. Blairc6294a52012-08-17 10:19:48 -07001045 correct_messages = ['initial commit', 'A-1', 'C-1']
James E. Blairbb1fe502014-03-04 10:15:06 -08001046 # Ensure the right commits are in the history for this ref
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001047 self.assertEqual(repo_messages, correct_messages)
James E. Blairbb1fe502014-03-04 10:15:06 -08001048 # Ensure ZUUL_REF -> ZUUL_COMMIT
1049 self.assertEqual(repo_shas[0], commit_C)
James E. Blairc6294a52012-08-17 10:19:48 -07001050
1051 repo_messages = [c.message.strip()
James E. Blairf750aa02013-07-15 14:11:24 -07001052 for c in repo.iter_commits(ref_B)]
James E. Blairbb1fe502014-03-04 10:15:06 -08001053 repo_shas = [c.hexsha for c in repo.iter_commits(ref_B)]
James E. Blairc6294a52012-08-17 10:19:48 -07001054 repo_messages.reverse()
James E. Blairc6294a52012-08-17 10:19:48 -07001055 correct_messages = ['initial commit', 'mp commit', 'B-1']
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001056 self.assertEqual(repo_messages, correct_messages)
James E. Blairbb1fe502014-03-04 10:15:06 -08001057 self.assertEqual(repo_shas[0], commit_B)
1058
1059 repo_messages = [c.message.strip()
1060 for c in repo.iter_commits(ref_A)]
1061 repo_shas = [c.hexsha for c in repo.iter_commits(ref_A)]
1062 repo_messages.reverse()
1063 correct_messages = ['initial commit', 'A-1']
1064 self.assertEqual(repo_messages, correct_messages)
1065 self.assertEqual(repo_shas[0], commit_A)
1066
1067 self.assertNotEqual(ref_A, ref_B, ref_C)
1068 self.assertNotEqual(commit_A, commit_B, commit_C)
James E. Blair7f71c802012-08-22 13:04:32 -07001069
1070 def test_one_job_project(self):
1071 "Test that queueing works with one job"
1072 A = self.fake_gerrit.addFakeChange('org/one-job-project',
1073 'master', 'A')
1074 B = self.fake_gerrit.addFakeChange('org/one-job-project',
1075 'master', 'B')
1076 A.addApproval('CRVW', 2)
1077 B.addApproval('CRVW', 2)
1078 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1079 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
1080 self.waitUntilSettled()
1081
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001082 self.assertEqual(A.data['status'], 'MERGED')
1083 self.assertEqual(A.reported, 2)
1084 self.assertEqual(B.data['status'], 'MERGED')
1085 self.assertEqual(B.reported, 2)
James E. Blaircaec0c52012-08-22 14:52:22 -07001086
Antoine Musso80edd5a2013-02-13 15:37:53 +01001087 def test_job_from_templates_launched(self):
1088 "Test whether a job generated via a template can be launched"
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001089
Antoine Musso80edd5a2013-02-13 15:37:53 +01001090 A = self.fake_gerrit.addFakeChange(
1091 'org/templated-project', 'master', 'A')
1092 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
1093 self.waitUntilSettled()
Antoine Musso80edd5a2013-02-13 15:37:53 +01001094
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001095 self.assertEqual(self.getJobFromHistory('project-test1').result,
1096 'SUCCESS')
1097 self.assertEqual(self.getJobFromHistory('project-test2').result,
1098 'SUCCESS')
Antoine Musso80edd5a2013-02-13 15:37:53 +01001099
James E. Blair3e98c022013-12-16 15:25:38 -08001100 def test_layered_templates(self):
1101 "Test whether a job generated via a template can be launched"
1102
1103 A = self.fake_gerrit.addFakeChange(
1104 'org/layered-project', 'master', 'A')
1105 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
1106 self.waitUntilSettled()
1107
1108 self.assertEqual(self.getJobFromHistory('project-test1').result,
1109 'SUCCESS')
1110 self.assertEqual(self.getJobFromHistory('project-test2').result,
1111 'SUCCESS')
James E. Blairaea6cf62013-12-16 15:38:12 -08001112 self.assertEqual(self.getJobFromHistory('layered-project-test3'
1113 ).result, 'SUCCESS')
1114 self.assertEqual(self.getJobFromHistory('layered-project-test4'
1115 ).result, 'SUCCESS')
James E. Blair12a92b12014-03-26 11:54:53 -07001116 self.assertEqual(self.getJobFromHistory('layered-project-foo-test5'
1117 ).result, 'SUCCESS')
James E. Blair3e98c022013-12-16 15:25:38 -08001118 self.assertEqual(self.getJobFromHistory('project-test6').result,
1119 'SUCCESS')
1120
James E. Blaircaec0c52012-08-22 14:52:22 -07001121 def test_dependent_changes_dequeue(self):
1122 "Test that dependent patches are not needlessly tested"
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001123
James E. Blaircaec0c52012-08-22 14:52:22 -07001124 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
1125 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
1126 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
1127 A.addApproval('CRVW', 2)
1128 B.addApproval('CRVW', 2)
1129 C.addApproval('CRVW', 2)
1130
1131 M1 = self.fake_gerrit.addFakeChange('org/project', 'master', 'M1')
1132 M1.setMerged()
1133
1134 # C -> B -> A -> M1
1135
1136 C.setDependsOn(B, 1)
1137 B.setDependsOn(A, 1)
1138 A.setDependsOn(M1, 1)
1139
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001140 self.worker.addFailTest('project-merge', A)
James E. Blaircaec0c52012-08-22 14:52:22 -07001141
1142 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
1143 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
1144 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1145
1146 self.waitUntilSettled()
1147
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001148 self.assertEqual(A.data['status'], 'NEW')
1149 self.assertEqual(A.reported, 2)
1150 self.assertEqual(B.data['status'], 'NEW')
1151 self.assertEqual(B.reported, 2)
1152 self.assertEqual(C.data['status'], 'NEW')
1153 self.assertEqual(C.reported, 2)
1154 self.assertEqual(len(self.history), 1)
James E. Blairec590122012-08-22 15:19:31 -07001155
James E. Blair972e3c72013-08-29 12:04:55 -07001156 def test_failing_dependent_changes(self):
1157 "Test that failing dependent patches are taken out of stream"
1158 self.worker.hold_jobs_in_build = True
1159 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
1160 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
1161 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
1162 D = self.fake_gerrit.addFakeChange('org/project', 'master', 'D')
1163 E = self.fake_gerrit.addFakeChange('org/project', 'master', 'E')
1164 A.addApproval('CRVW', 2)
1165 B.addApproval('CRVW', 2)
1166 C.addApproval('CRVW', 2)
1167 D.addApproval('CRVW', 2)
1168 E.addApproval('CRVW', 2)
1169
1170 # E, D -> C -> B, A
1171
1172 D.setDependsOn(C, 1)
1173 C.setDependsOn(B, 1)
1174
1175 self.worker.addFailTest('project-test1', B)
1176
1177 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1178 self.fake_gerrit.addEvent(D.addApproval('APRV', 1))
1179 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
1180 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
1181 self.fake_gerrit.addEvent(E.addApproval('APRV', 1))
1182
1183 self.waitUntilSettled()
1184 self.worker.release('.*-merge')
1185 self.waitUntilSettled()
1186 self.worker.release('.*-merge')
1187 self.waitUntilSettled()
1188 self.worker.release('.*-merge')
1189 self.waitUntilSettled()
1190 self.worker.release('.*-merge')
1191 self.waitUntilSettled()
1192 self.worker.release('.*-merge')
1193 self.waitUntilSettled()
1194
1195 self.worker.hold_jobs_in_build = False
1196 for build in self.builds:
1197 if build.parameters['ZUUL_CHANGE'] != '1':
1198 build.release()
1199 self.waitUntilSettled()
1200
1201 self.worker.release()
1202 self.waitUntilSettled()
1203
1204 self.assertEqual(A.data['status'], 'MERGED')
1205 self.assertEqual(A.reported, 2)
Jeremy Stanley10837132014-08-02 16:10:56 +00001206 self.assertIn('Build succeeded', A.messages[1])
James E. Blair972e3c72013-08-29 12:04:55 -07001207 self.assertEqual(B.data['status'], 'NEW')
1208 self.assertEqual(B.reported, 2)
Jeremy Stanley10837132014-08-02 16:10:56 +00001209 self.assertIn('Build failed', B.messages[1])
James E. Blair972e3c72013-08-29 12:04:55 -07001210 self.assertEqual(C.data['status'], 'NEW')
1211 self.assertEqual(C.reported, 2)
Jeremy Stanley10837132014-08-02 16:10:56 +00001212 self.assertIn('depends on a change', C.messages[1])
James E. Blair972e3c72013-08-29 12:04:55 -07001213 self.assertEqual(D.data['status'], 'NEW')
1214 self.assertEqual(D.reported, 2)
Jeremy Stanley10837132014-08-02 16:10:56 +00001215 self.assertIn('depends on a change', D.messages[1])
James E. Blair972e3c72013-08-29 12:04:55 -07001216 self.assertEqual(E.data['status'], 'MERGED')
1217 self.assertEqual(E.reported, 2)
Jeremy Stanley10837132014-08-02 16:10:56 +00001218 self.assertIn('Build succeeded', E.messages[1])
James E. Blair972e3c72013-08-29 12:04:55 -07001219 self.assertEqual(len(self.history), 18)
1220
James E. Blairec590122012-08-22 15:19:31 -07001221 def test_head_is_dequeued_once(self):
James E. Blair2fa50962013-01-30 21:50:41 -08001222 "Test that if a change at the head fails it is dequeued only once"
James E. Blairec590122012-08-22 15:19:31 -07001223 # If it's dequeued more than once, we should see extra
1224 # aborted jobs.
James E. Blairec590122012-08-22 15:19:31 -07001225
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001226 self.worker.hold_jobs_in_build = True
James E. Blairec590122012-08-22 15:19:31 -07001227 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
1228 B = self.fake_gerrit.addFakeChange('org/project1', 'master', 'B')
1229 C = self.fake_gerrit.addFakeChange('org/project1', 'master', 'C')
1230 A.addApproval('CRVW', 2)
1231 B.addApproval('CRVW', 2)
1232 C.addApproval('CRVW', 2)
1233
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001234 self.worker.addFailTest('project1-test1', A)
1235 self.worker.addFailTest('project1-test2', A)
1236 self.worker.addFailTest('project1-project2-integration', A)
James E. Blairec590122012-08-22 15:19:31 -07001237
1238 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1239 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
1240 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
1241
1242 self.waitUntilSettled()
James E. Blairec590122012-08-22 15:19:31 -07001243
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001244 self.assertEqual(len(self.builds), 1)
1245 self.assertEqual(self.builds[0].name, 'project1-merge')
1246 self.assertTrue(self.job_has_changes(self.builds[0], A))
James E. Blairec590122012-08-22 15:19:31 -07001247
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001248 self.worker.release('.*-merge')
James E. Blairec590122012-08-22 15:19:31 -07001249 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001250 self.worker.release('.*-merge')
James E. Blairec590122012-08-22 15:19:31 -07001251 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001252 self.worker.release('.*-merge')
James E. Blairec590122012-08-22 15:19:31 -07001253 self.waitUntilSettled()
1254
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001255 self.assertEqual(len(self.builds), 9)
1256 self.assertEqual(self.builds[0].name, 'project1-test1')
1257 self.assertEqual(self.builds[1].name, 'project1-test2')
1258 self.assertEqual(self.builds[2].name, 'project1-project2-integration')
1259 self.assertEqual(self.builds[3].name, 'project1-test1')
1260 self.assertEqual(self.builds[4].name, 'project1-test2')
1261 self.assertEqual(self.builds[5].name, 'project1-project2-integration')
1262 self.assertEqual(self.builds[6].name, 'project1-test1')
1263 self.assertEqual(self.builds[7].name, 'project1-test2')
1264 self.assertEqual(self.builds[8].name, 'project1-project2-integration')
James E. Blairec590122012-08-22 15:19:31 -07001265
Monty Taylor6bef8ef2013-06-02 08:17:12 -04001266 self.release(self.builds[0])
James E. Blairec590122012-08-22 15:19:31 -07001267 self.waitUntilSettled()
1268
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001269 self.assertEqual(len(self.builds), 3) # test2,integration, merge for B
1270 self.assertEqual(self.countJobResults(self.history, 'ABORTED'), 6)
James E. Blairec590122012-08-22 15:19:31 -07001271
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001272 self.worker.hold_jobs_in_build = False
1273 self.worker.release()
James E. Blairec590122012-08-22 15:19:31 -07001274 self.waitUntilSettled()
1275
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001276 self.assertEqual(len(self.builds), 0)
1277 self.assertEqual(len(self.history), 20)
James E. Blaircaec0c52012-08-22 14:52:22 -07001278
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001279 self.assertEqual(A.data['status'], 'NEW')
1280 self.assertEqual(B.data['status'], 'MERGED')
1281 self.assertEqual(C.data['status'], 'MERGED')
1282 self.assertEqual(A.reported, 2)
1283 self.assertEqual(B.reported, 2)
1284 self.assertEqual(C.reported, 2)
James E. Blair4ec821f2012-08-23 15:28:28 -07001285
1286 def test_nonvoting_job(self):
1287 "Test that non-voting jobs don't vote."
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001288
James E. Blair4ec821f2012-08-23 15:28:28 -07001289 A = self.fake_gerrit.addFakeChange('org/nonvoting-project',
1290 'master', 'A')
1291 A.addApproval('CRVW', 2)
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001292 self.worker.addFailTest('nonvoting-project-test2', A)
James E. Blair4ec821f2012-08-23 15:28:28 -07001293 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1294
1295 self.waitUntilSettled()
James E. Blair4ec821f2012-08-23 15:28:28 -07001296
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001297 self.assertEqual(A.data['status'], 'MERGED')
1298 self.assertEqual(A.reported, 2)
1299 self.assertEqual(
1300 self.getJobFromHistory('nonvoting-project-merge').result,
1301 'SUCCESS')
1302 self.assertEqual(
1303 self.getJobFromHistory('nonvoting-project-test1').result,
1304 'SUCCESS')
1305 self.assertEqual(
1306 self.getJobFromHistory('nonvoting-project-test2').result,
1307 'FAILURE')
James E. Blaire0487072012-08-29 17:38:31 -07001308
James E. Blair5821bd92015-09-16 08:48:15 -07001309 for build in self.builds:
1310 self.assertEqual(build.parameters['ZUUL_VOTING'], '0')
1311
James E. Blaire0487072012-08-29 17:38:31 -07001312 def test_check_queue_success(self):
1313 "Test successful check queue jobs."
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001314
James E. Blaire0487072012-08-29 17:38:31 -07001315 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
1316 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
1317
1318 self.waitUntilSettled()
James E. Blaire0487072012-08-29 17:38:31 -07001319
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001320 self.assertEqual(A.data['status'], 'NEW')
1321 self.assertEqual(A.reported, 1)
1322 self.assertEqual(self.getJobFromHistory('project-merge').result,
1323 'SUCCESS')
1324 self.assertEqual(self.getJobFromHistory('project-test1').result,
1325 'SUCCESS')
1326 self.assertEqual(self.getJobFromHistory('project-test2').result,
1327 'SUCCESS')
James E. Blaire0487072012-08-29 17:38:31 -07001328
1329 def test_check_queue_failure(self):
1330 "Test failed check queue jobs."
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001331
James E. Blaire0487072012-08-29 17:38:31 -07001332 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001333 self.worker.addFailTest('project-test2', A)
James E. Blaire0487072012-08-29 17:38:31 -07001334 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
1335
1336 self.waitUntilSettled()
James E. Blaire0487072012-08-29 17:38:31 -07001337
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001338 self.assertEqual(A.data['status'], 'NEW')
1339 self.assertEqual(A.reported, 1)
1340 self.assertEqual(self.getJobFromHistory('project-merge').result,
James E. Blair78e31b32013-07-09 09:11:34 -07001341 'SUCCESS')
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001342 self.assertEqual(self.getJobFromHistory('project-test1').result,
1343 'SUCCESS')
1344 self.assertEqual(self.getJobFromHistory('project-test2').result,
1345 'FAILURE')
James E. Blair127bc182012-08-28 15:55:15 -07001346
1347 def test_dependent_behind_dequeue(self):
1348 "test that dependent changes behind dequeued changes work"
1349 # This complicated test is a reproduction of a real life bug
1350 self.sched.reconfigure(self.config)
James E. Blair127bc182012-08-28 15:55:15 -07001351
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001352 self.worker.hold_jobs_in_build = True
James E. Blair127bc182012-08-28 15:55:15 -07001353 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
1354 B = self.fake_gerrit.addFakeChange('org/project1', 'master', 'B')
1355 C = self.fake_gerrit.addFakeChange('org/project2', 'master', 'C')
1356 D = self.fake_gerrit.addFakeChange('org/project2', 'master', 'D')
1357 E = self.fake_gerrit.addFakeChange('org/project2', 'master', 'E')
1358 F = self.fake_gerrit.addFakeChange('org/project3', 'master', 'F')
1359 D.setDependsOn(C, 1)
1360 E.setDependsOn(D, 1)
1361 A.addApproval('CRVW', 2)
1362 B.addApproval('CRVW', 2)
1363 C.addApproval('CRVW', 2)
1364 D.addApproval('CRVW', 2)
1365 E.addApproval('CRVW', 2)
1366 F.addApproval('CRVW', 2)
1367
1368 A.fail_merge = True
James E. Blair127bc182012-08-28 15:55:15 -07001369
1370 # Change object re-use in the gerrit trigger is hidden if
1371 # changes are added in quick succession; waiting makes it more
1372 # like real life.
1373 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1374 self.waitUntilSettled()
1375 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
1376 self.waitUntilSettled()
1377
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001378 self.worker.release('.*-merge')
James E. Blair127bc182012-08-28 15:55:15 -07001379 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001380 self.worker.release('.*-merge')
James E. Blair127bc182012-08-28 15:55:15 -07001381 self.waitUntilSettled()
1382
1383 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
1384 self.waitUntilSettled()
1385 self.fake_gerrit.addEvent(D.addApproval('APRV', 1))
1386 self.waitUntilSettled()
1387 self.fake_gerrit.addEvent(E.addApproval('APRV', 1))
1388 self.waitUntilSettled()
1389 self.fake_gerrit.addEvent(F.addApproval('APRV', 1))
1390 self.waitUntilSettled()
1391
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001392 self.worker.release('.*-merge')
James E. Blair127bc182012-08-28 15:55:15 -07001393 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001394 self.worker.release('.*-merge')
James E. Blair127bc182012-08-28 15:55:15 -07001395 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001396 self.worker.release('.*-merge')
James E. Blair127bc182012-08-28 15:55:15 -07001397 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001398 self.worker.release('.*-merge')
James E. Blair127bc182012-08-28 15:55:15 -07001399 self.waitUntilSettled()
1400
1401 # all jobs running
James E. Blaire955e062012-10-08 09:49:03 -07001402
1403 # Grab pointers to the jobs we want to release before
1404 # releasing any, because list indexes may change as
1405 # the jobs complete.
Monty Taylor6bef8ef2013-06-02 08:17:12 -04001406 a, b, c = self.builds[:3]
James E. Blaire955e062012-10-08 09:49:03 -07001407 a.release()
1408 b.release()
1409 c.release()
James E. Blair127bc182012-08-28 15:55:15 -07001410 self.waitUntilSettled()
1411
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001412 self.worker.hold_jobs_in_build = False
1413 self.worker.release()
James E. Blair127bc182012-08-28 15:55:15 -07001414 self.waitUntilSettled()
1415
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001416 self.assertEqual(A.data['status'], 'NEW')
1417 self.assertEqual(B.data['status'], 'MERGED')
1418 self.assertEqual(C.data['status'], 'MERGED')
1419 self.assertEqual(D.data['status'], 'MERGED')
1420 self.assertEqual(E.data['status'], 'MERGED')
1421 self.assertEqual(F.data['status'], 'MERGED')
James E. Blair127bc182012-08-28 15:55:15 -07001422
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001423 self.assertEqual(A.reported, 2)
1424 self.assertEqual(B.reported, 2)
1425 self.assertEqual(C.reported, 2)
1426 self.assertEqual(D.reported, 2)
1427 self.assertEqual(E.reported, 2)
1428 self.assertEqual(F.reported, 2)
James E. Blair127bc182012-08-28 15:55:15 -07001429
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001430 self.assertEqual(self.countJobResults(self.history, 'ABORTED'), 15)
1431 self.assertEqual(len(self.history), 44)
James E. Blair05fed602012-09-07 12:45:24 -07001432
1433 def test_merger_repack(self):
1434 "Test that the merger works after a repack"
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001435
James E. Blair05fed602012-09-07 12:45:24 -07001436 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
1437 A.addApproval('CRVW', 2)
1438 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1439 self.waitUntilSettled()
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001440 self.assertEqual(self.getJobFromHistory('project-merge').result,
1441 'SUCCESS')
1442 self.assertEqual(self.getJobFromHistory('project-test1').result,
1443 'SUCCESS')
1444 self.assertEqual(self.getJobFromHistory('project-test2').result,
1445 'SUCCESS')
1446 self.assertEqual(A.data['status'], 'MERGED')
1447 self.assertEqual(A.reported, 2)
James E. Blair05fed602012-09-07 12:45:24 -07001448 self.assertEmptyQueues()
James E. Blair4ca985f2013-05-30 12:27:43 -07001449 self.worker.build_history = []
James E. Blair05fed602012-09-07 12:45:24 -07001450
Monty Taylorbc758832013-06-17 17:22:42 -04001451 path = os.path.join(self.git_root, "org/project")
1452 print repack_repo(path)
James E. Blair05fed602012-09-07 12:45:24 -07001453
1454 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
1455 A.addApproval('CRVW', 2)
1456 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1457 self.waitUntilSettled()
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001458 self.assertEqual(self.getJobFromHistory('project-merge').result,
1459 'SUCCESS')
1460 self.assertEqual(self.getJobFromHistory('project-test1').result,
1461 'SUCCESS')
1462 self.assertEqual(self.getJobFromHistory('project-test2').result,
1463 'SUCCESS')
1464 self.assertEqual(A.data['status'], 'MERGED')
1465 self.assertEqual(A.reported, 2)
James E. Blair7ee88a22012-09-12 18:59:31 +02001466
James E. Blair4886f282012-11-15 09:27:33 -08001467 def test_merger_repack_large_change(self):
1468 "Test that the merger works with large changes after a repack"
1469 # https://bugs.launchpad.net/zuul/+bug/1078946
James E. Blairac2c3242014-01-24 13:38:51 -08001470 # This test assumes the repo is already cloned; make sure it is
Joshua Hesketh352264b2015-08-11 23:42:08 +10001471 url = self.fake_gerrit.getGitUrl(
James E. Blairac2c3242014-01-24 13:38:51 -08001472 self.sched.layout.projects['org/project1'])
James E. Blair4076e2b2014-01-28 12:42:20 -08001473 self.merge_server.merger.addProject('org/project1', url)
James E. Blair4886f282012-11-15 09:27:33 -08001474 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
1475 A.addPatchset(large=True)
Monty Taylorbc758832013-06-17 17:22:42 -04001476 path = os.path.join(self.upstream_root, "org/project1")
1477 print repack_repo(path)
1478 path = os.path.join(self.git_root, "org/project1")
1479 print repack_repo(path)
James E. Blair4886f282012-11-15 09:27:33 -08001480
1481 A.addApproval('CRVW', 2)
1482 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1483 self.waitUntilSettled()
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001484 self.assertEqual(self.getJobFromHistory('project1-merge').result,
1485 'SUCCESS')
1486 self.assertEqual(self.getJobFromHistory('project1-test1').result,
1487 'SUCCESS')
1488 self.assertEqual(self.getJobFromHistory('project1-test2').result,
1489 'SUCCESS')
1490 self.assertEqual(A.data['status'], 'MERGED')
1491 self.assertEqual(A.reported, 2)
James E. Blair4886f282012-11-15 09:27:33 -08001492
James E. Blair7ee88a22012-09-12 18:59:31 +02001493 def test_nonexistent_job(self):
1494 "Test launching a job that doesn't exist"
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001495 # Set to the state immediately after a restart
1496 self.resetGearmanServer()
1497 self.launcher.negative_function_cache_ttl = 0
James E. Blair7ee88a22012-09-12 18:59:31 +02001498
1499 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
1500 A.addApproval('CRVW', 2)
1501 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1502 # There may be a thread about to report a lost change
1503 while A.reported < 2:
1504 self.waitUntilSettled()
Monty Taylor6bef8ef2013-06-02 08:17:12 -04001505 job_names = [x.name for x in self.history]
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001506 self.assertFalse(job_names)
1507 self.assertEqual(A.data['status'], 'NEW')
1508 self.assertEqual(A.reported, 2)
James E. Blair7ee88a22012-09-12 18:59:31 +02001509 self.assertEmptyQueues()
1510
1511 # Make sure things still work:
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001512 self.registerJobs()
James E. Blair7ee88a22012-09-12 18:59:31 +02001513 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
1514 A.addApproval('CRVW', 2)
1515 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1516 self.waitUntilSettled()
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001517 self.assertEqual(self.getJobFromHistory('project-merge').result,
1518 'SUCCESS')
1519 self.assertEqual(self.getJobFromHistory('project-test1').result,
1520 'SUCCESS')
1521 self.assertEqual(self.getJobFromHistory('project-test2').result,
1522 'SUCCESS')
1523 self.assertEqual(A.data['status'], 'MERGED')
1524 self.assertEqual(A.reported, 2)
James E. Blairf62d4282012-12-31 17:01:50 -08001525
1526 def test_single_nonexistent_post_job(self):
1527 "Test launching a single post job that doesn't exist"
James E. Blairf62d4282012-12-31 17:01:50 -08001528 e = {
1529 "type": "ref-updated",
1530 "submitter": {
1531 "name": "User Name",
1532 },
1533 "refUpdate": {
1534 "oldRev": "90f173846e3af9154517b88543ffbd1691f31366",
1535 "newRev": "d479a0bfcb34da57a31adb2a595c0cf687812543",
1536 "refName": "master",
1537 "project": "org/project",
1538 }
1539 }
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001540 # Set to the state immediately after a restart
1541 self.resetGearmanServer()
1542 self.launcher.negative_function_cache_ttl = 0
1543
James E. Blairf62d4282012-12-31 17:01:50 -08001544 self.fake_gerrit.addEvent(e)
1545 self.waitUntilSettled()
1546
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001547 self.assertEqual(len(self.history), 0)
James E. Blair2fa50962013-01-30 21:50:41 -08001548
1549 def test_new_patchset_dequeues_old(self):
1550 "Test that a new patchset causes the old to be dequeued"
1551 # D -> C (depends on B) -> B (depends on A) -> A -> M
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001552 self.worker.hold_jobs_in_build = True
James E. Blair2fa50962013-01-30 21:50:41 -08001553 M = self.fake_gerrit.addFakeChange('org/project', 'master', 'M')
1554 M.setMerged()
1555
1556 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
1557 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
1558 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
1559 D = self.fake_gerrit.addFakeChange('org/project', 'master', 'D')
1560 A.addApproval('CRVW', 2)
1561 B.addApproval('CRVW', 2)
1562 C.addApproval('CRVW', 2)
1563 D.addApproval('CRVW', 2)
1564
1565 C.setDependsOn(B, 1)
1566 B.setDependsOn(A, 1)
1567 A.setDependsOn(M, 1)
1568
1569 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1570 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
1571 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
1572 self.fake_gerrit.addEvent(D.addApproval('APRV', 1))
1573 self.waitUntilSettled()
1574
1575 B.addPatchset()
1576 self.fake_gerrit.addEvent(B.getPatchsetCreatedEvent(2))
1577 self.waitUntilSettled()
1578
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001579 self.worker.hold_jobs_in_build = False
1580 self.worker.release()
James E. Blair2fa50962013-01-30 21:50:41 -08001581 self.waitUntilSettled()
1582
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001583 self.assertEqual(A.data['status'], 'MERGED')
1584 self.assertEqual(A.reported, 2)
1585 self.assertEqual(B.data['status'], 'NEW')
1586 self.assertEqual(B.reported, 2)
1587 self.assertEqual(C.data['status'], 'NEW')
1588 self.assertEqual(C.reported, 2)
1589 self.assertEqual(D.data['status'], 'MERGED')
1590 self.assertEqual(D.reported, 2)
1591 self.assertEqual(len(self.history), 9) # 3 each for A, B, D.
James E. Blair2fa50962013-01-30 21:50:41 -08001592
James E. Blairba437362015-02-07 11:41:52 -08001593 def test_new_patchset_check(self):
1594 "Test a new patchset in check"
Antoine Mussobd86a312014-01-08 14:51:33 +01001595
1596 self.worker.hold_jobs_in_build = True
1597
1598 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
James E. Blairba437362015-02-07 11:41:52 -08001599 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
1600 check_pipeline = self.sched.layout.pipelines['check']
1601
1602 # Add two git-dependent changes
1603 B.setDependsOn(A, 1)
1604 self.fake_gerrit.addEvent(B.getPatchsetCreatedEvent(1))
1605 self.waitUntilSettled()
Antoine Mussobd86a312014-01-08 14:51:33 +01001606 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
1607 self.waitUntilSettled()
James E. Blairba437362015-02-07 11:41:52 -08001608
1609 # A live item, and a non-live/live pair
1610 items = check_pipeline.getAllItems()
1611 self.assertEqual(len(items), 3)
1612
1613 self.assertEqual(items[0].change.number, '1')
1614 self.assertEqual(items[0].change.patchset, '1')
1615 self.assertFalse(items[0].live)
1616
1617 self.assertEqual(items[1].change.number, '2')
1618 self.assertEqual(items[1].change.patchset, '1')
1619 self.assertTrue(items[1].live)
1620
1621 self.assertEqual(items[2].change.number, '1')
1622 self.assertEqual(items[2].change.patchset, '1')
1623 self.assertTrue(items[2].live)
1624
1625 # Add a new patchset to A
1626 A.addPatchset()
1627 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(2))
1628 self.waitUntilSettled()
1629
1630 # The live copy of A,1 should be gone, but the non-live and B
1631 # should continue, and we should have a new A,2
1632 items = check_pipeline.getAllItems()
1633 self.assertEqual(len(items), 3)
1634
1635 self.assertEqual(items[0].change.number, '1')
1636 self.assertEqual(items[0].change.patchset, '1')
1637 self.assertFalse(items[0].live)
1638
1639 self.assertEqual(items[1].change.number, '2')
1640 self.assertEqual(items[1].change.patchset, '1')
1641 self.assertTrue(items[1].live)
1642
1643 self.assertEqual(items[2].change.number, '1')
1644 self.assertEqual(items[2].change.patchset, '2')
1645 self.assertTrue(items[2].live)
1646
1647 # Add a new patchset to B
1648 B.addPatchset()
1649 self.fake_gerrit.addEvent(B.getPatchsetCreatedEvent(2))
1650 self.waitUntilSettled()
1651
1652 # The live copy of B,1 should be gone, and it's non-live copy of A,1
1653 # but we should have a new B,2 (still based on A,1)
1654 items = check_pipeline.getAllItems()
1655 self.assertEqual(len(items), 3)
1656
1657 self.assertEqual(items[0].change.number, '1')
1658 self.assertEqual(items[0].change.patchset, '2')
1659 self.assertTrue(items[0].live)
1660
1661 self.assertEqual(items[1].change.number, '1')
1662 self.assertEqual(items[1].change.patchset, '1')
1663 self.assertFalse(items[1].live)
1664
1665 self.assertEqual(items[2].change.number, '2')
1666 self.assertEqual(items[2].change.patchset, '2')
1667 self.assertTrue(items[2].live)
1668
1669 self.builds[0].release()
1670 self.waitUntilSettled()
1671 self.builds[0].release()
1672 self.waitUntilSettled()
1673 self.worker.hold_jobs_in_build = False
1674 self.worker.release()
1675 self.waitUntilSettled()
1676
1677 self.assertEqual(A.reported, 1)
1678 self.assertEqual(B.reported, 1)
1679 self.assertEqual(self.history[0].result, 'ABORTED')
1680 self.assertEqual(self.history[0].changes, '1,1')
1681 self.assertEqual(self.history[1].result, 'ABORTED')
1682 self.assertEqual(self.history[1].changes, '1,1 2,1')
1683 self.assertEqual(self.history[2].result, 'SUCCESS')
1684 self.assertEqual(self.history[2].changes, '1,2')
1685 self.assertEqual(self.history[3].result, 'SUCCESS')
1686 self.assertEqual(self.history[3].changes, '1,1 2,2')
1687
1688 def test_abandoned_gate(self):
1689 "Test that an abandoned change is dequeued from gate"
1690
1691 self.worker.hold_jobs_in_build = True
1692
1693 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
1694 A.addApproval('CRVW', 2)
1695 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1696 self.waitUntilSettled()
Antoine Mussobd86a312014-01-08 14:51:33 +01001697 self.assertEqual(len(self.builds), 1, "One job being built (on hold)")
1698 self.assertEqual(self.builds[0].name, 'project-merge')
1699
1700 self.fake_gerrit.addEvent(A.getChangeAbandonedEvent())
1701 self.waitUntilSettled()
1702
Antoine Mussobd86a312014-01-08 14:51:33 +01001703 self.worker.release('.*-merge')
1704 self.waitUntilSettled()
1705
1706 self.assertEqual(len(self.builds), 0, "No job running")
Antoine Mussobd86a312014-01-08 14:51:33 +01001707 self.assertEqual(len(self.history), 1, "Only one build in history")
1708 self.assertEqual(self.history[0].result, 'ABORTED',
James E. Blairba437362015-02-07 11:41:52 -08001709 "Build should have been aborted")
1710 self.assertEqual(A.reported, 1,
1711 "Abandoned gate change should report only start")
1712
1713 def test_abandoned_check(self):
1714 "Test that an abandoned change is dequeued from check"
1715
1716 self.worker.hold_jobs_in_build = True
1717
1718 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
1719 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
1720 check_pipeline = self.sched.layout.pipelines['check']
1721
1722 # Add two git-dependent changes
1723 B.setDependsOn(A, 1)
1724 self.fake_gerrit.addEvent(B.getPatchsetCreatedEvent(1))
1725 self.waitUntilSettled()
1726 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
1727 self.waitUntilSettled()
1728 # A live item, and a non-live/live pair
1729 items = check_pipeline.getAllItems()
1730 self.assertEqual(len(items), 3)
1731
1732 self.assertEqual(items[0].change.number, '1')
1733 self.assertFalse(items[0].live)
1734
1735 self.assertEqual(items[1].change.number, '2')
1736 self.assertTrue(items[1].live)
1737
1738 self.assertEqual(items[2].change.number, '1')
1739 self.assertTrue(items[2].live)
1740
1741 # Abandon A
1742 self.fake_gerrit.addEvent(A.getChangeAbandonedEvent())
1743 self.waitUntilSettled()
1744
1745 # The live copy of A should be gone, but the non-live and B
1746 # should continue
1747 items = check_pipeline.getAllItems()
1748 self.assertEqual(len(items), 2)
1749
1750 self.assertEqual(items[0].change.number, '1')
1751 self.assertFalse(items[0].live)
1752
1753 self.assertEqual(items[1].change.number, '2')
1754 self.assertTrue(items[1].live)
1755
1756 self.worker.hold_jobs_in_build = False
1757 self.worker.release()
1758 self.waitUntilSettled()
1759
1760 self.assertEqual(len(self.history), 4)
1761 self.assertEqual(self.history[0].result, 'ABORTED',
Antoine Mussobd86a312014-01-08 14:51:33 +01001762 'Build should have been aborted')
1763 self.assertEqual(A.reported, 0, "Abandoned change should not report")
James E. Blairba437362015-02-07 11:41:52 -08001764 self.assertEqual(B.reported, 1, "Change should report")
Antoine Mussobd86a312014-01-08 14:51:33 +01001765
Steve Varnau7b78b312015-04-03 14:49:46 -07001766 def test_abandoned_not_timer(self):
1767 "Test that an abandoned change does not cancel timer jobs"
1768
1769 self.worker.hold_jobs_in_build = True
1770
1771 # Start timer trigger - also org/project
1772 self.config.set('zuul', 'layout_config',
1773 'tests/fixtures/layout-idle.yaml')
1774 self.sched.reconfigure(self.config)
1775 self.registerJobs()
1776 # The pipeline triggers every second, so we should have seen
1777 # several by now.
1778 time.sleep(5)
1779 self.waitUntilSettled()
1780 # Stop queuing timer triggered jobs so that the assertions
1781 # below don't race against more jobs being queued.
1782 self.config.set('zuul', 'layout_config',
1783 'tests/fixtures/layout-no-timer.yaml')
1784 self.sched.reconfigure(self.config)
1785 self.registerJobs()
1786 self.assertEqual(len(self.builds), 2, "Two timer jobs")
1787
1788 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
1789 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
1790 self.waitUntilSettled()
1791 self.assertEqual(len(self.builds), 3, "One change plus two timer jobs")
1792
1793 self.fake_gerrit.addEvent(A.getChangeAbandonedEvent())
1794 self.waitUntilSettled()
1795
1796 self.assertEqual(len(self.builds), 2, "Two timer jobs remain")
1797
1798 self.worker.release()
1799 self.waitUntilSettled()
1800
Arx Cruzb1b010d2013-10-28 19:49:59 -02001801 def test_zuul_url_return(self):
1802 "Test if ZUUL_URL is returning when zuul_url is set in zuul.conf"
James E. Blair4076e2b2014-01-28 12:42:20 -08001803 self.assertTrue(self.sched.config.has_option('merger', 'zuul_url'))
Arx Cruzb1b010d2013-10-28 19:49:59 -02001804 self.worker.hold_jobs_in_build = True
1805
1806 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
1807 A.addApproval('CRVW', 2)
1808 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1809 self.waitUntilSettled()
1810
1811 self.assertEqual(len(self.builds), 1)
1812 for build in self.builds:
1813 self.assertTrue('ZUUL_URL' in build.parameters)
1814
1815 self.worker.hold_jobs_in_build = False
1816 self.worker.release()
1817 self.waitUntilSettled()
1818
James E. Blair2fa50962013-01-30 21:50:41 -08001819 def test_new_patchset_dequeues_old_on_head(self):
1820 "Test that a new patchset causes the old to be dequeued (at head)"
1821 # D -> C (depends on B) -> B (depends on A) -> A -> M
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001822 self.worker.hold_jobs_in_build = True
James E. Blair2fa50962013-01-30 21:50:41 -08001823 M = self.fake_gerrit.addFakeChange('org/project', 'master', 'M')
1824 M.setMerged()
1825 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
1826 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
1827 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
1828 D = self.fake_gerrit.addFakeChange('org/project', 'master', 'D')
1829 A.addApproval('CRVW', 2)
1830 B.addApproval('CRVW', 2)
1831 C.addApproval('CRVW', 2)
1832 D.addApproval('CRVW', 2)
1833
1834 C.setDependsOn(B, 1)
1835 B.setDependsOn(A, 1)
1836 A.setDependsOn(M, 1)
1837
1838 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1839 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
1840 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
1841 self.fake_gerrit.addEvent(D.addApproval('APRV', 1))
1842 self.waitUntilSettled()
1843
1844 A.addPatchset()
1845 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(2))
1846 self.waitUntilSettled()
1847
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001848 self.worker.hold_jobs_in_build = False
1849 self.worker.release()
James E. Blair2fa50962013-01-30 21:50:41 -08001850 self.waitUntilSettled()
1851
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001852 self.assertEqual(A.data['status'], 'NEW')
1853 self.assertEqual(A.reported, 2)
1854 self.assertEqual(B.data['status'], 'NEW')
1855 self.assertEqual(B.reported, 2)
1856 self.assertEqual(C.data['status'], 'NEW')
1857 self.assertEqual(C.reported, 2)
1858 self.assertEqual(D.data['status'], 'MERGED')
1859 self.assertEqual(D.reported, 2)
1860 self.assertEqual(len(self.history), 7)
James E. Blair2fa50962013-01-30 21:50:41 -08001861
1862 def test_new_patchset_dequeues_old_without_dependents(self):
1863 "Test that a new patchset causes only the old to be dequeued"
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001864 self.worker.hold_jobs_in_build = True
James E. Blair2fa50962013-01-30 21:50:41 -08001865 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
1866 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
1867 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
1868 A.addApproval('CRVW', 2)
1869 B.addApproval('CRVW', 2)
1870 C.addApproval('CRVW', 2)
1871
1872 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
1873 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
1874 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1875 self.waitUntilSettled()
1876
1877 B.addPatchset()
1878 self.fake_gerrit.addEvent(B.getPatchsetCreatedEvent(2))
1879 self.waitUntilSettled()
1880
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001881 self.worker.hold_jobs_in_build = False
1882 self.worker.release()
James E. Blair2fa50962013-01-30 21:50:41 -08001883 self.waitUntilSettled()
1884
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001885 self.assertEqual(A.data['status'], 'MERGED')
1886 self.assertEqual(A.reported, 2)
1887 self.assertEqual(B.data['status'], 'NEW')
1888 self.assertEqual(B.reported, 2)
1889 self.assertEqual(C.data['status'], 'MERGED')
1890 self.assertEqual(C.reported, 2)
1891 self.assertEqual(len(self.history), 9)
James E. Blair2fa50962013-01-30 21:50:41 -08001892
1893 def test_new_patchset_dequeues_old_independent_queue(self):
1894 "Test that a new patchset causes the old to be dequeued (independent)"
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001895 self.worker.hold_jobs_in_build = True
James E. Blair2fa50962013-01-30 21:50:41 -08001896 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
1897 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
1898 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
1899 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
1900 self.fake_gerrit.addEvent(B.getPatchsetCreatedEvent(1))
1901 self.fake_gerrit.addEvent(C.getPatchsetCreatedEvent(1))
1902 self.waitUntilSettled()
1903
1904 B.addPatchset()
1905 self.fake_gerrit.addEvent(B.getPatchsetCreatedEvent(2))
1906 self.waitUntilSettled()
1907
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001908 self.worker.hold_jobs_in_build = False
1909 self.worker.release()
James E. Blair2fa50962013-01-30 21:50:41 -08001910 self.waitUntilSettled()
1911
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001912 self.assertEqual(A.data['status'], 'NEW')
1913 self.assertEqual(A.reported, 1)
1914 self.assertEqual(B.data['status'], 'NEW')
1915 self.assertEqual(B.reported, 1)
1916 self.assertEqual(C.data['status'], 'NEW')
1917 self.assertEqual(C.reported, 1)
1918 self.assertEqual(len(self.history), 10)
1919 self.assertEqual(self.countJobResults(self.history, 'ABORTED'), 1)
James E. Blair7d0dedc2013-02-21 17:26:09 -08001920
James E. Blair18c64442014-03-18 10:14:45 -07001921 def test_noop_job(self):
1922 "Test that the internal noop job works"
1923 A = self.fake_gerrit.addFakeChange('org/noop-project', 'master', 'A')
1924 A.addApproval('CRVW', 2)
1925 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1926 self.waitUntilSettled()
1927
1928 self.assertEqual(len(self.gearman_server.getQueue()), 0)
1929 self.assertTrue(self.sched._areAllBuildsComplete())
1930 self.assertEqual(len(self.history), 0)
1931 self.assertEqual(A.data['status'], 'MERGED')
1932 self.assertEqual(A.reported, 2)
1933
Evgeny Antyshevd6e546c2015-06-11 15:13:57 +00001934 def test_no_job_project(self):
1935 "Test that reports with no jobs don't get sent"
1936 A = self.fake_gerrit.addFakeChange('org/no-jobs-project',
1937 'master', 'A')
1938 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
1939 self.waitUntilSettled()
1940
1941 # Change wasn't reported to
1942 self.assertEqual(A.reported, False)
1943
1944 # Check queue is empty afterwards
1945 check_pipeline = self.sched.layout.pipelines['check']
1946 items = check_pipeline.getAllItems()
1947 self.assertEqual(len(items), 0)
1948
1949 self.assertEqual(len(self.history), 0)
1950
James E. Blair7d0dedc2013-02-21 17:26:09 -08001951 def test_zuul_refs(self):
1952 "Test that zuul refs exist and have the right changes"
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001953 self.worker.hold_jobs_in_build = True
James E. Blair7d0dedc2013-02-21 17:26:09 -08001954 M1 = self.fake_gerrit.addFakeChange('org/project1', 'master', 'M1')
1955 M1.setMerged()
1956 M2 = self.fake_gerrit.addFakeChange('org/project2', 'master', 'M2')
1957 M2.setMerged()
1958
1959 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
1960 B = self.fake_gerrit.addFakeChange('org/project1', 'master', 'B')
1961 C = self.fake_gerrit.addFakeChange('org/project2', 'master', 'C')
1962 D = self.fake_gerrit.addFakeChange('org/project2', 'master', 'D')
1963 A.addApproval('CRVW', 2)
1964 B.addApproval('CRVW', 2)
1965 C.addApproval('CRVW', 2)
1966 D.addApproval('CRVW', 2)
1967 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
1968 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
1969 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
1970 self.fake_gerrit.addEvent(D.addApproval('APRV', 1))
1971
1972 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001973 self.worker.release('.*-merge')
James E. Blair7d0dedc2013-02-21 17:26:09 -08001974 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001975 self.worker.release('.*-merge')
James E. Blair7d0dedc2013-02-21 17:26:09 -08001976 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001977 self.worker.release('.*-merge')
James E. Blair7d0dedc2013-02-21 17:26:09 -08001978 self.waitUntilSettled()
James E. Blair1f4c2bb2013-04-26 08:40:46 -07001979 self.worker.release('.*-merge')
James E. Blair7d0dedc2013-02-21 17:26:09 -08001980 self.waitUntilSettled()
1981
James E. Blair7d0dedc2013-02-21 17:26:09 -08001982 a_zref = b_zref = c_zref = d_zref = None
Monty Taylor6bef8ef2013-06-02 08:17:12 -04001983 for x in self.builds:
James E. Blair7d0dedc2013-02-21 17:26:09 -08001984 if x.parameters['ZUUL_CHANGE'] == '3':
1985 a_zref = x.parameters['ZUUL_REF']
1986 if x.parameters['ZUUL_CHANGE'] == '4':
1987 b_zref = x.parameters['ZUUL_REF']
1988 if x.parameters['ZUUL_CHANGE'] == '5':
1989 c_zref = x.parameters['ZUUL_REF']
1990 if x.parameters['ZUUL_CHANGE'] == '6':
1991 d_zref = x.parameters['ZUUL_REF']
1992
1993 # There are... four... refs.
Monty Taylor98f0f3e2013-07-06 16:02:31 -04001994 self.assertIsNotNone(a_zref)
1995 self.assertIsNotNone(b_zref)
1996 self.assertIsNotNone(c_zref)
1997 self.assertIsNotNone(d_zref)
James E. Blair7d0dedc2013-02-21 17:26:09 -08001998
1999 # And they should all be different
2000 refs = set([a_zref, b_zref, c_zref, d_zref])
Monty Taylor98f0f3e2013-07-06 16:02:31 -04002001 self.assertEqual(len(refs), 4)
James E. Blair7d0dedc2013-02-21 17:26:09 -08002002
2003 # a ref should have a, not b, and should not be in project2
Monty Taylor98f0f3e2013-07-06 16:02:31 -04002004 self.assertTrue(self.ref_has_change(a_zref, A))
2005 self.assertFalse(self.ref_has_change(a_zref, B))
2006 self.assertFalse(self.ref_has_change(a_zref, M2))
James E. Blair7d0dedc2013-02-21 17:26:09 -08002007
2008 # b ref should have a and b, and should not be in project2
Monty Taylor98f0f3e2013-07-06 16:02:31 -04002009 self.assertTrue(self.ref_has_change(b_zref, A))
2010 self.assertTrue(self.ref_has_change(b_zref, B))
2011 self.assertFalse(self.ref_has_change(b_zref, M2))
James E. Blair7d0dedc2013-02-21 17:26:09 -08002012
2013 # c ref should have a and b in 1, c in 2
Monty Taylor98f0f3e2013-07-06 16:02:31 -04002014 self.assertTrue(self.ref_has_change(c_zref, A))
2015 self.assertTrue(self.ref_has_change(c_zref, B))
2016 self.assertTrue(self.ref_has_change(c_zref, C))
2017 self.assertFalse(self.ref_has_change(c_zref, D))
James E. Blair7d0dedc2013-02-21 17:26:09 -08002018
2019 # d ref should have a and b in 1, c and d in 2
Monty Taylor98f0f3e2013-07-06 16:02:31 -04002020 self.assertTrue(self.ref_has_change(d_zref, A))
2021 self.assertTrue(self.ref_has_change(d_zref, B))
2022 self.assertTrue(self.ref_has_change(d_zref, C))
2023 self.assertTrue(self.ref_has_change(d_zref, D))
James E. Blair7d0dedc2013-02-21 17:26:09 -08002024
James E. Blair1f4c2bb2013-04-26 08:40:46 -07002025 self.worker.hold_jobs_in_build = False
2026 self.worker.release()
James E. Blair7d0dedc2013-02-21 17:26:09 -08002027 self.waitUntilSettled()
2028
Monty Taylor98f0f3e2013-07-06 16:02:31 -04002029 self.assertEqual(A.data['status'], 'MERGED')
2030 self.assertEqual(A.reported, 2)
2031 self.assertEqual(B.data['status'], 'MERGED')
2032 self.assertEqual(B.reported, 2)
2033 self.assertEqual(C.data['status'], 'MERGED')
2034 self.assertEqual(C.reported, 2)
2035 self.assertEqual(D.data['status'], 'MERGED')
2036 self.assertEqual(D.reported, 2)
James E. Blair70c71582013-03-06 08:50:50 -08002037
James E. Blair4a28a882013-08-23 15:17:33 -07002038 def test_rerun_on_error(self):
2039 "Test that if a worker fails to run a job, it is run again"
2040 self.worker.hold_jobs_in_build = True
2041 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
2042 A.addApproval('CRVW', 2)
2043 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
2044 self.waitUntilSettled()
2045
2046 self.builds[0].run_error = True
2047 self.worker.hold_jobs_in_build = False
2048 self.worker.release()
2049 self.waitUntilSettled()
2050 self.assertEqual(self.countJobResults(self.history, 'RUN_ERROR'), 1)
2051 self.assertEqual(self.countJobResults(self.history, 'SUCCESS'), 3)
2052
James E. Blair412e5582013-04-22 15:50:12 -07002053 def test_statsd(self):
2054 "Test each of the statsd methods used in the scheduler"
2055 import extras
2056 statsd = extras.try_import('statsd.statsd')
2057 statsd.incr('test-incr')
2058 statsd.timing('test-timing', 3)
Alex Gaynor813d39b2014-05-17 16:17:16 -07002059 statsd.gauge('test-gauge', 12)
James E. Blair412e5582013-04-22 15:50:12 -07002060 self.assertReportedStat('test-incr', '1|c')
2061 self.assertReportedStat('test-timing', '3|ms')
Alex Gaynor813d39b2014-05-17 16:17:16 -07002062 self.assertReportedStat('test-gauge', '12|g')
James E. Blair412e5582013-04-22 15:50:12 -07002063
James E. Blairdad52252014-02-07 16:59:17 -08002064 def test_stuck_job_cleanup(self):
2065 "Test that pending jobs are cleaned up if removed from layout"
James E. Blair18c64442014-03-18 10:14:45 -07002066 # This job won't be registered at startup because it is not in
2067 # the standard layout, but we need it to already be registerd
2068 # for when we reconfigure, as that is when Zuul will attempt
2069 # to run the new job.
2070 self.worker.registerFunction('build:gate-noop')
James E. Blairdad52252014-02-07 16:59:17 -08002071 self.gearman_server.hold_jobs_in_queue = True
2072 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
2073 A.addApproval('CRVW', 2)
2074 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
2075 self.waitUntilSettled()
2076 self.assertEqual(len(self.gearman_server.getQueue()), 1)
2077
2078 self.config.set('zuul', 'layout_config',
2079 'tests/fixtures/layout-no-jobs.yaml')
2080 self.sched.reconfigure(self.config)
2081 self.waitUntilSettled()
2082
James E. Blair18c64442014-03-18 10:14:45 -07002083 self.gearman_server.release('gate-noop')
James E. Blairdad52252014-02-07 16:59:17 -08002084 self.waitUntilSettled()
2085 self.assertEqual(len(self.gearman_server.getQueue()), 0)
2086 self.assertTrue(self.sched._areAllBuildsComplete())
2087
2088 self.assertEqual(len(self.history), 1)
James E. Blair18c64442014-03-18 10:14:45 -07002089 self.assertEqual(self.history[0].name, 'gate-noop')
James E. Blairdad52252014-02-07 16:59:17 -08002090 self.assertEqual(self.history[0].result, 'SUCCESS')
2091
James E. Blair879dafb2015-07-17 14:04:49 -07002092 def test_file_head(self):
2093 # This is a regression test for an observed bug. A change
2094 # with a file named "HEAD" in the root directory of the repo
2095 # was processed by a merger. It then was unable to reset the
2096 # repo because of:
2097 # GitCommandError: 'git reset --hard HEAD' returned
2098 # with exit code 128
2099 # stderr: 'fatal: ambiguous argument 'HEAD': both revision
2100 # and filename
2101 # Use '--' to separate filenames from revisions'
2102
2103 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
2104 A.addPatchset(['HEAD'])
2105 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
2106
2107 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(2))
2108 self.waitUntilSettled()
2109
2110 self.fake_gerrit.addEvent(B.getPatchsetCreatedEvent(1))
2111 self.waitUntilSettled()
2112
2113 self.assertIn('Build succeeded', A.messages[0])
2114 self.assertIn('Build succeeded', B.messages[0])
2115
James E. Blair70c71582013-03-06 08:50:50 -08002116 def test_file_jobs(self):
2117 "Test that file jobs run only when appropriate"
2118 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
2119 A.addPatchset(['pip-requires'])
2120 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
2121 A.addApproval('CRVW', 2)
2122 B.addApproval('CRVW', 2)
2123 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
2124 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
2125 self.waitUntilSettled()
2126
Monty Taylor6bef8ef2013-06-02 08:17:12 -04002127 testfile_jobs = [x for x in self.history
James E. Blair70c71582013-03-06 08:50:50 -08002128 if x.name == 'project-testfile']
2129
Monty Taylor98f0f3e2013-07-06 16:02:31 -04002130 self.assertEqual(len(testfile_jobs), 1)
2131 self.assertEqual(testfile_jobs[0].changes, '1,2')
2132 self.assertEqual(A.data['status'], 'MERGED')
2133 self.assertEqual(A.reported, 2)
2134 self.assertEqual(B.data['status'], 'MERGED')
2135 self.assertEqual(B.reported, 2)
James E. Blair3c5e5b52013-04-26 11:17:03 -07002136
Maru Newby3fe5f852015-01-13 04:22:14 +00002137 def _test_skip_if_jobs(self, branch, should_skip):
2138 "Test that jobs with a skip-if filter run only when appropriate"
2139 self.config.set('zuul', 'layout_config',
2140 'tests/fixtures/layout-skip-if.yaml')
2141 self.sched.reconfigure(self.config)
2142 self.registerJobs()
2143
2144 change = self.fake_gerrit.addFakeChange('org/project',
2145 branch,
2146 'test skip-if')
2147 self.fake_gerrit.addEvent(change.getPatchsetCreatedEvent(1))
2148 self.waitUntilSettled()
2149
2150 tested_change_ids = [x.changes[0] for x in self.history
2151 if x.name == 'project-test-skip-if']
2152
2153 if should_skip:
2154 self.assertEqual([], tested_change_ids)
2155 else:
2156 self.assertIn(change.data['number'], tested_change_ids)
2157
2158 def test_skip_if_match_skips_job(self):
2159 self._test_skip_if_jobs(branch='master', should_skip=True)
2160
2161 def test_skip_if_no_match_runs_job(self):
2162 self._test_skip_if_jobs(branch='mp', should_skip=False)
2163
James E. Blair3c5e5b52013-04-26 11:17:03 -07002164 def test_test_config(self):
2165 "Test that we can test the config"
Joshua Hesketh352264b2015-08-11 23:42:08 +10002166 self.sched.testConfig(self.config.get('zuul', 'layout_config'),
2167 self.connections)
James E. Blair1f4c2bb2013-04-26 08:40:46 -07002168
2169 def test_build_description(self):
2170 "Test that build descriptions update"
James E. Blair1f4c2bb2013-04-26 08:40:46 -07002171 self.worker.registerFunction('set_description:' +
2172 self.worker.worker_id)
2173
2174 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
2175 A.addApproval('CRVW', 2)
2176 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
2177 self.waitUntilSettled()
Monty Taylor6bef8ef2013-06-02 08:17:12 -04002178 desc = self.history[0].description
James E. Blair1f4c2bb2013-04-26 08:40:46 -07002179 self.log.debug("Description: %s" % desc)
Monty Taylor98f0f3e2013-07-06 16:02:31 -04002180 self.assertTrue(re.search("Branch.*master", desc))
2181 self.assertTrue(re.search("Pipeline.*gate", desc))
2182 self.assertTrue(re.search("project-merge.*SUCCESS", desc))
2183 self.assertTrue(re.search("project-test1.*SUCCESS", desc))
2184 self.assertTrue(re.search("project-test2.*SUCCESS", desc))
2185 self.assertTrue(re.search("Reported result.*SUCCESS", desc))
James E. Blair1f4c2bb2013-04-26 08:40:46 -07002186
James E. Blairc8a1e052014-02-25 09:29:26 -08002187 def test_queue_names(self):
2188 "Test shared change queue names"
2189 project1 = self.sched.layout.projects['org/project1']
2190 project2 = self.sched.layout.projects['org/project2']
2191 q1 = self.sched.layout.pipelines['gate'].getQueue(project1)
2192 q2 = self.sched.layout.pipelines['gate'].getQueue(project2)
2193 self.assertEqual(q1.name, 'integration')
2194 self.assertEqual(q2.name, 'integration')
2195
2196 self.config.set('zuul', 'layout_config',
2197 'tests/fixtures/layout-bad-queue.yaml')
2198 with testtools.ExpectedException(
2199 Exception, "More than one name assigned to change queue"):
2200 self.sched.reconfigure(self.config)
2201
James E. Blair64ed6f22013-07-10 14:07:23 -07002202 def test_queue_precedence(self):
2203 "Test that queue precedence works"
2204
2205 self.gearman_server.hold_jobs_in_queue = True
James E. Blair8de58bd2013-07-18 16:23:33 -07002206 self.worker.hold_jobs_in_build = True
James E. Blair64ed6f22013-07-10 14:07:23 -07002207 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
2208 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
2209 A.addApproval('CRVW', 2)
2210 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
2211
2212 self.waitUntilSettled()
2213 self.gearman_server.hold_jobs_in_queue = False
2214 self.gearman_server.release()
2215 self.waitUntilSettled()
2216
James E. Blair8de58bd2013-07-18 16:23:33 -07002217 # Run one build at a time to ensure non-race order:
James E. Blairb8c16472015-05-05 14:55:26 -07002218 self.orderedRelease()
James E. Blair8de58bd2013-07-18 16:23:33 -07002219 self.worker.hold_jobs_in_build = False
2220 self.waitUntilSettled()
2221
James E. Blair64ed6f22013-07-10 14:07:23 -07002222 self.log.debug(self.history)
2223 self.assertEqual(self.history[0].pipeline, 'gate')
2224 self.assertEqual(self.history[1].pipeline, 'check')
2225 self.assertEqual(self.history[2].pipeline, 'gate')
2226 self.assertEqual(self.history[3].pipeline, 'gate')
2227 self.assertEqual(self.history[4].pipeline, 'check')
2228 self.assertEqual(self.history[5].pipeline, 'check')
2229
Clark Boylana5edbe42014-06-03 16:39:10 -07002230 def test_json_status(self):
James E. Blair1843a552013-07-03 14:19:52 -07002231 "Test that we can retrieve JSON status info"
2232 self.worker.hold_jobs_in_build = True
2233 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
2234 A.addApproval('CRVW', 2)
2235 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
2236 self.waitUntilSettled()
2237
2238 port = self.webapp.server.socket.getsockname()[1]
2239
Yuriy Taradaya6d452f2014-04-16 12:36:20 +04002240 req = urllib2.Request("http://localhost:%s/status.json" % port)
Yuriy Taradaya6d452f2014-04-16 12:36:20 +04002241 f = urllib2.urlopen(req)
Clark Boylanaa4f2e72014-06-03 21:22:40 -07002242 headers = f.info()
2243 self.assertIn('Content-Length', headers)
2244 self.assertIn('Content-Type', headers)
Sachi Kingdc963fc2016-03-23 16:00:33 +11002245 self.assertIsNotNone(re.match('^application/json(; charset=UTF-8)?$',
2246 headers['Content-Type']))
Timo Tijhof0ebd2932015-04-02 12:11:21 +01002247 self.assertIn('Access-Control-Allow-Origin', headers)
2248 self.assertIn('Cache-Control', headers)
Clark Boylanaa4f2e72014-06-03 21:22:40 -07002249 self.assertIn('Last-Modified', headers)
Timo Tijhof0ebd2932015-04-02 12:11:21 +01002250 self.assertIn('Expires', headers)
James E. Blair1843a552013-07-03 14:19:52 -07002251 data = f.read()
2252
2253 self.worker.hold_jobs_in_build = False
2254 self.worker.release()
2255 self.waitUntilSettled()
2256
2257 data = json.loads(data)
2258 status_jobs = set()
2259 for p in data['pipelines']:
2260 for q in p['change_queues']:
James E. Blairbfb8e042014-12-30 17:01:44 -08002261 if p['name'] in ['gate', 'conflict']:
Clark Boylanaf2476f2014-01-23 14:47:36 -08002262 self.assertEqual(q['window'], 20)
2263 else:
2264 self.assertEqual(q['window'], 0)
James E. Blair1843a552013-07-03 14:19:52 -07002265 for head in q['heads']:
2266 for change in head:
Clark Boylanaf2476f2014-01-23 14:47:36 -08002267 self.assertTrue(change['active'])
Monty Taylor98f0f3e2013-07-06 16:02:31 -04002268 self.assertEqual(change['id'], '1,1')
James E. Blair1843a552013-07-03 14:19:52 -07002269 for job in change['jobs']:
2270 status_jobs.add(job['name'])
Monty Taylor98f0f3e2013-07-06 16:02:31 -04002271 self.assertIn('project-merge', status_jobs)
2272 self.assertIn('project-test1', status_jobs)
2273 self.assertIn('project-test2', status_jobs)
James E. Blair1843a552013-07-03 14:19:52 -07002274
James E. Blairc3d428e2013-12-03 15:06:48 -08002275 def test_merging_queues(self):
2276 "Test that transitively-connected change queues are merged"
2277 self.config.set('zuul', 'layout_config',
2278 'tests/fixtures/layout-merge-queues.yaml')
2279 self.sched.reconfigure(self.config)
2280 self.assertEqual(len(self.sched.layout.pipelines['gate'].queues), 1)
2281
James E. Blairaf17a972016-02-03 15:07:18 -08002282 def test_mutex(self):
2283 "Test job mutexes"
2284 self.config.set('zuul', 'layout_config',
2285 'tests/fixtures/layout-mutex.yaml')
2286 self.sched.reconfigure(self.config)
2287
2288 self.worker.hold_jobs_in_build = True
2289 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
2290 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
2291 self.assertFalse('test-mutex' in self.sched.mutex.mutexes)
2292
2293 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
2294 self.fake_gerrit.addEvent(B.getPatchsetCreatedEvent(1))
2295 self.waitUntilSettled()
2296 self.assertEqual(len(self.builds), 3)
2297 self.assertEqual(self.builds[0].name, 'project-test1')
2298 self.assertEqual(self.builds[1].name, 'mutex-one')
2299 self.assertEqual(self.builds[2].name, 'project-test1')
2300
2301 self.worker.release('mutex-one')
2302 self.waitUntilSettled()
2303
2304 self.assertEqual(len(self.builds), 3)
2305 self.assertEqual(self.builds[0].name, 'project-test1')
2306 self.assertEqual(self.builds[1].name, 'project-test1')
2307 self.assertEqual(self.builds[2].name, 'mutex-two')
2308 self.assertTrue('test-mutex' in self.sched.mutex.mutexes)
2309
2310 self.worker.release('mutex-two')
2311 self.waitUntilSettled()
2312
2313 self.assertEqual(len(self.builds), 3)
2314 self.assertEqual(self.builds[0].name, 'project-test1')
2315 self.assertEqual(self.builds[1].name, 'project-test1')
2316 self.assertEqual(self.builds[2].name, 'mutex-one')
2317 self.assertTrue('test-mutex' in self.sched.mutex.mutexes)
2318
2319 self.worker.release('mutex-one')
2320 self.waitUntilSettled()
2321
2322 self.assertEqual(len(self.builds), 3)
2323 self.assertEqual(self.builds[0].name, 'project-test1')
2324 self.assertEqual(self.builds[1].name, 'project-test1')
2325 self.assertEqual(self.builds[2].name, 'mutex-two')
2326 self.assertTrue('test-mutex' in self.sched.mutex.mutexes)
2327
2328 self.worker.release('mutex-two')
2329 self.waitUntilSettled()
2330
2331 self.assertEqual(len(self.builds), 2)
2332 self.assertEqual(self.builds[0].name, 'project-test1')
2333 self.assertEqual(self.builds[1].name, 'project-test1')
2334 self.assertFalse('test-mutex' in self.sched.mutex.mutexes)
2335
2336 self.worker.hold_jobs_in_build = False
2337 self.worker.release()
2338
2339 self.waitUntilSettled()
2340 self.assertEqual(len(self.builds), 0)
2341
2342 self.assertEqual(A.reported, 1)
2343 self.assertEqual(B.reported, 1)
2344 self.assertFalse('test-mutex' in self.sched.mutex.mutexes)
2345
James E. Blair1f4c2bb2013-04-26 08:40:46 -07002346 def test_node_label(self):
2347 "Test that a job runs on a specific node label"
James E. Blair1f4c2bb2013-04-26 08:40:46 -07002348 self.worker.registerFunction('build:node-project-test1:debian')
2349
2350 A = self.fake_gerrit.addFakeChange('org/node-project', 'master', 'A')
2351 A.addApproval('CRVW', 2)
2352 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
2353 self.waitUntilSettled()
James E. Blair4ca985f2013-05-30 12:27:43 -07002354
Monty Taylor98f0f3e2013-07-06 16:02:31 -04002355 self.assertIsNone(self.getJobFromHistory('node-project-merge').node)
2356 self.assertEqual(self.getJobFromHistory('node-project-test1').node,
2357 'debian')
2358 self.assertIsNone(self.getJobFromHistory('node-project-test2').node)
James E. Blaircdccd972013-07-01 12:10:22 -07002359
2360 def test_live_reconfiguration(self):
2361 "Test that live reconfiguration works"
2362 self.worker.hold_jobs_in_build = True
2363 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
2364 A.addApproval('CRVW', 2)
2365 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
2366 self.waitUntilSettled()
2367
2368 self.sched.reconfigure(self.config)
2369
2370 self.worker.hold_jobs_in_build = False
2371 self.worker.release()
2372 self.waitUntilSettled()
Monty Taylor98f0f3e2013-07-06 16:02:31 -04002373 self.assertEqual(self.getJobFromHistory('project-merge').result,
2374 'SUCCESS')
2375 self.assertEqual(self.getJobFromHistory('project-test1').result,
2376 'SUCCESS')
2377 self.assertEqual(self.getJobFromHistory('project-test2').result,
2378 'SUCCESS')
2379 self.assertEqual(A.data['status'], 'MERGED')
2380 self.assertEqual(A.reported, 2)
James E. Blair287c06d2013-07-24 10:39:30 -07002381
James E. Blair6bc782d2015-07-17 16:20:21 -07002382 def test_live_reconfiguration_merge_conflict(self):
2383 # A real-world bug: a change in a gate queue has a merge
2384 # conflict and a job is added to its project while it's
2385 # sitting in the queue. The job gets added to the change and
2386 # enqueued and the change gets stuck.
2387 self.worker.registerFunction('build:project-test3')
2388 self.worker.hold_jobs_in_build = True
2389
2390 # This change is fine. It's here to stop the queue long
2391 # enough for the next change to be subject to the
2392 # reconfiguration, as well as to provide a conflict for the
2393 # next change. This change will succeed and merge.
2394 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
2395 A.addPatchset(['conflict'])
2396 A.addApproval('CRVW', 2)
James E. Blair6bc782d2015-07-17 16:20:21 -07002397
2398 # This change will be in merge conflict. During the
2399 # reconfiguration, we will add a job. We want to make sure
2400 # that doesn't cause it to get stuck.
2401 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
2402 B.addPatchset(['conflict'])
2403 B.addApproval('CRVW', 2)
James E. Blair4eb21fa2015-07-27 14:56:47 -07002404
2405 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
James E. Blair6bc782d2015-07-17 16:20:21 -07002406 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
2407
2408 self.waitUntilSettled()
2409
2410 # No jobs have run yet
2411 self.assertEqual(A.data['status'], 'NEW')
2412 self.assertEqual(A.reported, 1)
2413 self.assertEqual(B.data['status'], 'NEW')
2414 self.assertEqual(B.reported, 1)
2415 self.assertEqual(len(self.history), 0)
2416
2417 # Add the "project-test3" job.
2418 self.config.set('zuul', 'layout_config',
2419 'tests/fixtures/layout-live-'
2420 'reconfiguration-add-job.yaml')
2421 self.sched.reconfigure(self.config)
2422 self.waitUntilSettled()
2423
2424 self.worker.hold_jobs_in_build = False
2425 self.worker.release()
2426 self.waitUntilSettled()
2427
2428 self.assertEqual(A.data['status'], 'MERGED')
2429 self.assertEqual(A.reported, 2)
2430 self.assertEqual(B.data['status'], 'NEW')
2431 self.assertEqual(B.reported, 2)
2432 self.assertEqual(self.getJobFromHistory('project-merge').result,
2433 'SUCCESS')
2434 self.assertEqual(self.getJobFromHistory('project-test1').result,
2435 'SUCCESS')
2436 self.assertEqual(self.getJobFromHistory('project-test2').result,
2437 'SUCCESS')
2438 self.assertEqual(self.getJobFromHistory('project-test3').result,
2439 'SUCCESS')
2440 self.assertEqual(len(self.history), 4)
2441
James E. Blair400e8fd2015-07-30 17:44:45 -07002442 def test_live_reconfiguration_failed_root(self):
James E. Blair6bc782d2015-07-17 16:20:21 -07002443 # An extrapolation of test_live_reconfiguration_merge_conflict
2444 # that tests a job added to a job tree with a failed root does
2445 # not run.
2446 self.worker.registerFunction('build:project-test3')
2447 self.worker.hold_jobs_in_build = True
2448
2449 # This change is fine. It's here to stop the queue long
2450 # enough for the next change to be subject to the
2451 # reconfiguration. This change will succeed and merge.
2452 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
2453 A.addPatchset(['conflict'])
2454 A.addApproval('CRVW', 2)
2455 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
2456 self.waitUntilSettled()
2457 self.worker.release('.*-merge')
2458 self.waitUntilSettled()
2459
2460 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
2461 self.worker.addFailTest('project-merge', B)
2462 B.addApproval('CRVW', 2)
2463 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
2464 self.waitUntilSettled()
2465
2466 self.worker.release('.*-merge')
2467 self.waitUntilSettled()
2468
2469 # Both -merge jobs have run, but no others.
2470 self.assertEqual(A.data['status'], 'NEW')
2471 self.assertEqual(A.reported, 1)
2472 self.assertEqual(B.data['status'], 'NEW')
2473 self.assertEqual(B.reported, 1)
2474 self.assertEqual(self.history[0].result, 'SUCCESS')
2475 self.assertEqual(self.history[0].name, 'project-merge')
2476 self.assertEqual(self.history[1].result, 'FAILURE')
2477 self.assertEqual(self.history[1].name, 'project-merge')
2478 self.assertEqual(len(self.history), 2)
2479
2480 # Add the "project-test3" job.
2481 self.config.set('zuul', 'layout_config',
2482 'tests/fixtures/layout-live-'
2483 'reconfiguration-add-job.yaml')
2484 self.sched.reconfigure(self.config)
2485 self.waitUntilSettled()
2486
2487 self.worker.hold_jobs_in_build = False
2488 self.worker.release()
2489 self.waitUntilSettled()
2490
2491 self.assertEqual(A.data['status'], 'MERGED')
2492 self.assertEqual(A.reported, 2)
2493 self.assertEqual(B.data['status'], 'NEW')
2494 self.assertEqual(B.reported, 2)
2495 self.assertEqual(self.history[0].result, 'SUCCESS')
2496 self.assertEqual(self.history[0].name, 'project-merge')
2497 self.assertEqual(self.history[1].result, 'FAILURE')
2498 self.assertEqual(self.history[1].name, 'project-merge')
2499 self.assertEqual(self.history[2].result, 'SUCCESS')
2500 self.assertEqual(self.history[3].result, 'SUCCESS')
2501 self.assertEqual(self.history[4].result, 'SUCCESS')
2502 self.assertEqual(len(self.history), 5)
2503
James E. Blair400e8fd2015-07-30 17:44:45 -07002504 def test_live_reconfiguration_failed_job(self):
2505 # Test that a change with a removed failing job does not
2506 # disrupt reconfiguration. If a change has a failed job and
2507 # that job is removed during a reconfiguration, we observed a
2508 # bug where the code to re-set build statuses would run on
2509 # that build and raise an exception because the job no longer
2510 # existed.
2511 self.worker.hold_jobs_in_build = True
2512
2513 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
2514
2515 # This change will fail and later be removed by the reconfiguration.
2516 self.worker.addFailTest('project-test1', A)
2517
2518 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
2519 self.waitUntilSettled()
2520 self.worker.release('.*-merge')
2521 self.waitUntilSettled()
2522 self.worker.release('project-test1')
2523 self.waitUntilSettled()
2524
2525 self.assertEqual(A.data['status'], 'NEW')
2526 self.assertEqual(A.reported, 0)
2527
2528 self.assertEqual(self.getJobFromHistory('project-merge').result,
2529 'SUCCESS')
2530 self.assertEqual(self.getJobFromHistory('project-test1').result,
2531 'FAILURE')
2532 self.assertEqual(len(self.history), 2)
2533
2534 # Remove the test1 job.
2535 self.config.set('zuul', 'layout_config',
2536 'tests/fixtures/layout-live-'
2537 'reconfiguration-failed-job.yaml')
2538 self.sched.reconfigure(self.config)
2539 self.waitUntilSettled()
2540
2541 self.worker.hold_jobs_in_build = False
2542 self.worker.release()
2543 self.waitUntilSettled()
2544
2545 self.assertEqual(self.getJobFromHistory('project-test2').result,
2546 'SUCCESS')
2547 self.assertEqual(self.getJobFromHistory('project-testfile').result,
2548 'SUCCESS')
2549 self.assertEqual(len(self.history), 4)
2550
2551 self.assertEqual(A.data['status'], 'NEW')
2552 self.assertEqual(A.reported, 1)
2553 self.assertIn('Build succeeded', A.messages[0])
2554 # Ensure the removed job was not included in the report.
2555 self.assertNotIn('project-test1', A.messages[0])
2556
James E. Blairfe707d12015-08-05 15:18:15 -07002557 def test_live_reconfiguration_shared_queue(self):
2558 # Test that a change with a failing job which was removed from
2559 # this project but otherwise still exists in the system does
2560 # not disrupt reconfiguration.
2561
2562 self.worker.hold_jobs_in_build = True
2563
2564 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
2565
2566 self.worker.addFailTest('project1-project2-integration', A)
2567
2568 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
2569 self.waitUntilSettled()
2570 self.worker.release('.*-merge')
2571 self.waitUntilSettled()
2572 self.worker.release('project1-project2-integration')
2573 self.waitUntilSettled()
2574
2575 self.assertEqual(A.data['status'], 'NEW')
2576 self.assertEqual(A.reported, 0)
2577
2578 self.assertEqual(self.getJobFromHistory('project1-merge').result,
2579 'SUCCESS')
2580 self.assertEqual(self.getJobFromHistory(
2581 'project1-project2-integration').result, 'FAILURE')
2582 self.assertEqual(len(self.history), 2)
2583
2584 # Remove the integration job.
2585 self.config.set('zuul', 'layout_config',
2586 'tests/fixtures/layout-live-'
2587 'reconfiguration-shared-queue.yaml')
2588 self.sched.reconfigure(self.config)
2589 self.waitUntilSettled()
2590
2591 self.worker.hold_jobs_in_build = False
2592 self.worker.release()
2593 self.waitUntilSettled()
2594
2595 self.assertEqual(self.getJobFromHistory('project1-merge').result,
2596 'SUCCESS')
2597 self.assertEqual(self.getJobFromHistory('project1-test1').result,
2598 'SUCCESS')
2599 self.assertEqual(self.getJobFromHistory('project1-test2').result,
2600 'SUCCESS')
2601 self.assertEqual(self.getJobFromHistory(
2602 'project1-project2-integration').result, 'FAILURE')
2603 self.assertEqual(len(self.history), 4)
2604
2605 self.assertEqual(A.data['status'], 'NEW')
2606 self.assertEqual(A.reported, 1)
2607 self.assertIn('Build succeeded', A.messages[0])
2608 # Ensure the removed job was not included in the report.
2609 self.assertNotIn('project1-project2-integration', A.messages[0])
2610
Joshua Hesketh4bd7da32016-02-17 20:58:47 +11002611 def test_double_live_reconfiguration_shared_queue(self):
2612 # This was a real-world regression. A change is added to
2613 # gate; a reconfigure happens, a second change which depends
2614 # on the first is added, and a second reconfiguration happens.
2615 # Ensure that both changes merge.
2616
2617 # A failure may indicate incorrect caching or cleaning up of
2618 # references during a reconfiguration.
2619 self.worker.hold_jobs_in_build = True
2620
2621 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
2622 B = self.fake_gerrit.addFakeChange('org/project1', 'master', 'B')
2623 B.setDependsOn(A, 1)
2624 A.addApproval('CRVW', 2)
2625 B.addApproval('CRVW', 2)
2626
2627 # Add the parent change.
2628 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
2629 self.waitUntilSettled()
2630 self.worker.release('.*-merge')
2631 self.waitUntilSettled()
2632
2633 # Reconfigure (with only one change in the pipeline).
2634 self.sched.reconfigure(self.config)
2635 self.waitUntilSettled()
2636
2637 # Add the child change.
2638 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
2639 self.waitUntilSettled()
2640 self.worker.release('.*-merge')
2641 self.waitUntilSettled()
2642
2643 # Reconfigure (with both in the pipeline).
2644 self.sched.reconfigure(self.config)
2645 self.waitUntilSettled()
2646
2647 self.worker.hold_jobs_in_build = False
2648 self.worker.release()
2649 self.waitUntilSettled()
2650
2651 self.assertEqual(len(self.history), 8)
2652
2653 self.assertEqual(A.data['status'], 'MERGED')
2654 self.assertEqual(A.reported, 2)
2655 self.assertEqual(B.data['status'], 'MERGED')
2656 self.assertEqual(B.reported, 2)
2657
Evgeny Antyshev0deaaad2015-08-03 20:22:56 +00002658 def test_live_reconfiguration_del_project(self):
2659 # Test project deletion from layout
2660 # while changes are enqueued
2661
2662 self.worker.hold_jobs_in_build = True
2663 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
2664 B = self.fake_gerrit.addFakeChange('org/project1', 'master', 'B')
2665 C = self.fake_gerrit.addFakeChange('org/project1', 'master', 'C')
2666
2667 # A Depends-On: B
2668 A.data['commitMessage'] = '%s\n\nDepends-On: %s\n' % (
2669 A.subject, B.data['id'])
2670 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
2671
2672 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
2673 self.fake_gerrit.addEvent(C.getPatchsetCreatedEvent(1))
2674 self.waitUntilSettled()
2675 self.worker.release('.*-merge')
2676 self.waitUntilSettled()
2677 self.assertEqual(len(self.builds), 5)
2678
2679 # This layout defines only org/project, not org/project1
2680 self.config.set('zuul', 'layout_config',
2681 'tests/fixtures/layout-live-'
2682 'reconfiguration-del-project.yaml')
2683 self.sched.reconfigure(self.config)
2684 self.waitUntilSettled()
2685
2686 # Builds for C aborted, builds for A succeed,
2687 # and have change B applied ahead
2688 job_c = self.getJobFromHistory('project1-test1')
2689 self.assertEqual(job_c.changes, '3,1')
2690 self.assertEqual(job_c.result, 'ABORTED')
2691
2692 self.worker.hold_jobs_in_build = False
2693 self.worker.release()
2694 self.waitUntilSettled()
2695
2696 self.assertEqual(self.getJobFromHistory('project-test1').changes,
2697 '2,1 1,1')
2698
2699 self.assertEqual(A.data['status'], 'NEW')
2700 self.assertEqual(B.data['status'], 'NEW')
2701 self.assertEqual(C.data['status'], 'NEW')
2702 self.assertEqual(A.reported, 1)
2703 self.assertEqual(B.reported, 0)
2704 self.assertEqual(C.reported, 0)
2705
2706 self.assertEqual(len(self.sched.layout.pipelines['check'].queues), 0)
2707 self.assertIn('Build succeeded', A.messages[0])
2708
James E. Blaire712d9f2013-07-31 11:40:11 -07002709 def test_live_reconfiguration_functions(self):
2710 "Test live reconfiguration with a custom function"
2711 self.worker.registerFunction('build:node-project-test1:debian')
2712 self.worker.registerFunction('build:node-project-test1:wheezy')
2713 A = self.fake_gerrit.addFakeChange('org/node-project', 'master', 'A')
2714 A.addApproval('CRVW', 2)
2715 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
2716 self.waitUntilSettled()
2717
2718 self.assertIsNone(self.getJobFromHistory('node-project-merge').node)
2719 self.assertEqual(self.getJobFromHistory('node-project-test1').node,
2720 'debian')
2721 self.assertIsNone(self.getJobFromHistory('node-project-test2').node)
2722
2723 self.config.set('zuul', 'layout_config',
2724 'tests/fixtures/layout-live-'
2725 'reconfiguration-functions.yaml')
2726 self.sched.reconfigure(self.config)
2727 self.worker.build_history = []
2728
2729 B = self.fake_gerrit.addFakeChange('org/node-project', 'master', 'B')
2730 B.addApproval('CRVW', 2)
2731 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
2732 self.waitUntilSettled()
2733
2734 self.assertIsNone(self.getJobFromHistory('node-project-merge').node)
2735 self.assertEqual(self.getJobFromHistory('node-project-test1').node,
2736 'wheezy')
2737 self.assertIsNone(self.getJobFromHistory('node-project-test2').node)
2738
James E. Blair287c06d2013-07-24 10:39:30 -07002739 def test_delayed_repo_init(self):
2740 self.config.set('zuul', 'layout_config',
2741 'tests/fixtures/layout-delayed-repo-init.yaml')
2742 self.sched.reconfigure(self.config)
2743
2744 self.init_repo("org/new-project")
2745 A = self.fake_gerrit.addFakeChange('org/new-project', 'master', 'A')
2746
2747 A.addApproval('CRVW', 2)
2748 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
2749 self.waitUntilSettled()
2750 self.assertEqual(self.getJobFromHistory('project-merge').result,
2751 'SUCCESS')
2752 self.assertEqual(self.getJobFromHistory('project-test1').result,
2753 'SUCCESS')
2754 self.assertEqual(self.getJobFromHistory('project-test2').result,
2755 'SUCCESS')
2756 self.assertEqual(A.data['status'], 'MERGED')
2757 self.assertEqual(A.reported, 2)
James E. Blair63bb0ef2013-07-29 17:14:51 -07002758
Clark Boylan6dbbc482013-10-18 10:57:31 -07002759 def test_repo_deleted(self):
2760 self.config.set('zuul', 'layout_config',
2761 'tests/fixtures/layout-repo-deleted.yaml')
2762 self.sched.reconfigure(self.config)
2763
2764 self.init_repo("org/delete-project")
2765 A = self.fake_gerrit.addFakeChange('org/delete-project', 'master', 'A')
2766
2767 A.addApproval('CRVW', 2)
2768 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
2769 self.waitUntilSettled()
2770 self.assertEqual(self.getJobFromHistory('project-merge').result,
2771 'SUCCESS')
2772 self.assertEqual(self.getJobFromHistory('project-test1').result,
2773 'SUCCESS')
2774 self.assertEqual(self.getJobFromHistory('project-test2').result,
2775 'SUCCESS')
2776 self.assertEqual(A.data['status'], 'MERGED')
2777 self.assertEqual(A.reported, 2)
2778
2779 # Delete org/new-project zuul repo. Should be recloned.
2780 shutil.rmtree(os.path.join(self.git_root, "org/delete-project"))
2781
2782 B = self.fake_gerrit.addFakeChange('org/delete-project', 'master', 'B')
2783
2784 B.addApproval('CRVW', 2)
2785 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
2786 self.waitUntilSettled()
2787 self.assertEqual(self.getJobFromHistory('project-merge').result,
2788 'SUCCESS')
2789 self.assertEqual(self.getJobFromHistory('project-test1').result,
2790 'SUCCESS')
2791 self.assertEqual(self.getJobFromHistory('project-test2').result,
2792 'SUCCESS')
2793 self.assertEqual(B.data['status'], 'MERGED')
2794 self.assertEqual(B.reported, 2)
2795
James E. Blair456f2fb2016-02-09 09:29:33 -08002796 def test_tags(self):
2797 "Test job tags"
2798 self.config.set('zuul', 'layout_config',
2799 'tests/fixtures/layout-tags.yaml')
2800 self.sched.reconfigure(self.config)
2801
2802 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
2803 B = self.fake_gerrit.addFakeChange('org/project2', 'master', 'B')
2804 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
2805 self.fake_gerrit.addEvent(B.getPatchsetCreatedEvent(1))
2806 self.waitUntilSettled()
2807
2808 results = {'project1-merge': 'extratag merge project1',
2809 'project2-merge': 'merge'}
2810
2811 for build in self.history:
2812 self.assertEqual(results.get(build.name, ''),
2813 build.parameters.get('BUILD_TAGS'))
2814
James E. Blair63bb0ef2013-07-29 17:14:51 -07002815 def test_timer(self):
2816 "Test that a periodic job is triggered"
2817 self.worker.hold_jobs_in_build = True
2818 self.config.set('zuul', 'layout_config',
2819 'tests/fixtures/layout-timer.yaml')
2820 self.sched.reconfigure(self.config)
2821 self.registerJobs()
2822
Clark Boylan3ee090a2014-04-03 20:55:09 -07002823 # The pipeline triggers every second, so we should have seen
2824 # several by now.
2825 time.sleep(5)
James E. Blair63bb0ef2013-07-29 17:14:51 -07002826 self.waitUntilSettled()
Clark Boylan3ee090a2014-04-03 20:55:09 -07002827
2828 self.assertEqual(len(self.builds), 2)
2829
James E. Blair63bb0ef2013-07-29 17:14:51 -07002830 port = self.webapp.server.socket.getsockname()[1]
2831
2832 f = urllib.urlopen("http://localhost:%s/status.json" % port)
2833 data = f.read()
2834
2835 self.worker.hold_jobs_in_build = False
Clark Boylan3ee090a2014-04-03 20:55:09 -07002836 # Stop queuing timer triggered jobs so that the assertions
2837 # below don't race against more jobs being queued.
2838 self.config.set('zuul', 'layout_config',
2839 'tests/fixtures/layout-no-timer.yaml')
2840 self.sched.reconfigure(self.config)
2841 self.registerJobs()
James E. Blair63bb0ef2013-07-29 17:14:51 -07002842 self.worker.release()
2843 self.waitUntilSettled()
2844
2845 self.assertEqual(self.getJobFromHistory(
2846 'project-bitrot-stable-old').result, 'SUCCESS')
2847 self.assertEqual(self.getJobFromHistory(
2848 'project-bitrot-stable-older').result, 'SUCCESS')
2849
2850 data = json.loads(data)
2851 status_jobs = set()
2852 for p in data['pipelines']:
2853 for q in p['change_queues']:
2854 for head in q['heads']:
2855 for change in head:
Alex Gaynorddb9ef32013-09-16 21:04:58 -07002856 self.assertEqual(change['id'], None)
James E. Blair63bb0ef2013-07-29 17:14:51 -07002857 for job in change['jobs']:
2858 status_jobs.add(job['name'])
2859 self.assertIn('project-bitrot-stable-old', status_jobs)
2860 self.assertIn('project-bitrot-stable-older', status_jobs)
Joshua Hesketh5fea8672013-08-19 17:32:01 +10002861
James E. Blair4f6033c2014-03-27 15:49:09 -07002862 def test_idle(self):
2863 "Test that frequent periodic jobs work"
2864 self.worker.hold_jobs_in_build = True
James E. Blair4f6033c2014-03-27 15:49:09 -07002865
Clark Boylan3ee090a2014-04-03 20:55:09 -07002866 for x in range(1, 3):
2867 # Test that timer triggers periodic jobs even across
2868 # layout config reloads.
2869 # Start timer trigger
2870 self.config.set('zuul', 'layout_config',
2871 'tests/fixtures/layout-idle.yaml')
2872 self.sched.reconfigure(self.config)
2873 self.registerJobs()
James E. Blair995fc0f2016-02-04 16:48:31 -08002874 self.waitUntilSettled()
James E. Blair4f6033c2014-03-27 15:49:09 -07002875
Clark Boylan3ee090a2014-04-03 20:55:09 -07002876 # The pipeline triggers every second, so we should have seen
2877 # several by now.
2878 time.sleep(5)
Clark Boylan3ee090a2014-04-03 20:55:09 -07002879
2880 # Stop queuing timer triggered jobs so that the assertions
2881 # below don't race against more jobs being queued.
2882 self.config.set('zuul', 'layout_config',
2883 'tests/fixtures/layout-no-timer.yaml')
2884 self.sched.reconfigure(self.config)
2885 self.registerJobs()
James E. Blair995fc0f2016-02-04 16:48:31 -08002886 self.waitUntilSettled()
Clark Boylan3ee090a2014-04-03 20:55:09 -07002887
2888 self.assertEqual(len(self.builds), 2)
2889 self.worker.release('.*')
2890 self.waitUntilSettled()
2891 self.assertEqual(len(self.builds), 0)
2892 self.assertEqual(len(self.history), x * 2)
James E. Blair4f6033c2014-03-27 15:49:09 -07002893
Joshua Hesketh5fea8672013-08-19 17:32:01 +10002894 def test_check_smtp_pool(self):
2895 self.config.set('zuul', 'layout_config',
2896 'tests/fixtures/layout-smtp.yaml')
2897 self.sched.reconfigure(self.config)
2898
2899 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
2900 self.waitUntilSettled()
2901
2902 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
2903 self.waitUntilSettled()
2904
James E. Blairff80a2f2013-12-27 13:24:06 -08002905 self.assertEqual(len(self.smtp_messages), 2)
Joshua Hesketh5fea8672013-08-19 17:32:01 +10002906
2907 # A.messages only holds what FakeGerrit places in it. Thus we
2908 # work on the knowledge of what the first message should be as
2909 # it is only configured to go to SMTP.
2910
2911 self.assertEqual('zuul@example.com',
James E. Blairff80a2f2013-12-27 13:24:06 -08002912 self.smtp_messages[0]['from_email'])
Joshua Hesketh5fea8672013-08-19 17:32:01 +10002913 self.assertEqual(['you@example.com'],
James E. Blairff80a2f2013-12-27 13:24:06 -08002914 self.smtp_messages[0]['to_email'])
Joshua Hesketh5fea8672013-08-19 17:32:01 +10002915 self.assertEqual('Starting check jobs.',
James E. Blairff80a2f2013-12-27 13:24:06 -08002916 self.smtp_messages[0]['body'])
Joshua Hesketh5fea8672013-08-19 17:32:01 +10002917
2918 self.assertEqual('zuul_from@example.com',
James E. Blairff80a2f2013-12-27 13:24:06 -08002919 self.smtp_messages[1]['from_email'])
Joshua Hesketh5fea8672013-08-19 17:32:01 +10002920 self.assertEqual(['alternative_me@example.com'],
James E. Blairff80a2f2013-12-27 13:24:06 -08002921 self.smtp_messages[1]['to_email'])
Joshua Hesketh5fea8672013-08-19 17:32:01 +10002922 self.assertEqual(A.messages[0],
James E. Blairff80a2f2013-12-27 13:24:06 -08002923 self.smtp_messages[1]['body'])
James E. Blairad28e912013-11-27 10:43:22 -08002924
James E. Blaire5910202013-12-27 09:50:31 -08002925 def test_timer_smtp(self):
2926 "Test that a periodic job is triggered"
Clark Boylan3ee090a2014-04-03 20:55:09 -07002927 self.worker.hold_jobs_in_build = True
James E. Blaire5910202013-12-27 09:50:31 -08002928 self.config.set('zuul', 'layout_config',
2929 'tests/fixtures/layout-timer-smtp.yaml')
2930 self.sched.reconfigure(self.config)
2931 self.registerJobs()
2932
Clark Boylan3ee090a2014-04-03 20:55:09 -07002933 # The pipeline triggers every second, so we should have seen
2934 # several by now.
2935 time.sleep(5)
James E. Blaire5910202013-12-27 09:50:31 -08002936 self.waitUntilSettled()
2937
Clark Boylan3ee090a2014-04-03 20:55:09 -07002938 self.assertEqual(len(self.builds), 2)
2939 self.worker.release('.*')
2940 self.waitUntilSettled()
2941 self.assertEqual(len(self.history), 2)
2942
James E. Blaire5910202013-12-27 09:50:31 -08002943 self.assertEqual(self.getJobFromHistory(
2944 'project-bitrot-stable-old').result, 'SUCCESS')
2945 self.assertEqual(self.getJobFromHistory(
2946 'project-bitrot-stable-older').result, 'SUCCESS')
2947
James E. Blairff80a2f2013-12-27 13:24:06 -08002948 self.assertEqual(len(self.smtp_messages), 1)
James E. Blaire5910202013-12-27 09:50:31 -08002949
2950 # A.messages only holds what FakeGerrit places in it. Thus we
2951 # work on the knowledge of what the first message should be as
2952 # it is only configured to go to SMTP.
2953
2954 self.assertEqual('zuul_from@example.com',
James E. Blairff80a2f2013-12-27 13:24:06 -08002955 self.smtp_messages[0]['from_email'])
James E. Blaire5910202013-12-27 09:50:31 -08002956 self.assertEqual(['alternative_me@example.com'],
James E. Blairff80a2f2013-12-27 13:24:06 -08002957 self.smtp_messages[0]['to_email'])
James E. Blaire5910202013-12-27 09:50:31 -08002958 self.assertIn('Subject: Periodic check for org/project succeeded',
James E. Blairff80a2f2013-12-27 13:24:06 -08002959 self.smtp_messages[0]['headers'])
James E. Blaire5910202013-12-27 09:50:31 -08002960
Clark Boylan3ee090a2014-04-03 20:55:09 -07002961 # Stop queuing timer triggered jobs and let any that may have
2962 # queued through so that end of test assertions pass.
2963 self.config.set('zuul', 'layout_config',
2964 'tests/fixtures/layout-no-timer.yaml')
2965 self.sched.reconfigure(self.config)
2966 self.registerJobs()
James E. Blairf8058972014-08-15 16:09:16 -07002967 self.waitUntilSettled()
Clark Boylan3ee090a2014-04-03 20:55:09 -07002968 self.worker.release('.*')
2969 self.waitUntilSettled()
2970
James E. Blair91e34592015-07-31 16:45:59 -07002971 def test_client_enqueue_change(self):
James E. Blairad28e912013-11-27 10:43:22 -08002972 "Test that the RPC client can enqueue a change"
2973 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
2974 A.addApproval('CRVW', 2)
2975 A.addApproval('APRV', 1)
2976
2977 client = zuul.rpcclient.RPCClient('127.0.0.1',
2978 self.gearman_server.port)
2979 r = client.enqueue(pipeline='gate',
2980 project='org/project',
2981 trigger='gerrit',
James E. Blair36658cf2013-12-06 17:53:48 -08002982 change='1,1')
James E. Blairad28e912013-11-27 10:43:22 -08002983 self.waitUntilSettled()
2984 self.assertEqual(self.getJobFromHistory('project-merge').result,
2985 'SUCCESS')
2986 self.assertEqual(self.getJobFromHistory('project-test1').result,
2987 'SUCCESS')
2988 self.assertEqual(self.getJobFromHistory('project-test2').result,
2989 'SUCCESS')
2990 self.assertEqual(A.data['status'], 'MERGED')
2991 self.assertEqual(A.reported, 2)
2992 self.assertEqual(r, True)
2993
James E. Blair91e34592015-07-31 16:45:59 -07002994 def test_client_enqueue_ref(self):
2995 "Test that the RPC client can enqueue a ref"
2996
2997 client = zuul.rpcclient.RPCClient('127.0.0.1',
2998 self.gearman_server.port)
2999 r = client.enqueue_ref(
3000 pipeline='post',
3001 project='org/project',
3002 trigger='gerrit',
3003 ref='master',
3004 oldrev='90f173846e3af9154517b88543ffbd1691f31366',
3005 newrev='d479a0bfcb34da57a31adb2a595c0cf687812543')
3006 self.waitUntilSettled()
3007 job_names = [x.name for x in self.history]
3008 self.assertEqual(len(self.history), 1)
3009 self.assertIn('project-post', job_names)
3010 self.assertEqual(r, True)
3011
James E. Blairad28e912013-11-27 10:43:22 -08003012 def test_client_enqueue_negative(self):
3013 "Test that the RPC client returns errors"
3014 client = zuul.rpcclient.RPCClient('127.0.0.1',
3015 self.gearman_server.port)
3016 with testtools.ExpectedException(zuul.rpcclient.RPCFailure,
3017 "Invalid project"):
3018 r = client.enqueue(pipeline='gate',
3019 project='project-does-not-exist',
3020 trigger='gerrit',
James E. Blair36658cf2013-12-06 17:53:48 -08003021 change='1,1')
James E. Blairad28e912013-11-27 10:43:22 -08003022 client.shutdown()
3023 self.assertEqual(r, False)
3024
3025 with testtools.ExpectedException(zuul.rpcclient.RPCFailure,
3026 "Invalid pipeline"):
3027 r = client.enqueue(pipeline='pipeline-does-not-exist',
3028 project='org/project',
3029 trigger='gerrit',
James E. Blair36658cf2013-12-06 17:53:48 -08003030 change='1,1')
James E. Blairad28e912013-11-27 10:43:22 -08003031 client.shutdown()
3032 self.assertEqual(r, False)
3033
3034 with testtools.ExpectedException(zuul.rpcclient.RPCFailure,
3035 "Invalid trigger"):
3036 r = client.enqueue(pipeline='gate',
3037 project='org/project',
3038 trigger='trigger-does-not-exist',
James E. Blair36658cf2013-12-06 17:53:48 -08003039 change='1,1')
James E. Blairad28e912013-11-27 10:43:22 -08003040 client.shutdown()
3041 self.assertEqual(r, False)
3042
3043 with testtools.ExpectedException(zuul.rpcclient.RPCFailure,
3044 "Invalid change"):
3045 r = client.enqueue(pipeline='gate',
3046 project='org/project',
3047 trigger='gerrit',
James E. Blair36658cf2013-12-06 17:53:48 -08003048 change='1,1')
James E. Blairad28e912013-11-27 10:43:22 -08003049 client.shutdown()
3050 self.assertEqual(r, False)
3051
3052 self.waitUntilSettled()
3053 self.assertEqual(len(self.history), 0)
3054 self.assertEqual(len(self.builds), 0)
James E. Blair36658cf2013-12-06 17:53:48 -08003055
3056 def test_client_promote(self):
3057 "Test that the RPC client can promote a change"
3058 self.worker.hold_jobs_in_build = True
3059 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
3060 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
3061 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
3062 A.addApproval('CRVW', 2)
3063 B.addApproval('CRVW', 2)
3064 C.addApproval('CRVW', 2)
3065
3066 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3067 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
3068 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
3069
3070 self.waitUntilSettled()
3071
Sean Daguef39b9ca2014-01-10 21:34:35 -05003072 items = self.sched.layout.pipelines['gate'].getAllItems()
3073 enqueue_times = {}
3074 for item in items:
3075 enqueue_times[str(item.change)] = item.enqueue_time
3076
James E. Blair36658cf2013-12-06 17:53:48 -08003077 client = zuul.rpcclient.RPCClient('127.0.0.1',
3078 self.gearman_server.port)
3079 r = client.promote(pipeline='gate',
3080 change_ids=['2,1', '3,1'])
3081
Sean Daguef39b9ca2014-01-10 21:34:35 -05003082 # ensure that enqueue times are durable
3083 items = self.sched.layout.pipelines['gate'].getAllItems()
3084 for item in items:
3085 self.assertEqual(
3086 enqueue_times[str(item.change)], item.enqueue_time)
3087
James E. Blair78acec92014-02-06 07:11:32 -08003088 self.waitUntilSettled()
James E. Blair36658cf2013-12-06 17:53:48 -08003089 self.worker.release('.*-merge')
3090 self.waitUntilSettled()
3091 self.worker.release('.*-merge')
3092 self.waitUntilSettled()
3093 self.worker.release('.*-merge')
3094 self.waitUntilSettled()
3095
3096 self.assertEqual(len(self.builds), 6)
3097 self.assertEqual(self.builds[0].name, 'project-test1')
3098 self.assertEqual(self.builds[1].name, 'project-test2')
3099 self.assertEqual(self.builds[2].name, 'project-test1')
3100 self.assertEqual(self.builds[3].name, 'project-test2')
3101 self.assertEqual(self.builds[4].name, 'project-test1')
3102 self.assertEqual(self.builds[5].name, 'project-test2')
3103
3104 self.assertTrue(self.job_has_changes(self.builds[0], B))
3105 self.assertFalse(self.job_has_changes(self.builds[0], A))
3106 self.assertFalse(self.job_has_changes(self.builds[0], C))
3107
3108 self.assertTrue(self.job_has_changes(self.builds[2], B))
3109 self.assertTrue(self.job_has_changes(self.builds[2], C))
3110 self.assertFalse(self.job_has_changes(self.builds[2], A))
3111
3112 self.assertTrue(self.job_has_changes(self.builds[4], B))
3113 self.assertTrue(self.job_has_changes(self.builds[4], C))
3114 self.assertTrue(self.job_has_changes(self.builds[4], A))
3115
3116 self.worker.release()
3117 self.waitUntilSettled()
3118
3119 self.assertEqual(A.data['status'], 'MERGED')
3120 self.assertEqual(A.reported, 2)
3121 self.assertEqual(B.data['status'], 'MERGED')
3122 self.assertEqual(B.reported, 2)
3123 self.assertEqual(C.data['status'], 'MERGED')
3124 self.assertEqual(C.reported, 2)
3125
3126 client.shutdown()
3127 self.assertEqual(r, True)
3128
3129 def test_client_promote_dependent(self):
3130 "Test that the RPC client can promote a dependent change"
3131 # C (depends on B) -> B -> A ; then promote C to get:
3132 # A -> C (depends on B) -> B
3133 self.worker.hold_jobs_in_build = True
3134 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
3135 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
3136 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
3137
3138 C.setDependsOn(B, 1)
3139
3140 A.addApproval('CRVW', 2)
3141 B.addApproval('CRVW', 2)
3142 C.addApproval('CRVW', 2)
3143
3144 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3145 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
3146 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
3147
3148 self.waitUntilSettled()
3149
3150 client = zuul.rpcclient.RPCClient('127.0.0.1',
3151 self.gearman_server.port)
3152 r = client.promote(pipeline='gate',
3153 change_ids=['3,1'])
3154
James E. Blair78acec92014-02-06 07:11:32 -08003155 self.waitUntilSettled()
James E. Blair36658cf2013-12-06 17:53:48 -08003156 self.worker.release('.*-merge')
3157 self.waitUntilSettled()
3158 self.worker.release('.*-merge')
3159 self.waitUntilSettled()
3160 self.worker.release('.*-merge')
3161 self.waitUntilSettled()
3162
3163 self.assertEqual(len(self.builds), 6)
3164 self.assertEqual(self.builds[0].name, 'project-test1')
3165 self.assertEqual(self.builds[1].name, 'project-test2')
3166 self.assertEqual(self.builds[2].name, 'project-test1')
3167 self.assertEqual(self.builds[3].name, 'project-test2')
3168 self.assertEqual(self.builds[4].name, 'project-test1')
3169 self.assertEqual(self.builds[5].name, 'project-test2')
3170
3171 self.assertTrue(self.job_has_changes(self.builds[0], B))
3172 self.assertFalse(self.job_has_changes(self.builds[0], A))
3173 self.assertFalse(self.job_has_changes(self.builds[0], C))
3174
3175 self.assertTrue(self.job_has_changes(self.builds[2], B))
3176 self.assertTrue(self.job_has_changes(self.builds[2], C))
3177 self.assertFalse(self.job_has_changes(self.builds[2], A))
3178
3179 self.assertTrue(self.job_has_changes(self.builds[4], B))
3180 self.assertTrue(self.job_has_changes(self.builds[4], C))
3181 self.assertTrue(self.job_has_changes(self.builds[4], A))
3182
3183 self.worker.release()
3184 self.waitUntilSettled()
3185
3186 self.assertEqual(A.data['status'], 'MERGED')
3187 self.assertEqual(A.reported, 2)
3188 self.assertEqual(B.data['status'], 'MERGED')
3189 self.assertEqual(B.reported, 2)
3190 self.assertEqual(C.data['status'], 'MERGED')
3191 self.assertEqual(C.reported, 2)
3192
3193 client.shutdown()
3194 self.assertEqual(r, True)
3195
3196 def test_client_promote_negative(self):
3197 "Test that the RPC client returns errors for promotion"
3198 self.worker.hold_jobs_in_build = True
3199 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
3200 A.addApproval('CRVW', 2)
3201 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3202 self.waitUntilSettled()
3203
3204 client = zuul.rpcclient.RPCClient('127.0.0.1',
3205 self.gearman_server.port)
3206
3207 with testtools.ExpectedException(zuul.rpcclient.RPCFailure):
3208 r = client.promote(pipeline='nonexistent',
3209 change_ids=['2,1', '3,1'])
3210 client.shutdown()
3211 self.assertEqual(r, False)
3212
3213 with testtools.ExpectedException(zuul.rpcclient.RPCFailure):
3214 r = client.promote(pipeline='gate',
3215 change_ids=['4,1'])
3216 client.shutdown()
3217 self.assertEqual(r, False)
3218
3219 self.worker.hold_jobs_in_build = False
3220 self.worker.release()
3221 self.waitUntilSettled()
Clark Boylan7603a372014-01-21 11:43:20 -08003222
3223 def test_queue_rate_limiting(self):
3224 "Test that DependentPipelines are rate limited with dep across window"
3225 self.config.set('zuul', 'layout_config',
3226 'tests/fixtures/layout-rate-limit.yaml')
3227 self.sched.reconfigure(self.config)
3228 self.worker.hold_jobs_in_build = True
3229 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
3230 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
3231 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
3232
3233 C.setDependsOn(B, 1)
3234 self.worker.addFailTest('project-test1', A)
3235
3236 A.addApproval('CRVW', 2)
3237 B.addApproval('CRVW', 2)
3238 C.addApproval('CRVW', 2)
3239
3240 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3241 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
3242 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
3243 self.waitUntilSettled()
3244
3245 # Only A and B will have their merge jobs queued because
3246 # window is 2.
3247 self.assertEqual(len(self.builds), 2)
3248 self.assertEqual(self.builds[0].name, 'project-merge')
3249 self.assertEqual(self.builds[1].name, 'project-merge')
3250
3251 self.worker.release('.*-merge')
3252 self.waitUntilSettled()
3253 self.worker.release('.*-merge')
3254 self.waitUntilSettled()
3255
3256 # Only A and B will have their test jobs queued because
3257 # window is 2.
3258 self.assertEqual(len(self.builds), 4)
3259 self.assertEqual(self.builds[0].name, 'project-test1')
3260 self.assertEqual(self.builds[1].name, 'project-test2')
3261 self.assertEqual(self.builds[2].name, 'project-test1')
3262 self.assertEqual(self.builds[3].name, 'project-test2')
3263
3264 self.worker.release('project-.*')
3265 self.waitUntilSettled()
3266
3267 queue = self.sched.layout.pipelines['gate'].queues[0]
3268 # A failed so window is reduced by 1 to 1.
3269 self.assertEqual(queue.window, 1)
3270 self.assertEqual(queue.window_floor, 1)
3271 self.assertEqual(A.data['status'], 'NEW')
3272
3273 # Gate is reset and only B's merge job is queued because
3274 # window shrunk to 1.
3275 self.assertEqual(len(self.builds), 1)
3276 self.assertEqual(self.builds[0].name, 'project-merge')
3277
3278 self.worker.release('.*-merge')
3279 self.waitUntilSettled()
3280
3281 # Only B's test jobs are queued because window is still 1.
3282 self.assertEqual(len(self.builds), 2)
3283 self.assertEqual(self.builds[0].name, 'project-test1')
3284 self.assertEqual(self.builds[1].name, 'project-test2')
3285
3286 self.worker.release('project-.*')
3287 self.waitUntilSettled()
3288
3289 # B was successfully merged so window is increased to 2.
3290 self.assertEqual(queue.window, 2)
3291 self.assertEqual(queue.window_floor, 1)
3292 self.assertEqual(B.data['status'], 'MERGED')
3293
3294 # Only C is left and its merge job is queued.
3295 self.assertEqual(len(self.builds), 1)
3296 self.assertEqual(self.builds[0].name, 'project-merge')
3297
3298 self.worker.release('.*-merge')
3299 self.waitUntilSettled()
3300
3301 # After successful merge job the test jobs for C are queued.
3302 self.assertEqual(len(self.builds), 2)
3303 self.assertEqual(self.builds[0].name, 'project-test1')
3304 self.assertEqual(self.builds[1].name, 'project-test2')
3305
3306 self.worker.release('project-.*')
3307 self.waitUntilSettled()
3308
3309 # C successfully merged so window is bumped to 3.
3310 self.assertEqual(queue.window, 3)
3311 self.assertEqual(queue.window_floor, 1)
3312 self.assertEqual(C.data['status'], 'MERGED')
3313
3314 def test_queue_rate_limiting_dependent(self):
3315 "Test that DependentPipelines are rate limited with dep in window"
3316 self.config.set('zuul', 'layout_config',
3317 'tests/fixtures/layout-rate-limit.yaml')
3318 self.sched.reconfigure(self.config)
3319 self.worker.hold_jobs_in_build = True
3320 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
3321 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
3322 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
3323
3324 B.setDependsOn(A, 1)
3325
3326 self.worker.addFailTest('project-test1', A)
3327
3328 A.addApproval('CRVW', 2)
3329 B.addApproval('CRVW', 2)
3330 C.addApproval('CRVW', 2)
3331
3332 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3333 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
3334 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
3335 self.waitUntilSettled()
3336
3337 # Only A and B will have their merge jobs queued because
3338 # window is 2.
3339 self.assertEqual(len(self.builds), 2)
3340 self.assertEqual(self.builds[0].name, 'project-merge')
3341 self.assertEqual(self.builds[1].name, 'project-merge')
3342
3343 self.worker.release('.*-merge')
3344 self.waitUntilSettled()
3345 self.worker.release('.*-merge')
3346 self.waitUntilSettled()
3347
3348 # Only A and B will have their test jobs queued because
3349 # window is 2.
3350 self.assertEqual(len(self.builds), 4)
3351 self.assertEqual(self.builds[0].name, 'project-test1')
3352 self.assertEqual(self.builds[1].name, 'project-test2')
3353 self.assertEqual(self.builds[2].name, 'project-test1')
3354 self.assertEqual(self.builds[3].name, 'project-test2')
3355
3356 self.worker.release('project-.*')
3357 self.waitUntilSettled()
3358
3359 queue = self.sched.layout.pipelines['gate'].queues[0]
3360 # A failed so window is reduced by 1 to 1.
3361 self.assertEqual(queue.window, 1)
3362 self.assertEqual(queue.window_floor, 1)
3363 self.assertEqual(A.data['status'], 'NEW')
3364 self.assertEqual(B.data['status'], 'NEW')
3365
3366 # Gate is reset and only C's merge job is queued because
3367 # window shrunk to 1 and A and B were dequeued.
3368 self.assertEqual(len(self.builds), 1)
3369 self.assertEqual(self.builds[0].name, 'project-merge')
3370
3371 self.worker.release('.*-merge')
3372 self.waitUntilSettled()
3373
3374 # Only C's test jobs are queued because window is still 1.
3375 self.assertEqual(len(self.builds), 2)
3376 self.assertEqual(self.builds[0].name, 'project-test1')
3377 self.assertEqual(self.builds[1].name, 'project-test2')
3378
3379 self.worker.release('project-.*')
3380 self.waitUntilSettled()
3381
3382 # C was successfully merged so window is increased to 2.
3383 self.assertEqual(queue.window, 2)
3384 self.assertEqual(queue.window_floor, 1)
3385 self.assertEqual(C.data['status'], 'MERGED')
Joshua Heskethba8776a2014-01-12 14:35:40 +08003386
3387 def test_worker_update_metadata(self):
3388 "Test if a worker can send back metadata about itself"
3389 self.worker.hold_jobs_in_build = True
3390
3391 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
3392 A.addApproval('CRVW', 2)
3393 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3394 self.waitUntilSettled()
3395
3396 self.assertEqual(len(self.launcher.builds), 1)
3397
3398 self.log.debug('Current builds:')
3399 self.log.debug(self.launcher.builds)
3400
3401 start = time.time()
3402 while True:
3403 if time.time() - start > 10:
3404 raise Exception("Timeout waiting for gearman server to report "
3405 + "back to the client")
3406 build = self.launcher.builds.values()[0]
3407 if build.worker.name == "My Worker":
3408 break
3409 else:
3410 time.sleep(0)
3411
3412 self.log.debug(build)
3413 self.assertEqual("My Worker", build.worker.name)
3414 self.assertEqual("localhost", build.worker.hostname)
3415 self.assertEqual(['127.0.0.1', '192.168.1.1'], build.worker.ips)
3416 self.assertEqual("zuul.example.org", build.worker.fqdn)
3417 self.assertEqual("FakeBuilder", build.worker.program)
3418 self.assertEqual("v1.1", build.worker.version)
3419 self.assertEqual({'something': 'else'}, build.worker.extra)
3420
3421 self.worker.hold_jobs_in_build = False
3422 self.worker.release()
3423 self.waitUntilSettled()
Joshua Hesketh3979e3e2014-03-04 11:21:10 +11003424
3425 def test_footer_message(self):
3426 "Test a pipeline's footer message is correctly added to the report."
3427 self.config.set('zuul', 'layout_config',
3428 'tests/fixtures/layout-footer-message.yaml')
3429 self.sched.reconfigure(self.config)
3430 self.registerJobs()
3431
3432 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
3433 A.addApproval('CRVW', 2)
3434 self.worker.addFailTest('test1', A)
3435 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3436 self.waitUntilSettled()
3437
3438 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
3439 B.addApproval('CRVW', 2)
3440 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
3441 self.waitUntilSettled()
3442
3443 self.assertEqual(2, len(self.smtp_messages))
3444
3445 failure_body = """\
3446Build failed. For information on how to proceed, see \
3447http://wiki.example.org/Test_Failures
3448
3449- test1 http://logs.example.com/1/1/gate/test1/0 : FAILURE in 0s
3450- test2 http://logs.example.com/1/1/gate/test2/1 : SUCCESS in 0s
3451
3452For CI problems and help debugging, contact ci@example.org"""
3453
3454 success_body = """\
3455Build succeeded.
3456
3457- test1 http://logs.example.com/2/1/gate/test1/2 : SUCCESS in 0s
3458- test2 http://logs.example.com/2/1/gate/test2/3 : SUCCESS in 0s
3459
3460For CI problems and help debugging, contact ci@example.org"""
3461
3462 self.assertEqual(failure_body, self.smtp_messages[0]['body'])
3463 self.assertEqual(success_body, self.smtp_messages[1]['body'])
Joshua Heskethb7179772014-01-30 23:30:46 +11003464
3465 def test_merge_failure_reporters(self):
3466 """Check that the config is set up correctly"""
3467
3468 self.config.set('zuul', 'layout_config',
3469 'tests/fixtures/layout-merge-failure.yaml')
3470 self.sched.reconfigure(self.config)
3471 self.registerJobs()
3472
3473 self.assertEqual(
Jeremy Stanley1c2c3c22015-06-15 21:23:19 +00003474 "Merge Failed.\n\nThis change or one of its cross-repo "
3475 "dependencies was unable to be automatically merged with the "
3476 "current state of its repository. Please rebase the change and "
3477 "upload a new patchset.",
Joshua Heskethb7179772014-01-30 23:30:46 +11003478 self.sched.layout.pipelines['check'].merge_failure_message)
3479 self.assertEqual(
3480 "The merge failed! For more information...",
3481 self.sched.layout.pipelines['gate'].merge_failure_message)
3482
3483 self.assertEqual(
3484 len(self.sched.layout.pipelines['check'].merge_failure_actions), 1)
3485 self.assertEqual(
3486 len(self.sched.layout.pipelines['gate'].merge_failure_actions), 2)
3487
3488 self.assertTrue(isinstance(
Joshua Heskethde958652015-11-10 19:19:50 +11003489 self.sched.layout.pipelines['check'].merge_failure_actions[0],
3490 zuul.reporter.gerrit.GerritReporter))
Joshua Heskethb7179772014-01-30 23:30:46 +11003491
3492 self.assertTrue(
3493 (
3494 isinstance(self.sched.layout.pipelines['gate'].
Joshua Heskethde958652015-11-10 19:19:50 +11003495 merge_failure_actions[0],
Joshua Heskethffe42062014-09-05 21:43:52 +10003496 zuul.reporter.smtp.SMTPReporter) and
Joshua Heskethb7179772014-01-30 23:30:46 +11003497 isinstance(self.sched.layout.pipelines['gate'].
Joshua Heskethde958652015-11-10 19:19:50 +11003498 merge_failure_actions[1],
Joshua Heskethffe42062014-09-05 21:43:52 +10003499 zuul.reporter.gerrit.GerritReporter)
Joshua Heskethb7179772014-01-30 23:30:46 +11003500 ) or (
3501 isinstance(self.sched.layout.pipelines['gate'].
Joshua Heskethde958652015-11-10 19:19:50 +11003502 merge_failure_actions[0],
Joshua Heskethffe42062014-09-05 21:43:52 +10003503 zuul.reporter.gerrit.GerritReporter) and
Joshua Heskethb7179772014-01-30 23:30:46 +11003504 isinstance(self.sched.layout.pipelines['gate'].
Joshua Heskethde958652015-11-10 19:19:50 +11003505 merge_failure_actions[1],
Joshua Heskethffe42062014-09-05 21:43:52 +10003506 zuul.reporter.smtp.SMTPReporter)
Joshua Heskethb7179772014-01-30 23:30:46 +11003507 )
3508 )
3509
3510 def test_merge_failure_reports(self):
3511 """Check that when a change fails to merge the correct message is sent
3512 to the correct reporter"""
3513 self.config.set('zuul', 'layout_config',
3514 'tests/fixtures/layout-merge-failure.yaml')
3515 self.sched.reconfigure(self.config)
3516 self.registerJobs()
3517
3518 # Check a test failure isn't reported to SMTP
3519 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
3520 A.addApproval('CRVW', 2)
3521 self.worker.addFailTest('project-test1', A)
3522 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3523 self.waitUntilSettled()
3524
3525 self.assertEqual(3, len(self.history)) # 3 jobs
3526 self.assertEqual(0, len(self.smtp_messages))
3527
3528 # Check a merge failure is reported to SMTP
3529 # B should be merged, but C will conflict with B
3530 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
3531 B.addPatchset(['conflict'])
3532 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
3533 C.addPatchset(['conflict'])
3534 B.addApproval('CRVW', 2)
3535 C.addApproval('CRVW', 2)
3536 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
3537 self.fake_gerrit.addEvent(C.addApproval('APRV', 1))
3538 self.waitUntilSettled()
3539
3540 self.assertEqual(6, len(self.history)) # A and B jobs
3541 self.assertEqual(1, len(self.smtp_messages))
3542 self.assertEqual('The merge failed! For more information...',
3543 self.smtp_messages[0]['body'])
Joshua Hesketh36c3fa52014-01-22 11:40:52 +11003544
James E. Blairf760f0e2016-02-09 08:44:52 -08003545 def test_default_merge_failure_reports(self):
3546 """Check that the default merge failure reports are correct."""
3547
3548 # A should report success, B should report merge failure.
3549 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
3550 A.addPatchset(['conflict'])
3551 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
3552 B.addPatchset(['conflict'])
3553 A.addApproval('CRVW', 2)
3554 B.addApproval('CRVW', 2)
3555 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3556 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
3557 self.waitUntilSettled()
3558
3559 self.assertEqual(3, len(self.history)) # A jobs
3560 self.assertEqual(A.reported, 2)
3561 self.assertEqual(B.reported, 2)
3562 self.assertEqual(A.data['status'], 'MERGED')
3563 self.assertEqual(B.data['status'], 'NEW')
3564 self.assertIn('Build succeeded', A.messages[1])
3565 self.assertIn('Merge Failed', B.messages[1])
3566 self.assertIn('automatically merged', B.messages[1])
3567 self.assertNotIn('logs.example.com', B.messages[1])
3568 self.assertNotIn('SKIPPED', B.messages[1])
3569
Joshua Hesketh36c3fa52014-01-22 11:40:52 +11003570 def test_swift_instructions(self):
3571 "Test that the correct swift instructions are sent to the workers"
3572 self.config.set('zuul', 'layout_config',
3573 'tests/fixtures/layout-swift.yaml')
3574 self.sched.reconfigure(self.config)
3575 self.registerJobs()
3576
3577 self.worker.hold_jobs_in_build = True
3578 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
3579
3580 A.addApproval('CRVW', 2)
3581 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3582 self.waitUntilSettled()
3583
3584 self.assertEqual(
3585 "https://storage.example.org/V1/AUTH_account/merge_logs/1/1/1/"
3586 "gate/test-merge/",
Joshua Hesketh76dee532014-07-03 15:39:13 +10003587 self.builds[0].parameters['SWIFT_logs_URL'][:-7])
Joshua Hesketh36c3fa52014-01-22 11:40:52 +11003588 self.assertEqual(5,
3589 len(self.builds[0].parameters['SWIFT_logs_HMAC_BODY'].
3590 split('\n')))
3591 self.assertIn('SWIFT_logs_SIGNATURE', self.builds[0].parameters)
3592
3593 self.assertEqual(
3594 "https://storage.example.org/V1/AUTH_account/logs/1/1/1/"
3595 "gate/test-test/",
Joshua Hesketh76dee532014-07-03 15:39:13 +10003596 self.builds[1].parameters['SWIFT_logs_URL'][:-7])
Joshua Hesketh36c3fa52014-01-22 11:40:52 +11003597 self.assertEqual(5,
3598 len(self.builds[1].parameters['SWIFT_logs_HMAC_BODY'].
3599 split('\n')))
3600 self.assertIn('SWIFT_logs_SIGNATURE', self.builds[1].parameters)
3601
3602 self.assertEqual(
3603 "https://storage.example.org/V1/AUTH_account/stash/1/1/1/"
3604 "gate/test-test/",
Joshua Hesketh76dee532014-07-03 15:39:13 +10003605 self.builds[1].parameters['SWIFT_MOSTLY_URL'][:-7])
Joshua Hesketh36c3fa52014-01-22 11:40:52 +11003606 self.assertEqual(5,
3607 len(self.builds[1].
3608 parameters['SWIFT_MOSTLY_HMAC_BODY'].split('\n')))
3609 self.assertIn('SWIFT_MOSTLY_SIGNATURE', self.builds[1].parameters)
3610
3611 self.worker.hold_jobs_in_build = False
3612 self.worker.release()
3613 self.waitUntilSettled()
Joshua Hesketh85af4e92014-02-21 08:28:58 -08003614
3615 def test_client_get_running_jobs(self):
3616 "Test that the RPC client can get a list of running jobs"
3617 self.worker.hold_jobs_in_build = True
3618 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
3619 A.addApproval('CRVW', 2)
3620 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3621 self.waitUntilSettled()
3622
3623 client = zuul.rpcclient.RPCClient('127.0.0.1',
3624 self.gearman_server.port)
3625
3626 # Wait for gearman server to send the initial workData back to zuul
3627 start = time.time()
3628 while True:
3629 if time.time() - start > 10:
3630 raise Exception("Timeout waiting for gearman server to report "
3631 + "back to the client")
3632 build = self.launcher.builds.values()[0]
3633 if build.worker.name == "My Worker":
3634 break
3635 else:
3636 time.sleep(0)
3637
3638 running_items = client.get_running_jobs()
3639
3640 self.assertEqual(1, len(running_items))
3641 running_item = running_items[0]
3642 self.assertEqual([], running_item['failing_reasons'])
3643 self.assertEqual([], running_item['items_behind'])
3644 self.assertEqual('https://hostname/1', running_item['url'])
3645 self.assertEqual(None, running_item['item_ahead'])
3646 self.assertEqual('org/project', running_item['project'])
3647 self.assertEqual(None, running_item['remaining_time'])
3648 self.assertEqual(True, running_item['active'])
3649 self.assertEqual('1,1', running_item['id'])
3650
3651 self.assertEqual(3, len(running_item['jobs']))
3652 for job in running_item['jobs']:
3653 if job['name'] == 'project-merge':
3654 self.assertEqual('project-merge', job['name'])
3655 self.assertEqual('gate', job['pipeline'])
3656 self.assertEqual(False, job['retry'])
Joshua Hesketh85af4e92014-02-21 08:28:58 -08003657 self.assertEqual('https://server/job/project-merge/0/',
3658 job['url'])
3659 self.assertEqual(7, len(job['worker']))
3660 self.assertEqual(False, job['canceled'])
3661 self.assertEqual(True, job['voting'])
3662 self.assertEqual(None, job['result'])
3663 self.assertEqual('gate', job['pipeline'])
3664 break
3665
3666 self.worker.hold_jobs_in_build = False
3667 self.worker.release()
3668 self.waitUntilSettled()
3669
3670 running_items = client.get_running_jobs()
3671 self.assertEqual(0, len(running_items))
James E. Blairbadc1ad2014-04-28 13:55:14 -07003672
3673 def test_nonvoting_pipeline(self):
3674 "Test that a nonvoting pipeline (experimental) can still report"
3675
Joshua Heskethcc017ea2014-04-30 19:55:25 +10003676 A = self.fake_gerrit.addFakeChange('org/experimental-project',
3677 'master', 'A')
James E. Blairbadc1ad2014-04-28 13:55:14 -07003678 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
3679 self.waitUntilSettled()
Joshua Heskethcc017ea2014-04-30 19:55:25 +10003680 self.assertEqual(
3681 self.getJobFromHistory('experimental-project-test').result,
3682 'SUCCESS')
James E. Blairbadc1ad2014-04-28 13:55:14 -07003683 self.assertEqual(A.reported, 1)
James E. Blair5ee24252014-12-30 10:12:29 -08003684
3685 def test_crd_gate(self):
3686 "Test cross-repo dependencies"
3687 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
3688 B = self.fake_gerrit.addFakeChange('org/project2', 'master', 'B')
3689 A.addApproval('CRVW', 2)
3690 B.addApproval('CRVW', 2)
3691
3692 AM2 = self.fake_gerrit.addFakeChange('org/project1', 'master', 'AM2')
3693 AM1 = self.fake_gerrit.addFakeChange('org/project1', 'master', 'AM1')
3694 AM2.setMerged()
3695 AM1.setMerged()
3696
3697 BM2 = self.fake_gerrit.addFakeChange('org/project2', 'master', 'BM2')
3698 BM1 = self.fake_gerrit.addFakeChange('org/project2', 'master', 'BM1')
3699 BM2.setMerged()
3700 BM1.setMerged()
3701
3702 # A -> AM1 -> AM2
3703 # B -> BM1 -> BM2
3704 # A Depends-On: B
3705 # M2 is here to make sure it is never queried. If it is, it
3706 # means zuul is walking down the entire history of merged
3707 # changes.
3708
3709 B.setDependsOn(BM1, 1)
3710 BM1.setDependsOn(BM2, 1)
3711
3712 A.setDependsOn(AM1, 1)
3713 AM1.setDependsOn(AM2, 1)
3714
3715 A.data['commitMessage'] = '%s\n\nDepends-On: %s\n' % (
3716 A.subject, B.data['id'])
3717
3718 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3719 self.waitUntilSettled()
3720
3721 self.assertEqual(A.data['status'], 'NEW')
3722 self.assertEqual(B.data['status'], 'NEW')
3723
Joshua Hesketh4bd7da32016-02-17 20:58:47 +11003724 for connection in self.connections.values():
3725 connection.maintainCache([])
James E. Blair5ee24252014-12-30 10:12:29 -08003726
3727 self.worker.hold_jobs_in_build = True
3728 B.addApproval('APRV', 1)
3729 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3730 self.waitUntilSettled()
3731
3732 self.worker.release('.*-merge')
3733 self.waitUntilSettled()
3734 self.worker.release('.*-merge')
3735 self.waitUntilSettled()
3736 self.worker.hold_jobs_in_build = False
3737 self.worker.release()
3738 self.waitUntilSettled()
3739
3740 self.assertEqual(AM2.queried, 0)
3741 self.assertEqual(BM2.queried, 0)
3742 self.assertEqual(A.data['status'], 'MERGED')
3743 self.assertEqual(B.data['status'], 'MERGED')
3744 self.assertEqual(A.reported, 2)
3745 self.assertEqual(B.reported, 2)
3746
James E. Blair8f78d882015-02-05 08:51:37 -08003747 self.assertEqual(self.getJobFromHistory('project1-merge').changes,
3748 '2,1 1,1')
3749
3750 def test_crd_branch(self):
3751 "Test cross-repo dependencies in multiple branches"
3752 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
3753 B = self.fake_gerrit.addFakeChange('org/project2', 'master', 'B')
3754 C = self.fake_gerrit.addFakeChange('org/project2', 'mp', 'C')
3755 C.data['id'] = B.data['id']
3756 A.addApproval('CRVW', 2)
3757 B.addApproval('CRVW', 2)
3758 C.addApproval('CRVW', 2)
3759
3760 # A Depends-On: B+C
3761 A.data['commitMessage'] = '%s\n\nDepends-On: %s\n' % (
3762 A.subject, B.data['id'])
3763
3764 self.worker.hold_jobs_in_build = True
3765 B.addApproval('APRV', 1)
3766 C.addApproval('APRV', 1)
3767 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3768 self.waitUntilSettled()
3769
3770 self.worker.release('.*-merge')
3771 self.waitUntilSettled()
3772 self.worker.release('.*-merge')
3773 self.waitUntilSettled()
3774 self.worker.release('.*-merge')
3775 self.waitUntilSettled()
3776 self.worker.hold_jobs_in_build = False
3777 self.worker.release()
3778 self.waitUntilSettled()
3779
3780 self.assertEqual(A.data['status'], 'MERGED')
3781 self.assertEqual(B.data['status'], 'MERGED')
3782 self.assertEqual(C.data['status'], 'MERGED')
3783 self.assertEqual(A.reported, 2)
3784 self.assertEqual(B.reported, 2)
3785 self.assertEqual(C.reported, 2)
3786
3787 self.assertEqual(self.getJobFromHistory('project1-merge').changes,
3788 '2,1 3,1 1,1')
3789
3790 def test_crd_multiline(self):
3791 "Test multiple depends-on lines in commit"
3792 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
3793 B = self.fake_gerrit.addFakeChange('org/project2', 'master', 'B')
3794 C = self.fake_gerrit.addFakeChange('org/project2', 'master', 'C')
3795 A.addApproval('CRVW', 2)
3796 B.addApproval('CRVW', 2)
3797 C.addApproval('CRVW', 2)
3798
3799 # A Depends-On: B+C
3800 A.data['commitMessage'] = '%s\n\nDepends-On: %s\nDepends-On: %s\n' % (
3801 A.subject, B.data['id'], C.data['id'])
3802
3803 self.worker.hold_jobs_in_build = True
3804 B.addApproval('APRV', 1)
3805 C.addApproval('APRV', 1)
3806 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3807 self.waitUntilSettled()
3808
3809 self.worker.release('.*-merge')
3810 self.waitUntilSettled()
3811 self.worker.release('.*-merge')
3812 self.waitUntilSettled()
3813 self.worker.release('.*-merge')
3814 self.waitUntilSettled()
3815 self.worker.hold_jobs_in_build = False
3816 self.worker.release()
3817 self.waitUntilSettled()
3818
3819 self.assertEqual(A.data['status'], 'MERGED')
3820 self.assertEqual(B.data['status'], 'MERGED')
3821 self.assertEqual(C.data['status'], 'MERGED')
3822 self.assertEqual(A.reported, 2)
3823 self.assertEqual(B.reported, 2)
3824 self.assertEqual(C.reported, 2)
3825
3826 self.assertEqual(self.getJobFromHistory('project1-merge').changes,
3827 '2,1 3,1 1,1')
James E. Blair5ee24252014-12-30 10:12:29 -08003828
3829 def test_crd_unshared_gate(self):
3830 "Test cross-repo dependencies in unshared gate queues"
3831 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
3832 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
3833 A.addApproval('CRVW', 2)
3834 B.addApproval('CRVW', 2)
3835
3836 # A Depends-On: B
3837 A.data['commitMessage'] = '%s\n\nDepends-On: %s\n' % (
3838 A.subject, B.data['id'])
3839
3840 # A and B do not share a queue, make sure that A is unable to
3841 # enqueue B (and therefore, A is unable to be enqueued).
3842 B.addApproval('APRV', 1)
3843 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3844 self.waitUntilSettled()
3845
3846 self.assertEqual(A.data['status'], 'NEW')
3847 self.assertEqual(B.data['status'], 'NEW')
3848 self.assertEqual(A.reported, 0)
3849 self.assertEqual(B.reported, 0)
3850 self.assertEqual(len(self.history), 0)
3851
3852 # Enqueue and merge B alone.
3853 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
3854 self.waitUntilSettled()
3855
3856 self.assertEqual(B.data['status'], 'MERGED')
3857 self.assertEqual(B.reported, 2)
3858
3859 # Now that B is merged, A should be able to be enqueued and
3860 # merged.
3861 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3862 self.waitUntilSettled()
3863
3864 self.assertEqual(A.data['status'], 'MERGED')
3865 self.assertEqual(A.reported, 2)
3866
James E. Blair96698e22015-04-02 07:48:21 -07003867 def test_crd_gate_reverse(self):
3868 "Test reverse cross-repo dependencies"
3869 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
3870 B = self.fake_gerrit.addFakeChange('org/project2', 'master', 'B')
3871 A.addApproval('CRVW', 2)
3872 B.addApproval('CRVW', 2)
3873
3874 # A Depends-On: B
3875
3876 A.data['commitMessage'] = '%s\n\nDepends-On: %s\n' % (
3877 A.subject, B.data['id'])
3878
3879 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3880 self.waitUntilSettled()
3881
3882 self.assertEqual(A.data['status'], 'NEW')
3883 self.assertEqual(B.data['status'], 'NEW')
3884
3885 self.worker.hold_jobs_in_build = True
3886 A.addApproval('APRV', 1)
3887 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
3888 self.waitUntilSettled()
3889
3890 self.worker.release('.*-merge')
3891 self.waitUntilSettled()
3892 self.worker.release('.*-merge')
3893 self.waitUntilSettled()
3894 self.worker.hold_jobs_in_build = False
3895 self.worker.release()
3896 self.waitUntilSettled()
3897
3898 self.assertEqual(A.data['status'], 'MERGED')
3899 self.assertEqual(B.data['status'], 'MERGED')
3900 self.assertEqual(A.reported, 2)
3901 self.assertEqual(B.reported, 2)
3902
3903 self.assertEqual(self.getJobFromHistory('project1-merge').changes,
3904 '2,1 1,1')
3905
James E. Blair5ee24252014-12-30 10:12:29 -08003906 def test_crd_cycle(self):
3907 "Test cross-repo dependency cycles"
3908 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
3909 B = self.fake_gerrit.addFakeChange('org/project2', 'master', 'B')
3910 A.addApproval('CRVW', 2)
3911 B.addApproval('CRVW', 2)
3912
3913 # A -> B -> A (via commit-depends)
3914
3915 A.data['commitMessage'] = '%s\n\nDepends-On: %s\n' % (
3916 A.subject, B.data['id'])
3917 B.data['commitMessage'] = '%s\n\nDepends-On: %s\n' % (
3918 B.subject, A.data['id'])
3919
3920 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3921 self.waitUntilSettled()
3922
3923 self.assertEqual(A.reported, 0)
3924 self.assertEqual(B.reported, 0)
3925 self.assertEqual(A.data['status'], 'NEW')
3926 self.assertEqual(B.data['status'], 'NEW')
James E. Blairbfb8e042014-12-30 17:01:44 -08003927
Evgeny Antyshev0deaaad2015-08-03 20:22:56 +00003928 def test_crd_gate_unknown(self):
3929 "Test unknown projects in dependent pipeline"
3930 self.init_repo("org/unknown")
3931 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
3932 B = self.fake_gerrit.addFakeChange('org/unknown', 'master', 'B')
3933 A.addApproval('CRVW', 2)
3934 B.addApproval('CRVW', 2)
3935
3936 # A Depends-On: B
3937 A.data['commitMessage'] = '%s\n\nDepends-On: %s\n' % (
3938 A.subject, B.data['id'])
3939
3940 B.addApproval('APRV', 1)
3941 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3942 self.waitUntilSettled()
3943
3944 # Unknown projects cannot share a queue with any other
3945 # since they don't have common jobs with any other (they have no jobs).
3946 # Changes which depend on unknown project changes
3947 # should not be processed in dependent pipeline
3948 self.assertEqual(A.data['status'], 'NEW')
3949 self.assertEqual(B.data['status'], 'NEW')
3950 self.assertEqual(A.reported, 0)
3951 self.assertEqual(B.reported, 0)
3952 self.assertEqual(len(self.history), 0)
3953
3954 # Simulate change B being gated outside this layout
3955 self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
3956 B.setMerged()
3957 self.waitUntilSettled()
3958 self.assertEqual(len(self.history), 0)
3959
3960 # Now that B is merged, A should be able to be enqueued and
3961 # merged.
3962 self.fake_gerrit.addEvent(A.addApproval('APRV', 1))
3963 self.waitUntilSettled()
3964
3965 self.assertEqual(A.data['status'], 'MERGED')
3966 self.assertEqual(A.reported, 2)
3967 self.assertEqual(B.data['status'], 'MERGED')
3968 self.assertEqual(B.reported, 0)
3969
James E. Blairbfb8e042014-12-30 17:01:44 -08003970 def test_crd_check(self):
3971 "Test cross-repo dependencies in independent pipelines"
3972
3973 self.gearman_server.hold_jobs_in_queue = True
3974 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
3975 B = self.fake_gerrit.addFakeChange('org/project2', 'master', 'B')
3976
3977 # A Depends-On: B
3978 A.data['commitMessage'] = '%s\n\nDepends-On: %s\n' % (
3979 A.subject, B.data['id'])
3980
3981 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
3982 self.waitUntilSettled()
3983
3984 queue = self.gearman_server.getQueue()
3985 ref = self.getParameter(queue[-1], 'ZUUL_REF')
3986 self.gearman_server.hold_jobs_in_queue = False
3987 self.gearman_server.release()
3988 self.waitUntilSettled()
3989
3990 path = os.path.join(self.git_root, "org/project1")
3991 repo = git.Repo(path)
3992 repo_messages = [c.message.strip() for c in repo.iter_commits(ref)]
3993 repo_messages.reverse()
3994 correct_messages = ['initial commit', 'A-1']
3995 self.assertEqual(repo_messages, correct_messages)
3996
3997 path = os.path.join(self.git_root, "org/project2")
3998 repo = git.Repo(path)
3999 repo_messages = [c.message.strip() for c in repo.iter_commits(ref)]
4000 repo_messages.reverse()
4001 correct_messages = ['initial commit', 'B-1']
4002 self.assertEqual(repo_messages, correct_messages)
4003
4004 self.assertEqual(A.data['status'], 'NEW')
4005 self.assertEqual(B.data['status'], 'NEW')
4006 self.assertEqual(A.reported, 1)
4007 self.assertEqual(B.reported, 0)
4008
4009 self.assertEqual(self.history[0].changes, '2,1 1,1')
4010 self.assertEqual(len(self.sched.layout.pipelines['check'].queues), 0)
James E. Blair8f78d882015-02-05 08:51:37 -08004011
James E. Blairdbfe1cd2015-02-07 11:41:19 -08004012 def test_crd_check_git_depends(self):
4013 "Test single-repo dependencies in independent pipelines"
James E. Blairb8c16472015-05-05 14:55:26 -07004014 self.gearman_server.hold_jobs_in_build = True
James E. Blairdbfe1cd2015-02-07 11:41:19 -08004015 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
4016 B = self.fake_gerrit.addFakeChange('org/project1', 'master', 'B')
4017
4018 # Add two git-dependent changes and make sure they both report
4019 # success.
4020 B.setDependsOn(A, 1)
4021 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
4022 self.waitUntilSettled()
4023 self.fake_gerrit.addEvent(B.getPatchsetCreatedEvent(1))
4024 self.waitUntilSettled()
4025
James E. Blairb8c16472015-05-05 14:55:26 -07004026 self.orderedRelease()
4027 self.gearman_server.hold_jobs_in_build = False
James E. Blairdbfe1cd2015-02-07 11:41:19 -08004028 self.waitUntilSettled()
4029
4030 self.assertEqual(A.data['status'], 'NEW')
4031 self.assertEqual(B.data['status'], 'NEW')
4032 self.assertEqual(A.reported, 1)
4033 self.assertEqual(B.reported, 1)
4034
4035 self.assertEqual(self.history[0].changes, '1,1')
4036 self.assertEqual(self.history[-1].changes, '1,1 2,1')
4037 self.assertEqual(len(self.sched.layout.pipelines['check'].queues), 0)
4038
4039 self.assertIn('Build succeeded', A.messages[0])
4040 self.assertIn('Build succeeded', B.messages[0])
4041
4042 def test_crd_check_duplicate(self):
4043 "Test duplicate check in independent pipelines"
James E. Blair1e263032015-05-07 14:35:34 -07004044 self.worker.hold_jobs_in_build = True
James E. Blairdbfe1cd2015-02-07 11:41:19 -08004045 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
4046 B = self.fake_gerrit.addFakeChange('org/project1', 'master', 'B')
4047 check_pipeline = self.sched.layout.pipelines['check']
4048
4049 # Add two git-dependent changes...
4050 B.setDependsOn(A, 1)
4051 self.fake_gerrit.addEvent(B.getPatchsetCreatedEvent(1))
4052 self.waitUntilSettled()
4053 self.assertEqual(len(check_pipeline.getAllItems()), 2)
4054
4055 # ...make sure the live one is not duplicated...
4056 self.fake_gerrit.addEvent(B.getPatchsetCreatedEvent(1))
4057 self.waitUntilSettled()
4058 self.assertEqual(len(check_pipeline.getAllItems()), 2)
4059
4060 # ...but the non-live one is able to be.
4061 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
4062 self.waitUntilSettled()
4063 self.assertEqual(len(check_pipeline.getAllItems()), 3)
4064
Clark Boylandd849822015-03-02 12:38:14 -08004065 # Release jobs in order to avoid races with change A jobs
4066 # finishing before change B jobs.
James E. Blaird7650852015-05-07 15:47:37 -07004067 self.orderedRelease()
James E. Blair1e263032015-05-07 14:35:34 -07004068 self.worker.hold_jobs_in_build = False
4069 self.worker.release()
James E. Blairdbfe1cd2015-02-07 11:41:19 -08004070 self.waitUntilSettled()
4071
4072 self.assertEqual(A.data['status'], 'NEW')
4073 self.assertEqual(B.data['status'], 'NEW')
4074 self.assertEqual(A.reported, 1)
4075 self.assertEqual(B.reported, 1)
4076
4077 self.assertEqual(self.history[0].changes, '1,1 2,1')
4078 self.assertEqual(self.history[1].changes, '1,1')
4079 self.assertEqual(len(self.sched.layout.pipelines['check'].queues), 0)
4080
4081 self.assertIn('Build succeeded', A.messages[0])
4082 self.assertIn('Build succeeded', B.messages[0])
4083
Evgeny Antyshev0deaaad2015-08-03 20:22:56 +00004084 def _test_crd_check_reconfiguration(self, project1, project2):
James E. Blair8f78d882015-02-05 08:51:37 -08004085 "Test cross-repo dependencies re-enqueued in independent pipelines"
4086
4087 self.gearman_server.hold_jobs_in_queue = True
Evgeny Antyshev0deaaad2015-08-03 20:22:56 +00004088 A = self.fake_gerrit.addFakeChange(project1, 'master', 'A')
4089 B = self.fake_gerrit.addFakeChange(project2, 'master', 'B')
James E. Blair8f78d882015-02-05 08:51:37 -08004090
4091 # A Depends-On: B
4092 A.data['commitMessage'] = '%s\n\nDepends-On: %s\n' % (
4093 A.subject, B.data['id'])
4094
4095 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
4096 self.waitUntilSettled()
4097
4098 self.sched.reconfigure(self.config)
4099
4100 # Make sure the items still share a change queue, and the
4101 # first one is not live.
4102 self.assertEqual(len(self.sched.layout.pipelines['check'].queues), 1)
4103 queue = self.sched.layout.pipelines['check'].queues[0]
4104 first_item = queue.queue[0]
4105 for item in queue.queue:
4106 self.assertEqual(item.queue, first_item.queue)
4107 self.assertFalse(first_item.live)
4108 self.assertTrue(queue.queue[1].live)
4109
4110 self.gearman_server.hold_jobs_in_queue = False
4111 self.gearman_server.release()
4112 self.waitUntilSettled()
4113
4114 self.assertEqual(A.data['status'], 'NEW')
4115 self.assertEqual(B.data['status'], 'NEW')
4116 self.assertEqual(A.reported, 1)
4117 self.assertEqual(B.reported, 0)
4118
4119 self.assertEqual(self.history[0].changes, '2,1 1,1')
4120 self.assertEqual(len(self.sched.layout.pipelines['check'].queues), 0)
James E. Blair17dd6772015-02-09 14:45:18 -08004121
Evgeny Antyshev0deaaad2015-08-03 20:22:56 +00004122 def test_crd_check_reconfiguration(self):
4123 self._test_crd_check_reconfiguration('org/project1', 'org/project2')
4124
4125 def test_crd_undefined_project(self):
4126 """Test that undefined projects in dependencies are handled for
4127 independent pipelines"""
4128 # It's a hack for fake gerrit,
4129 # as it implies repo creation upon the creation of any change
4130 self.init_repo("org/unknown")
4131 self._test_crd_check_reconfiguration('org/project1', 'org/unknown')
4132
James E. Blair17dd6772015-02-09 14:45:18 -08004133 def test_crd_check_ignore_dependencies(self):
4134 "Test cross-repo dependencies can be ignored"
4135 self.config.set('zuul', 'layout_config',
4136 'tests/fixtures/layout-ignore-dependencies.yaml')
4137 self.sched.reconfigure(self.config)
4138 self.registerJobs()
4139
4140 self.gearman_server.hold_jobs_in_queue = True
4141 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
4142 B = self.fake_gerrit.addFakeChange('org/project2', 'master', 'B')
4143 C = self.fake_gerrit.addFakeChange('org/project2', 'master', 'C')
4144
4145 # A Depends-On: B
4146 A.data['commitMessage'] = '%s\n\nDepends-On: %s\n' % (
4147 A.subject, B.data['id'])
4148 # C git-depends on B
4149 C.setDependsOn(B, 1)
4150 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
4151 self.fake_gerrit.addEvent(B.getPatchsetCreatedEvent(1))
4152 self.fake_gerrit.addEvent(C.getPatchsetCreatedEvent(1))
4153 self.waitUntilSettled()
4154
4155 # Make sure none of the items share a change queue, and all
4156 # are live.
4157 check_pipeline = self.sched.layout.pipelines['check']
4158 self.assertEqual(len(check_pipeline.queues), 3)
4159 self.assertEqual(len(check_pipeline.getAllItems()), 3)
4160 for item in check_pipeline.getAllItems():
4161 self.assertTrue(item.live)
4162
4163 self.gearman_server.hold_jobs_in_queue = False
4164 self.gearman_server.release()
4165 self.waitUntilSettled()
4166
4167 self.assertEqual(A.data['status'], 'NEW')
4168 self.assertEqual(B.data['status'], 'NEW')
4169 self.assertEqual(C.data['status'], 'NEW')
4170 self.assertEqual(A.reported, 1)
4171 self.assertEqual(B.reported, 1)
4172 self.assertEqual(C.reported, 1)
4173
4174 # Each job should have tested exactly one change
4175 for job in self.history:
4176 self.assertEqual(len(job.changes.split()), 1)
James E. Blair96698e22015-04-02 07:48:21 -07004177
4178 def test_crd_check_transitive(self):
4179 "Test transitive cross-repo dependencies"
4180 # Specifically, if A -> B -> C, and C gets a new patchset and
4181 # A gets a new patchset, ensure the test of A,2 includes B,1
4182 # and C,2 (not C,1 which would indicate stale data in the
4183 # cache for B).
4184 A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
4185 B = self.fake_gerrit.addFakeChange('org/project2', 'master', 'B')
4186 C = self.fake_gerrit.addFakeChange('org/project3', 'master', 'C')
4187
4188 # A Depends-On: B
4189 A.data['commitMessage'] = '%s\n\nDepends-On: %s\n' % (
4190 A.subject, B.data['id'])
4191
4192 # B Depends-On: C
4193 B.data['commitMessage'] = '%s\n\nDepends-On: %s\n' % (
4194 B.subject, C.data['id'])
4195
4196 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
4197 self.waitUntilSettled()
4198 self.assertEqual(self.history[-1].changes, '3,1 2,1 1,1')
4199
4200 self.fake_gerrit.addEvent(B.getPatchsetCreatedEvent(1))
4201 self.waitUntilSettled()
4202 self.assertEqual(self.history[-1].changes, '3,1 2,1')
4203
4204 self.fake_gerrit.addEvent(C.getPatchsetCreatedEvent(1))
4205 self.waitUntilSettled()
4206 self.assertEqual(self.history[-1].changes, '3,1')
4207
4208 C.addPatchset()
4209 self.fake_gerrit.addEvent(C.getPatchsetCreatedEvent(2))
4210 self.waitUntilSettled()
4211 self.assertEqual(self.history[-1].changes, '3,2')
4212
4213 A.addPatchset()
4214 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(2))
4215 self.waitUntilSettled()
4216 self.assertEqual(self.history[-1].changes, '3,2 2,1 1,2')
Joshua Hesketh89e829d2015-02-10 16:29:45 +11004217
4218 def test_disable_at(self):
4219 "Test a pipeline will only report to the disabled trigger when failing"
4220
4221 self.config.set('zuul', 'layout_config',
4222 'tests/fixtures/layout-disable-at.yaml')
4223 self.sched.reconfigure(self.config)
4224
4225 self.assertEqual(3, self.sched.layout.pipelines['check'].disable_at)
4226 self.assertEqual(
4227 0, self.sched.layout.pipelines['check']._consecutive_failures)
4228 self.assertFalse(self.sched.layout.pipelines['check']._disabled)
4229
4230 A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
4231 B = self.fake_gerrit.addFakeChange('org/project', 'master', 'B')
4232 C = self.fake_gerrit.addFakeChange('org/project', 'master', 'C')
4233 D = self.fake_gerrit.addFakeChange('org/project', 'master', 'D')
4234 E = self.fake_gerrit.addFakeChange('org/project', 'master', 'E')
4235 F = self.fake_gerrit.addFakeChange('org/project', 'master', 'F')
4236 G = self.fake_gerrit.addFakeChange('org/project', 'master', 'G')
4237 H = self.fake_gerrit.addFakeChange('org/project', 'master', 'H')
4238 I = self.fake_gerrit.addFakeChange('org/project', 'master', 'I')
4239 J = self.fake_gerrit.addFakeChange('org/project', 'master', 'J')
4240 K = self.fake_gerrit.addFakeChange('org/project', 'master', 'K')
4241
4242 self.worker.addFailTest('project-test1', A)
4243 self.worker.addFailTest('project-test1', B)
4244 # Let C pass, resetting the counter
4245 self.worker.addFailTest('project-test1', D)
4246 self.worker.addFailTest('project-test1', E)
4247 self.worker.addFailTest('project-test1', F)
4248 self.worker.addFailTest('project-test1', G)
4249 self.worker.addFailTest('project-test1', H)
4250 # I also passes but should only report to the disabled reporters
4251 self.worker.addFailTest('project-test1', J)
4252 self.worker.addFailTest('project-test1', K)
4253
4254 self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
4255 self.fake_gerrit.addEvent(B.getPatchsetCreatedEvent(1))
4256 self.waitUntilSettled()
4257
4258 self.assertEqual(
4259 2, self.sched.layout.pipelines['check']._consecutive_failures)
4260 self.assertFalse(self.sched.layout.pipelines['check']._disabled)
4261
4262 self.fake_gerrit.addEvent(C.getPatchsetCreatedEvent(1))
4263 self.waitUntilSettled()
4264
4265 self.assertEqual(
4266 0, self.sched.layout.pipelines['check']._consecutive_failures)
4267 self.assertFalse(self.sched.layout.pipelines['check']._disabled)
4268
4269 self.fake_gerrit.addEvent(D.getPatchsetCreatedEvent(1))
4270 self.fake_gerrit.addEvent(E.getPatchsetCreatedEvent(1))
4271 self.fake_gerrit.addEvent(F.getPatchsetCreatedEvent(1))
4272 self.waitUntilSettled()
4273
4274 # We should be disabled now
4275 self.assertEqual(
4276 3, self.sched.layout.pipelines['check']._consecutive_failures)
4277 self.assertTrue(self.sched.layout.pipelines['check']._disabled)
4278
4279 # We need to wait between each of these patches to make sure the
4280 # smtp messages come back in an expected order
4281 self.fake_gerrit.addEvent(G.getPatchsetCreatedEvent(1))
4282 self.waitUntilSettled()
4283 self.fake_gerrit.addEvent(H.getPatchsetCreatedEvent(1))
4284 self.waitUntilSettled()
4285 self.fake_gerrit.addEvent(I.getPatchsetCreatedEvent(1))
4286 self.waitUntilSettled()
4287
4288 # The first 6 (ABCDEF) jobs should have reported back to gerrt thus
4289 # leaving a message on each change
4290 self.assertEqual(1, len(A.messages))
4291 self.assertIn('Build failed.', A.messages[0])
4292 self.assertEqual(1, len(B.messages))
4293 self.assertIn('Build failed.', B.messages[0])
4294 self.assertEqual(1, len(C.messages))
4295 self.assertIn('Build succeeded.', C.messages[0])
4296 self.assertEqual(1, len(D.messages))
4297 self.assertIn('Build failed.', D.messages[0])
4298 self.assertEqual(1, len(E.messages))
4299 self.assertIn('Build failed.', E.messages[0])
4300 self.assertEqual(1, len(F.messages))
4301 self.assertIn('Build failed.', F.messages[0])
4302
4303 # The last 3 (GHI) would have only reported via smtp.
4304 self.assertEqual(3, len(self.smtp_messages))
4305 self.assertEqual(0, len(G.messages))
4306 self.assertIn('Build failed.', self.smtp_messages[0]['body'])
4307 self.assertIn('/7/1/check', self.smtp_messages[0]['body'])
4308 self.assertEqual(0, len(H.messages))
4309 self.assertIn('Build failed.', self.smtp_messages[1]['body'])
4310 self.assertIn('/8/1/check', self.smtp_messages[1]['body'])
4311 self.assertEqual(0, len(I.messages))
4312 self.assertIn('Build succeeded.', self.smtp_messages[2]['body'])
4313 self.assertIn('/9/1/check', self.smtp_messages[2]['body'])
4314
4315 # Now reload the configuration (simulate a HUP) to check the pipeline
4316 # comes out of disabled
4317 self.sched.reconfigure(self.config)
4318
4319 self.assertEqual(3, self.sched.layout.pipelines['check'].disable_at)
4320 self.assertEqual(
4321 0, self.sched.layout.pipelines['check']._consecutive_failures)
4322 self.assertFalse(self.sched.layout.pipelines['check']._disabled)
4323
4324 self.fake_gerrit.addEvent(J.getPatchsetCreatedEvent(1))
4325 self.fake_gerrit.addEvent(K.getPatchsetCreatedEvent(1))
4326 self.waitUntilSettled()
4327
4328 self.assertEqual(
4329 2, self.sched.layout.pipelines['check']._consecutive_failures)
4330 self.assertFalse(self.sched.layout.pipelines['check']._disabled)
4331
4332 # J and K went back to gerrit
4333 self.assertEqual(1, len(J.messages))
4334 self.assertIn('Build failed.', J.messages[0])
4335 self.assertEqual(1, len(K.messages))
4336 self.assertIn('Build failed.', K.messages[0])
4337 # No more messages reported via smtp
4338 self.assertEqual(3, len(self.smtp_messages))