James E. Blair | dbfe1cd | 2015-02-07 11:41:19 -0800 | [diff] [blame] | 1 | # Copyright 2012-2015 Hewlett-Packard Development Company, L.P. |
James E. Blair | 4795838 | 2013-01-10 17:26:02 -0800 | [diff] [blame] | 2 | # Copyright 2013 OpenStack Foundation |
Antoine Musso | 80edd5a | 2013-02-13 15:37:53 +0100 | [diff] [blame] | 3 | # Copyright 2013 Antoine "hashar" Musso |
| 4 | # Copyright 2013 Wikimedia Foundation Inc. |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 5 | # |
| 6 | # Licensed under the Apache License, Version 2.0 (the "License"); you may |
| 7 | # not use this file except in compliance with the License. You may obtain |
| 8 | # a copy of the License at |
| 9 | # |
| 10 | # http://www.apache.org/licenses/LICENSE-2.0 |
| 11 | # |
| 12 | # Unless required by applicable law or agreed to in writing, software |
| 13 | # distributed under the License is distributed on an "AS IS" BASIS, WITHOUT |
| 14 | # WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the |
| 15 | # License for the specific language governing permissions and limitations |
| 16 | # under the License. |
| 17 | |
James E. Blair | 71e9412 | 2012-12-24 17:53:08 -0800 | [diff] [blame] | 18 | import extras |
James E. Blair | 8dbd56a | 2012-12-22 10:55:10 -0800 | [diff] [blame] | 19 | import json |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 20 | import logging |
Zhongyue Luo | 1c860d7 | 2012-07-19 11:03:56 +0800 | [diff] [blame] | 21 | import os |
James E. Blair | 5d5bc2b | 2012-07-06 10:24:01 -0700 | [diff] [blame] | 22 | import pickle |
Christian Berendt | 12d4d72 | 2014-06-07 21:03:45 +0200 | [diff] [blame] | 23 | from six.moves import queue as Queue |
Zhongyue Luo | 1c860d7 | 2012-07-19 11:03:56 +0800 | [diff] [blame] | 24 | import re |
James E. Blair | 36658cf | 2013-12-06 17:53:48 -0800 | [diff] [blame] | 25 | import sys |
Zhongyue Luo | 1c860d7 | 2012-07-19 11:03:56 +0800 | [diff] [blame] | 26 | import threading |
James E. Blair | 71e9412 | 2012-12-24 17:53:08 -0800 | [diff] [blame] | 27 | import time |
Zhongyue Luo | 1c860d7 | 2012-07-19 11:03:56 +0800 | [diff] [blame] | 28 | import yaml |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 29 | |
James E. Blair | 4795838 | 2013-01-10 17:26:02 -0800 | [diff] [blame] | 30 | import layoutvalidator |
James E. Blair | 4886cc1 | 2012-07-18 15:39:41 -0700 | [diff] [blame] | 31 | import model |
Joshua Hesketh | de95865 | 2015-11-10 19:19:50 +1100 | [diff] [blame] | 32 | from model import Pipeline, Project, ChangeQueue |
Joshua Hesketh | 70b1349 | 2015-08-11 23:40:42 +1000 | [diff] [blame] | 33 | from model import ChangeishFilter, NullChange |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 34 | from zuul import change_matcher, exceptions |
Sergey Lukjanov | 5ba961b | 2013-12-27 01:21:04 +0400 | [diff] [blame] | 35 | from zuul import version as zuul_version |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 36 | |
James E. Blair | 71e9412 | 2012-12-24 17:53:08 -0800 | [diff] [blame] | 37 | statsd = extras.try_import('statsd.statsd') |
| 38 | |
James E. Blair | 1e8dd89 | 2012-05-30 09:15:05 -0700 | [diff] [blame] | 39 | |
Antoine Musso | 80edd5a | 2013-02-13 15:37:53 +0100 | [diff] [blame] | 40 | def deep_format(obj, paramdict): |
| 41 | """Apply the paramdict via str.format() to all string objects found within |
| 42 | the supplied obj. Lists and dicts are traversed recursively. |
| 43 | |
| 44 | Borrowed from Jenkins Job Builder project""" |
| 45 | if isinstance(obj, str): |
| 46 | ret = obj.format(**paramdict) |
| 47 | elif isinstance(obj, list): |
| 48 | ret = [] |
| 49 | for item in obj: |
| 50 | ret.append(deep_format(item, paramdict)) |
| 51 | elif isinstance(obj, dict): |
| 52 | ret = {} |
| 53 | for item in obj: |
| 54 | exp_item = item.format(**paramdict) |
| 55 | |
| 56 | ret[exp_item] = deep_format(obj[item], paramdict) |
| 57 | else: |
| 58 | ret = obj |
| 59 | return ret |
| 60 | |
| 61 | |
James E. Blair | af17a97 | 2016-02-03 15:07:18 -0800 | [diff] [blame] | 62 | class MutexHandler(object): |
| 63 | log = logging.getLogger("zuul.MutexHandler") |
| 64 | |
| 65 | def __init__(self): |
| 66 | self.mutexes = {} |
| 67 | |
| 68 | def acquire(self, item, job): |
| 69 | if not job.mutex: |
| 70 | return True |
| 71 | mutex_name = job.mutex |
| 72 | m = self.mutexes.get(mutex_name) |
| 73 | if not m: |
| 74 | # The mutex is not held, acquire it |
| 75 | self._acquire(mutex_name, item, job.name) |
| 76 | return True |
| 77 | held_item, held_job_name = m |
| 78 | if held_item is item and held_job_name == job.name: |
| 79 | # This item already holds the mutex |
| 80 | return True |
| 81 | held_build = held_item.current_build_set.getBuild(held_job_name) |
| 82 | if held_build and held_build.result: |
| 83 | # The build that held the mutex is complete, release it |
| 84 | # and let the new item have it. |
| 85 | self.log.error("Held mutex %s being released because " |
| 86 | "the build that holds it is complete" % |
| 87 | (mutex_name,)) |
| 88 | self._release(mutex_name, item, job.name) |
| 89 | self._acquire(mutex_name, item, job.name) |
| 90 | return True |
| 91 | return False |
| 92 | |
| 93 | def release(self, item, job): |
| 94 | if not job.mutex: |
| 95 | return |
| 96 | mutex_name = job.mutex |
| 97 | m = self.mutexes.get(mutex_name) |
| 98 | if not m: |
| 99 | # The mutex is not held, nothing to do |
| 100 | self.log.error("Mutex can not be released for %s " |
| 101 | "because the mutex is not held" % |
| 102 | (item,)) |
| 103 | return |
| 104 | held_item, held_job_name = m |
| 105 | if held_item is item and held_job_name == job.name: |
| 106 | # This item holds the mutex |
| 107 | self._release(mutex_name, item, job.name) |
| 108 | return |
| 109 | self.log.error("Mutex can not be released for %s " |
| 110 | "which does not hold it" % |
| 111 | (item,)) |
| 112 | |
| 113 | def _acquire(self, mutex_name, item, job_name): |
| 114 | self.log.debug("Job %s of item %s acquiring mutex %s" % |
| 115 | (job_name, item, mutex_name)) |
| 116 | self.mutexes[mutex_name] = (item, job_name) |
| 117 | |
| 118 | def _release(self, mutex_name, item, job_name): |
| 119 | self.log.debug("Job %s of item %s releasing mutex %s" % |
| 120 | (job_name, item, mutex_name)) |
| 121 | del self.mutexes[mutex_name] |
| 122 | |
| 123 | |
James E. Blair | 468c851 | 2013-12-06 13:27:19 -0800 | [diff] [blame] | 124 | class ManagementEvent(object): |
| 125 | """An event that should be processed within the main queue run loop""" |
| 126 | def __init__(self): |
| 127 | self._wait_event = threading.Event() |
James E. Blair | 36658cf | 2013-12-06 17:53:48 -0800 | [diff] [blame] | 128 | self._exception = None |
| 129 | self._traceback = None |
James E. Blair | 468c851 | 2013-12-06 13:27:19 -0800 | [diff] [blame] | 130 | |
James E. Blair | 36658cf | 2013-12-06 17:53:48 -0800 | [diff] [blame] | 131 | def exception(self, e, tb): |
| 132 | self._exception = e |
| 133 | self._traceback = tb |
| 134 | self._wait_event.set() |
| 135 | |
| 136 | def done(self): |
James E. Blair | 468c851 | 2013-12-06 13:27:19 -0800 | [diff] [blame] | 137 | self._wait_event.set() |
| 138 | |
| 139 | def wait(self, timeout=None): |
| 140 | self._wait_event.wait(timeout) |
James E. Blair | 36658cf | 2013-12-06 17:53:48 -0800 | [diff] [blame] | 141 | if self._exception: |
| 142 | raise self._exception, None, self._traceback |
James E. Blair | 468c851 | 2013-12-06 13:27:19 -0800 | [diff] [blame] | 143 | return self._wait_event.is_set() |
| 144 | |
| 145 | |
| 146 | class ReconfigureEvent(ManagementEvent): |
| 147 | """Reconfigure the scheduler. The layout will be (re-)loaded from |
| 148 | the path specified in the configuration. |
| 149 | |
| 150 | :arg ConfigParser config: the new configuration |
| 151 | """ |
| 152 | def __init__(self, config): |
| 153 | super(ReconfigureEvent, self).__init__() |
| 154 | self.config = config |
| 155 | |
| 156 | |
James E. Blair | 36658cf | 2013-12-06 17:53:48 -0800 | [diff] [blame] | 157 | class PromoteEvent(ManagementEvent): |
| 158 | """Promote one or more changes to the head of the queue. |
| 159 | |
| 160 | :arg str pipeline_name: the name of the pipeline |
| 161 | :arg list change_ids: a list of strings of change ids in the form |
| 162 | 1234,1 |
| 163 | """ |
| 164 | |
| 165 | def __init__(self, pipeline_name, change_ids): |
| 166 | super(PromoteEvent, self).__init__() |
| 167 | self.pipeline_name = pipeline_name |
| 168 | self.change_ids = change_ids |
| 169 | |
| 170 | |
James E. Blair | d27a96d | 2014-07-10 13:25:13 -0700 | [diff] [blame] | 171 | class EnqueueEvent(ManagementEvent): |
| 172 | """Enqueue a change into a pipeline |
| 173 | |
| 174 | :arg TriggerEvent trigger_event: a TriggerEvent describing the |
| 175 | trigger, pipeline, and change to enqueue |
| 176 | """ |
| 177 | |
| 178 | def __init__(self, trigger_event): |
| 179 | super(EnqueueEvent, self).__init__() |
| 180 | self.trigger_event = trigger_event |
| 181 | |
| 182 | |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 183 | class ResultEvent(object): |
| 184 | """An event that needs to modify the pipeline state due to a |
| 185 | result from an external system.""" |
| 186 | |
| 187 | pass |
| 188 | |
| 189 | |
| 190 | class BuildStartedEvent(ResultEvent): |
| 191 | """A build has started. |
| 192 | |
| 193 | :arg Build build: The build which has started. |
| 194 | """ |
| 195 | |
| 196 | def __init__(self, build): |
| 197 | self.build = build |
| 198 | |
| 199 | |
| 200 | class BuildCompletedEvent(ResultEvent): |
| 201 | """A build has completed |
| 202 | |
| 203 | :arg Build build: The build which has completed. |
| 204 | """ |
| 205 | |
| 206 | def __init__(self, build): |
| 207 | self.build = build |
| 208 | |
| 209 | |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 210 | class MergeCompletedEvent(ResultEvent): |
| 211 | """A remote merge operation has completed |
| 212 | |
| 213 | :arg BuildSet build_set: The build_set which is ready. |
| 214 | :arg str zuul_url: The URL of the Zuul Merger. |
| 215 | :arg bool merged: Whether the merge succeeded (changes with refs). |
| 216 | :arg bool updated: Whether the repo was updated (changes without refs). |
| 217 | :arg str commit: The SHA of the merged commit (changes with refs). |
| 218 | """ |
| 219 | |
| 220 | def __init__(self, build_set, zuul_url, merged, updated, commit): |
| 221 | self.build_set = build_set |
| 222 | self.zuul_url = zuul_url |
| 223 | self.merged = merged |
| 224 | self.updated = updated |
| 225 | self.commit = commit |
| 226 | |
| 227 | |
Maru Newby | 3fe5f85 | 2015-01-13 04:22:14 +0000 | [diff] [blame] | 228 | def toList(item): |
| 229 | if not item: |
| 230 | return [] |
| 231 | if isinstance(item, list): |
| 232 | return item |
| 233 | return [item] |
| 234 | |
| 235 | |
James E. Blair | e9d45c3 | 2012-05-31 09:56:45 -0700 | [diff] [blame] | 236 | class Scheduler(threading.Thread): |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 237 | log = logging.getLogger("zuul.Scheduler") |
| 238 | |
James E. Blair | e4d229c | 2016-05-25 15:25:41 -0700 | [diff] [blame^] | 239 | def __init__(self, config, testonly=False): |
James E. Blair | e9d45c3 | 2012-05-31 09:56:45 -0700 | [diff] [blame] | 240 | threading.Thread.__init__(self) |
James E. Blair | 8a6f0c2 | 2013-07-01 12:31:34 -0400 | [diff] [blame] | 241 | self.daemon = True |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 242 | self.wake_event = threading.Event() |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 243 | self.layout_lock = threading.Lock() |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 244 | self.run_handler_lock = threading.Lock() |
James E. Blair | 5d5bc2b | 2012-07-06 10:24:01 -0700 | [diff] [blame] | 245 | self._pause = False |
James E. Blair | 5d5bc2b | 2012-07-06 10:24:01 -0700 | [diff] [blame] | 246 | self._exit = False |
James E. Blair | b0fcae4 | 2012-07-17 11:12:10 -0700 | [diff] [blame] | 247 | self._stopped = False |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 248 | self.launcher = None |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 249 | self.merger = None |
James E. Blair | af17a97 | 2016-02-03 15:07:18 -0800 | [diff] [blame] | 250 | self.mutex = MutexHandler() |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 251 | self.connections = dict() |
| 252 | # Despite triggers being part of the pipeline, there is one trigger set |
| 253 | # per scheduler. The pipeline handles the trigger filters but since |
| 254 | # the events are handled by the scheduler itself it needs to handle |
| 255 | # the loading of the triggers. |
| 256 | # self.triggers['connection_name'] = triggerObject |
James E. Blair | 6c358e7 | 2013-07-29 17:06:47 -0700 | [diff] [blame] | 257 | self.triggers = dict() |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 258 | self.config = config |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 259 | |
| 260 | self.trigger_event_queue = Queue.Queue() |
| 261 | self.result_event_queue = Queue.Queue() |
James E. Blair | 468c851 | 2013-12-06 13:27:19 -0800 | [diff] [blame] | 262 | self.management_event_queue = Queue.Queue() |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 263 | self.layout = model.Layout() |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 264 | |
James E. Blair | e4d229c | 2016-05-25 15:25:41 -0700 | [diff] [blame^] | 265 | if not testonly: |
| 266 | time_dir = self._get_time_database_dir() |
| 267 | self.time_database = model.TimeDataBase(time_dir) |
James E. Blair | ce8a213 | 2016-05-19 15:21:52 -0700 | [diff] [blame] | 268 | |
Jeremy Stanley | 98b38de | 2015-06-04 21:20:43 +0000 | [diff] [blame] | 269 | self.zuul_version = zuul_version.version_info.release_string() |
Sergey Lukjanov | 5d0438d | 2013-12-24 03:36:39 +0400 | [diff] [blame] | 270 | self.last_reconfigured = None |
Sergey Lukjanov | 5ba961b | 2013-12-27 01:21:04 +0400 | [diff] [blame] | 271 | |
Joshua Hesketh | de95865 | 2015-11-10 19:19:50 +1100 | [diff] [blame] | 272 | # A set of reporter configuration keys to action mapping |
| 273 | self._reporter_actions = { |
| 274 | 'start': 'start_actions', |
| 275 | 'success': 'success_actions', |
| 276 | 'failure': 'failure_actions', |
| 277 | 'merge-failure': 'merge_failure_actions', |
| 278 | 'disabled': 'disabled_actions', |
| 279 | } |
| 280 | |
James E. Blair | b0fcae4 | 2012-07-17 11:12:10 -0700 | [diff] [blame] | 281 | def stop(self): |
| 282 | self._stopped = True |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 283 | self._unloadDrivers() |
| 284 | self.stopConnections() |
James E. Blair | b0fcae4 | 2012-07-17 11:12:10 -0700 | [diff] [blame] | 285 | self.wake_event.set() |
| 286 | |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 287 | def testConfig(self, config_path, connections): |
| 288 | # Take the list of set up connections directly here rather than with |
| 289 | # registerConnections as we don't want to do the onLoad event yet. |
| 290 | return self._parseConfig(config_path, connections) |
James E. Blair | 4795838 | 2013-01-10 17:26:02 -0800 | [diff] [blame] | 291 | |
Maru Newby | 3fe5f85 | 2015-01-13 04:22:14 +0000 | [diff] [blame] | 292 | def _parseSkipIf(self, config_job): |
| 293 | cm = change_matcher |
| 294 | skip_matchers = [] |
| 295 | |
| 296 | for config_skip in config_job.get('skip-if', []): |
| 297 | nested_matchers = [] |
| 298 | |
| 299 | project_regex = config_skip.get('project') |
| 300 | if project_regex: |
| 301 | nested_matchers.append(cm.ProjectMatcher(project_regex)) |
| 302 | |
| 303 | branch_regex = config_skip.get('branch') |
| 304 | if branch_regex: |
| 305 | nested_matchers.append(cm.BranchMatcher(branch_regex)) |
| 306 | |
| 307 | file_regexes = toList(config_skip.get('all-files-match-any')) |
| 308 | if file_regexes: |
| 309 | file_matchers = [cm.FileMatcher(x) for x in file_regexes] |
| 310 | all_files_matcher = cm.MatchAllFiles(file_matchers) |
| 311 | nested_matchers.append(all_files_matcher) |
| 312 | |
| 313 | # All patterns need to match a given skip-if predicate |
| 314 | skip_matchers.append(cm.MatchAll(nested_matchers)) |
| 315 | |
| 316 | if skip_matchers: |
| 317 | # Any skip-if predicate can be matched to trigger a skip |
| 318 | return cm.MatchAny(skip_matchers) |
| 319 | |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 320 | def registerConnections(self, connections): |
| 321 | self.connections = connections |
| 322 | for connection_name, connection in self.connections.items(): |
| 323 | connection.registerScheduler(self) |
| 324 | connection.onLoad() |
| 325 | |
| 326 | def stopConnections(self): |
| 327 | for connection_name, connection in self.connections.items(): |
| 328 | connection.onStop() |
| 329 | |
| 330 | def _unloadDrivers(self): |
| 331 | for trigger in self.triggers.values(): |
| 332 | trigger.stop() |
Joshua Hesketh | 90b61db | 2016-02-03 14:22:15 +1100 | [diff] [blame] | 333 | self.triggers = {} |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 334 | for pipeline in self.layout.pipelines.values(): |
| 335 | pipeline.source.stop() |
Joshua Hesketh | de95865 | 2015-11-10 19:19:50 +1100 | [diff] [blame] | 336 | for action in self._reporter_actions.values(): |
| 337 | for reporter in pipeline.__getattribute__(action): |
| 338 | reporter.stop() |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 339 | |
| 340 | def _getDriver(self, dtype, connection_name, driver_config={}): |
| 341 | # Instantiate a driver such as a trigger, source or reporter |
| 342 | # TODO(jhesketh): Make this list dynamic or use entrypoints etc. |
| 343 | # Stevedore was not a good fit here due to the nature of triggers. |
| 344 | # Specifically we don't want to load a trigger per a pipeline as one |
| 345 | # trigger can listen to a stream (from gerrit, for example) and the |
| 346 | # scheduler decides which eventfilter to use. As such we want to load |
| 347 | # trigger+connection pairs uniquely. |
| 348 | drivers = { |
| 349 | 'source': { |
| 350 | 'gerrit': 'zuul.source.gerrit:GerritSource', |
| 351 | }, |
| 352 | 'trigger': { |
| 353 | 'gerrit': 'zuul.trigger.gerrit:GerritTrigger', |
| 354 | 'timer': 'zuul.trigger.timer:TimerTrigger', |
| 355 | 'zuul': 'zuul.trigger.zuultrigger:ZuulTrigger', |
| 356 | }, |
| 357 | 'reporter': { |
| 358 | 'gerrit': 'zuul.reporter.gerrit:GerritReporter', |
| 359 | 'smtp': 'zuul.reporter.smtp:SMTPReporter', |
| 360 | }, |
| 361 | } |
| 362 | |
| 363 | # TODO(jhesketh): Check the connection_name exists |
| 364 | if connection_name in self.connections.keys(): |
| 365 | driver_name = self.connections[connection_name].driver_name |
| 366 | connection = self.connections[connection_name] |
| 367 | else: |
| 368 | # In some cases a driver may not be related to a connection. For |
| 369 | # example, the 'timer' or 'zuul' triggers. |
| 370 | driver_name = connection_name |
| 371 | connection = None |
| 372 | driver = drivers[dtype][driver_name].split(':') |
| 373 | driver_instance = getattr( |
| 374 | __import__(driver[0], fromlist=['']), driver[1])( |
| 375 | driver_config, self, connection |
| 376 | ) |
| 377 | |
Joshua Hesketh | 811e2e9 | 2015-12-08 09:55:05 +1100 | [diff] [blame] | 378 | if connection: |
| 379 | connection.registerUse(dtype, driver_instance) |
| 380 | |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 381 | return driver_instance |
| 382 | |
| 383 | def _getSourceDriver(self, connection_name): |
| 384 | return self._getDriver('source', connection_name) |
| 385 | |
| 386 | def _getReporterDriver(self, connection_name, driver_config={}): |
| 387 | return self._getDriver('reporter', connection_name, driver_config) |
| 388 | |
| 389 | def _getTriggerDriver(self, connection_name, driver_config={}): |
| 390 | return self._getDriver('trigger', connection_name, driver_config) |
| 391 | |
| 392 | def _parseConfig(self, config_path, connections): |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 393 | layout = model.Layout() |
| 394 | project_templates = {} |
| 395 | |
James E. Blair | e5a847f | 2012-07-10 15:29:14 -0700 | [diff] [blame] | 396 | if config_path: |
| 397 | config_path = os.path.expanduser(config_path) |
| 398 | if not os.path.exists(config_path): |
| 399 | raise Exception("Unable to read layout config file at %s" % |
| 400 | config_path) |
Einst Crazy | ff9837b | 2015-11-17 17:32:37 +0800 | [diff] [blame] | 401 | with open(config_path) as config_file: |
| 402 | data = yaml.load(config_file) |
James E. Blair | e5a847f | 2012-07-10 15:29:14 -0700 | [diff] [blame] | 403 | |
James E. Blair | 4795838 | 2013-01-10 17:26:02 -0800 | [diff] [blame] | 404 | validator = layoutvalidator.LayoutValidator() |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 405 | validator.validate(data, connections) |
James E. Blair | 4795838 | 2013-01-10 17:26:02 -0800 | [diff] [blame] | 406 | |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 407 | config_env = {} |
James E. Blair | e5a847f | 2012-07-10 15:29:14 -0700 | [diff] [blame] | 408 | for include in data.get('includes', []): |
| 409 | if 'python-file' in include: |
| 410 | fn = include['python-file'] |
| 411 | if not os.path.isabs(fn): |
Antoine Musso | 9adc6d4 | 2014-11-14 15:37:48 +0100 | [diff] [blame] | 412 | base = os.path.dirname(os.path.realpath(config_path)) |
James E. Blair | e5a847f | 2012-07-10 15:29:14 -0700 | [diff] [blame] | 413 | fn = os.path.join(base, fn) |
| 414 | fn = os.path.expanduser(fn) |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 415 | execfile(fn, config_env) |
James E. Blair | 1e8dd89 | 2012-05-30 09:15:05 -0700 | [diff] [blame] | 416 | |
James E. Blair | 4aea70c | 2012-07-26 14:23:24 -0700 | [diff] [blame] | 417 | for conf_pipeline in data.get('pipelines', []): |
| 418 | pipeline = Pipeline(conf_pipeline['name']) |
James E. Blair | 8dbd56a | 2012-12-22 10:55:10 -0800 | [diff] [blame] | 419 | pipeline.description = conf_pipeline.get('description') |
James E. Blair | c0dedf8 | 2014-08-06 09:37:52 -0700 | [diff] [blame] | 420 | # TODO(jeblair): remove backwards compatibility: |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 421 | pipeline.source = self._getSourceDriver( |
| 422 | conf_pipeline.get('source', 'gerrit')) |
James E. Blair | 64ed6f2 | 2013-07-10 14:07:23 -0700 | [diff] [blame] | 423 | precedence = model.PRECEDENCE_MAP[conf_pipeline.get('precedence')] |
| 424 | pipeline.precedence = precedence |
James E. Blair | 5637019 | 2013-01-14 15:47:28 -0800 | [diff] [blame] | 425 | pipeline.failure_message = conf_pipeline.get('failure-message', |
| 426 | "Build failed.") |
Joshua Hesketh | b717977 | 2014-01-30 23:30:46 +1100 | [diff] [blame] | 427 | pipeline.merge_failure_message = conf_pipeline.get( |
Jeremy Stanley | 1c2c3c2 | 2015-06-15 21:23:19 +0000 | [diff] [blame] | 428 | 'merge-failure-message', "Merge Failed.\n\nThis change or one " |
| 429 | "of its cross-repo dependencies was unable to be " |
| 430 | "automatically merged with the current state of its " |
| 431 | "repository. Please rebase the change and upload a new " |
Joshua Hesketh | b717977 | 2014-01-30 23:30:46 +1100 | [diff] [blame] | 432 | "patchset.") |
James E. Blair | 5637019 | 2013-01-14 15:47:28 -0800 | [diff] [blame] | 433 | pipeline.success_message = conf_pipeline.get('success-message', |
| 434 | "Build succeeded.") |
Joshua Hesketh | 3979e3e | 2014-03-04 11:21:10 +1100 | [diff] [blame] | 435 | pipeline.footer_message = conf_pipeline.get('footer-message', "") |
James E. Blair | 2fa5096 | 2013-01-30 21:50:41 -0800 | [diff] [blame] | 436 | pipeline.dequeue_on_new_patchset = conf_pipeline.get( |
James E. Blair | 6736beb | 2013-07-11 15:18:15 -0700 | [diff] [blame] | 437 | 'dequeue-on-new-patchset', True) |
James E. Blair | 17dd677 | 2015-02-09 14:45:18 -0800 | [diff] [blame] | 438 | pipeline.ignore_dependencies = conf_pipeline.get( |
| 439 | 'ignore-dependencies', False) |
Joshua Hesketh | 1879cf7 | 2013-08-19 14:13:15 +1000 | [diff] [blame] | 440 | |
Joshua Hesketh | de95865 | 2015-11-10 19:19:50 +1100 | [diff] [blame] | 441 | for conf_key, action in self._reporter_actions.items(): |
| 442 | reporter_set = [] |
| 443 | if conf_pipeline.get(conf_key): |
Joshua Hesketh | 1879cf7 | 2013-08-19 14:13:15 +1000 | [diff] [blame] | 444 | for reporter_name, params \ |
Joshua Hesketh | de95865 | 2015-11-10 19:19:50 +1100 | [diff] [blame] | 445 | in conf_pipeline.get(conf_key).items(): |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 446 | reporter = self._getReporterDriver(reporter_name, |
| 447 | params) |
Joshua Hesketh | 385d11e | 2015-09-14 14:50:01 -0600 | [diff] [blame] | 448 | reporter.setAction(conf_key) |
Joshua Hesketh | de95865 | 2015-11-10 19:19:50 +1100 | [diff] [blame] | 449 | reporter_set.append(reporter) |
| 450 | setattr(pipeline, action, reporter_set) |
| 451 | |
| 452 | # If merge-failure actions aren't explicit, use the failure actions |
| 453 | if not pipeline.merge_failure_actions: |
| 454 | pipeline.merge_failure_actions = pipeline.failure_actions |
Joshua Hesketh | 1879cf7 | 2013-08-19 14:13:15 +1000 | [diff] [blame] | 455 | |
Joshua Hesketh | 89e829d | 2015-02-10 16:29:45 +1100 | [diff] [blame] | 456 | pipeline.disable_at = conf_pipeline.get( |
| 457 | 'disable-after-consecutive-failures', None) |
| 458 | |
Clark Boylan | 7603a37 | 2014-01-21 11:43:20 -0800 | [diff] [blame] | 459 | pipeline.window = conf_pipeline.get('window', 20) |
| 460 | pipeline.window_floor = conf_pipeline.get('window-floor', 3) |
| 461 | pipeline.window_increase_type = conf_pipeline.get( |
| 462 | 'window-increase-type', 'linear') |
| 463 | pipeline.window_increase_factor = conf_pipeline.get( |
| 464 | 'window-increase-factor', 1) |
| 465 | pipeline.window_decrease_type = conf_pipeline.get( |
| 466 | 'window-decrease-type', 'exponential') |
| 467 | pipeline.window_decrease_factor = conf_pipeline.get( |
| 468 | 'window-decrease-factor', 2) |
| 469 | |
James E. Blair | 4aea70c | 2012-07-26 14:23:24 -0700 | [diff] [blame] | 470 | manager = globals()[conf_pipeline['manager']](self, pipeline) |
| 471 | pipeline.setManager(manager) |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 472 | layout.pipelines[conf_pipeline['name']] = pipeline |
Joshua Hesketh | 1879cf7 | 2013-08-19 14:13:15 +1000 | [diff] [blame] | 473 | |
Joshua Hesketh | 66c8e52 | 2014-06-26 15:30:08 +1000 | [diff] [blame] | 474 | if 'require' in conf_pipeline or 'reject' in conf_pipeline: |
| 475 | require = conf_pipeline.get('require', {}) |
| 476 | reject = conf_pipeline.get('reject', {}) |
Clark Boylan | a9702ad | 2014-05-08 17:17:24 -0700 | [diff] [blame] | 477 | f = ChangeishFilter( |
| 478 | open=require.get('open'), |
| 479 | current_patchset=require.get('current-patchset'), |
| 480 | statuses=toList(require.get('status')), |
Joshua Hesketh | 66c8e52 | 2014-06-26 15:30:08 +1000 | [diff] [blame] | 481 | required_approvals=toList(require.get('approval')), |
| 482 | reject_approvals=toList(reject.get('approval')) |
| 483 | ) |
James E. Blair | 11041d2 | 2014-05-02 14:49:53 -0700 | [diff] [blame] | 484 | manager.changeish_filters.append(f) |
| 485 | |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 486 | for trigger_name, trigger_config\ |
| 487 | in conf_pipeline.get('trigger').items(): |
| 488 | if trigger_name not in self.triggers.keys(): |
| 489 | self.triggers[trigger_name] = \ |
| 490 | self._getTriggerDriver(trigger_name, trigger_config) |
| 491 | |
| 492 | for trigger_name, trigger in self.triggers.items(): |
| 493 | if trigger_name in conf_pipeline['trigger']: |
| 494 | manager.event_filters += trigger.getEventFilters( |
| 495 | conf_pipeline['trigger'][trigger_name]) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 496 | |
Antoine Musso | 80edd5a | 2013-02-13 15:37:53 +0100 | [diff] [blame] | 497 | for project_template in data.get('project-templates', []): |
| 498 | # Make sure the template only contains valid pipelines |
| 499 | tpl = dict( |
| 500 | (pipe_name, project_template.get(pipe_name)) |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 501 | for pipe_name in layout.pipelines.keys() |
Antoine Musso | 80edd5a | 2013-02-13 15:37:53 +0100 | [diff] [blame] | 502 | if pipe_name in project_template |
| 503 | ) |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 504 | project_templates[project_template.get('name')] = tpl |
Antoine Musso | 80edd5a | 2013-02-13 15:37:53 +0100 | [diff] [blame] | 505 | |
James E. Blair | 4795838 | 2013-01-10 17:26:02 -0800 | [diff] [blame] | 506 | for config_job in data.get('jobs', []): |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 507 | job = layout.getJob(config_job['name']) |
James E. Blair | b095465 | 2012-06-01 11:32:01 -0700 | [diff] [blame] | 508 | # Be careful to only set attributes explicitly present on |
| 509 | # this job, to avoid squashing attributes set by a meta-job. |
James E. Blair | c8a1e05 | 2014-02-25 09:29:26 -0800 | [diff] [blame] | 510 | m = config_job.get('queue-name', None) |
| 511 | if m: |
| 512 | job.queue_name = m |
James E. Blair | b095465 | 2012-06-01 11:32:01 -0700 | [diff] [blame] | 513 | m = config_job.get('failure-message', None) |
| 514 | if m: |
| 515 | job.failure_message = m |
| 516 | m = config_job.get('success-message', None) |
| 517 | if m: |
| 518 | job.success_message = m |
James E. Blair | 6aea36d | 2012-12-17 13:03:24 -0800 | [diff] [blame] | 519 | m = config_job.get('failure-pattern', None) |
| 520 | if m: |
| 521 | job.failure_pattern = m |
| 522 | m = config_job.get('success-pattern', None) |
| 523 | if m: |
| 524 | job.success_pattern = m |
James E. Blair | 222d498 | 2012-07-16 09:31:19 -0700 | [diff] [blame] | 525 | m = config_job.get('hold-following-changes', False) |
| 526 | if m: |
| 527 | job.hold_following_changes = True |
James E. Blair | 4ec821f | 2012-08-23 15:28:28 -0700 | [diff] [blame] | 528 | m = config_job.get('voting', None) |
| 529 | if m is not None: |
| 530 | job.voting = m |
James E. Blair | af17a97 | 2016-02-03 15:07:18 -0800 | [diff] [blame] | 531 | m = config_job.get('mutex', None) |
| 532 | if m is not None: |
| 533 | job.mutex = m |
James E. Blair | 456f2fb | 2016-02-09 09:29:33 -0800 | [diff] [blame] | 534 | tags = toList(config_job.get('tags')) |
| 535 | if tags: |
| 536 | # Tags are merged via a union rather than a |
| 537 | # destructive copy because they are intended to |
| 538 | # accumulate onto any previously applied tags from |
| 539 | # metajobs. |
| 540 | job.tags = job.tags.union(set(tags)) |
James E. Blair | e5a847f | 2012-07-10 15:29:14 -0700 | [diff] [blame] | 541 | fname = config_job.get('parameter-function', None) |
| 542 | if fname: |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 543 | func = config_env.get(fname, None) |
James E. Blair | e5a847f | 2012-07-10 15:29:14 -0700 | [diff] [blame] | 544 | if not func: |
| 545 | raise Exception("Unable to find function %s" % fname) |
| 546 | job.parameter_function = func |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 547 | branches = toList(config_job.get('branch')) |
| 548 | if branches: |
James E. Blair | e421a23 | 2012-07-25 16:59:21 -0700 | [diff] [blame] | 549 | job._branches = branches |
| 550 | job.branches = [re.compile(x) for x in branches] |
James E. Blair | 70c7158 | 2013-03-06 08:50:50 -0800 | [diff] [blame] | 551 | files = toList(config_job.get('files')) |
| 552 | if files: |
| 553 | job._files = files |
| 554 | job.files = [re.compile(x) for x in files] |
Maru Newby | 3fe5f85 | 2015-01-13 04:22:14 +0000 | [diff] [blame] | 555 | skip_if_matcher = self._parseSkipIf(config_job) |
| 556 | if skip_if_matcher: |
| 557 | job.skip_if_matcher = skip_if_matcher |
Joshua Hesketh | 36c3fa5 | 2014-01-22 11:40:52 +1100 | [diff] [blame] | 558 | swift = toList(config_job.get('swift')) |
| 559 | if swift: |
| 560 | for s in swift: |
| 561 | job.swift[s['name']] = s |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 562 | |
| 563 | def add_jobs(job_tree, config_jobs): |
| 564 | for job in config_jobs: |
| 565 | if isinstance(job, list): |
| 566 | for x in job: |
| 567 | add_jobs(job_tree, x) |
| 568 | if isinstance(job, dict): |
| 569 | for parent, children in job.items(): |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 570 | parent_tree = job_tree.addJob(layout.getJob(parent)) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 571 | add_jobs(parent_tree, children) |
| 572 | if isinstance(job, str): |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 573 | job_tree.addJob(layout.getJob(job)) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 574 | |
James E. Blair | 4795838 | 2013-01-10 17:26:02 -0800 | [diff] [blame] | 575 | for config_project in data.get('projects', []): |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 576 | project = Project(config_project['name']) |
James E. Blair | aea6cf6 | 2013-12-16 15:38:12 -0800 | [diff] [blame] | 577 | shortname = config_project['name'].split('/')[-1] |
Antoine Musso | 80edd5a | 2013-02-13 15:37:53 +0100 | [diff] [blame] | 578 | |
James E. Blair | 3e98c02 | 2013-12-16 15:25:38 -0800 | [diff] [blame] | 579 | # This is reversed due to the prepend operation below, so |
| 580 | # the ultimate order is templates (in order) followed by |
| 581 | # statically defined jobs. |
| 582 | for requested_template in reversed( |
| 583 | config_project.get('template', [])): |
Antoine Musso | 80edd5a | 2013-02-13 15:37:53 +0100 | [diff] [blame] | 584 | # Fetch the template from 'project-templates' |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 585 | tpl = project_templates.get( |
Antoine Musso | 80edd5a | 2013-02-13 15:37:53 +0100 | [diff] [blame] | 586 | requested_template.get('name')) |
| 587 | # Expand it with the project context |
James E. Blair | aea6cf6 | 2013-12-16 15:38:12 -0800 | [diff] [blame] | 588 | requested_template['name'] = shortname |
Antoine Musso | 80edd5a | 2013-02-13 15:37:53 +0100 | [diff] [blame] | 589 | expanded = deep_format(tpl, requested_template) |
James E. Blair | 3e98c02 | 2013-12-16 15:25:38 -0800 | [diff] [blame] | 590 | # Finally merge the expansion with whatever has been |
| 591 | # already defined for this project. Prepend our new |
| 592 | # jobs to existing ones (which may have been |
| 593 | # statically defined or defined by other templates). |
| 594 | for pipeline in layout.pipelines.values(): |
| 595 | if pipeline.name in expanded: |
| 596 | config_project.update( |
| 597 | {pipeline.name: expanded[pipeline.name] + |
| 598 | config_project.get(pipeline.name, [])}) |
Antoine Musso | 80edd5a | 2013-02-13 15:37:53 +0100 | [diff] [blame] | 599 | |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 600 | layout.projects[config_project['name']] = project |
James E. Blair | 19deff2 | 2013-08-25 13:17:35 -0700 | [diff] [blame] | 601 | mode = config_project.get('merge-mode', 'merge-resolve') |
| 602 | project.merge_mode = model.MERGER_MAP[mode] |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 603 | for pipeline in layout.pipelines.values(): |
James E. Blair | 4aea70c | 2012-07-26 14:23:24 -0700 | [diff] [blame] | 604 | if pipeline.name in config_project: |
| 605 | job_tree = pipeline.addProject(project) |
| 606 | config_jobs = config_project[pipeline.name] |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 607 | add_jobs(job_tree, config_jobs) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 608 | |
James E. Blair | b095465 | 2012-06-01 11:32:01 -0700 | [diff] [blame] | 609 | # All jobs should be defined at this point, get rid of |
| 610 | # metajobs so that getJob isn't doing anything weird. |
James E. Blair | c28d1b0 | 2013-07-19 11:37:06 -0700 | [diff] [blame] | 611 | layout.metajobs = [] |
James E. Blair | b095465 | 2012-06-01 11:32:01 -0700 | [diff] [blame] | 612 | |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 613 | for pipeline in layout.pipelines.values(): |
| 614 | pipeline.manager._postConfig(layout) |
| 615 | |
| 616 | return layout |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 617 | |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 618 | def setLauncher(self, launcher): |
| 619 | self.launcher = launcher |
| 620 | |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 621 | def setMerger(self, merger): |
| 622 | self.merger = merger |
| 623 | |
Evgeny Antyshev | 0deaaad | 2015-08-03 20:22:56 +0000 | [diff] [blame] | 624 | def getProject(self, name, create_foreign=False): |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 625 | self.layout_lock.acquire() |
| 626 | p = None |
| 627 | try: |
| 628 | p = self.layout.projects.get(name) |
Evgeny Antyshev | 0deaaad | 2015-08-03 20:22:56 +0000 | [diff] [blame] | 629 | if p is None and create_foreign: |
| 630 | self.log.info("Registering foreign project: %s" % name) |
| 631 | p = Project(name, foreign=True) |
| 632 | self.layout.projects[name] = p |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 633 | finally: |
| 634 | self.layout_lock.release() |
| 635 | return p |
| 636 | |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 637 | def addEvent(self, event): |
| 638 | self.log.debug("Adding trigger event: %s" % event) |
James E. Blair | 23ec1ba | 2013-01-04 18:06:10 -0800 | [diff] [blame] | 639 | try: |
| 640 | if statsd: |
| 641 | statsd.incr('gerrit.event.%s' % event.type) |
| 642 | except: |
| 643 | self.log.exception("Exception reporting event stats") |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 644 | self.trigger_event_queue.put(event) |
| 645 | self.wake_event.set() |
James E. Blair | f62d428 | 2012-12-31 17:01:50 -0800 | [diff] [blame] | 646 | self.log.debug("Done adding trigger event: %s" % event) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 647 | |
James E. Blair | 11700c3 | 2012-07-05 17:50:05 -0700 | [diff] [blame] | 648 | def onBuildStarted(self, build): |
| 649 | self.log.debug("Adding start event for build: %s" % build) |
James E. Blair | 71e9412 | 2012-12-24 17:53:08 -0800 | [diff] [blame] | 650 | build.start_time = time.time() |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 651 | event = BuildStartedEvent(build) |
| 652 | self.result_event_queue.put(event) |
James E. Blair | 11700c3 | 2012-07-05 17:50:05 -0700 | [diff] [blame] | 653 | self.wake_event.set() |
James E. Blair | f62d428 | 2012-12-31 17:01:50 -0800 | [diff] [blame] | 654 | self.log.debug("Done adding start event for build: %s" % build) |
James E. Blair | 11700c3 | 2012-07-05 17:50:05 -0700 | [diff] [blame] | 655 | |
James E. Blair | f035866 | 2015-07-20 15:19:12 -0700 | [diff] [blame] | 656 | def onBuildCompleted(self, build, result): |
| 657 | self.log.debug("Adding complete event for build: %s result: %s" % ( |
| 658 | build, result)) |
James E. Blair | 71e9412 | 2012-12-24 17:53:08 -0800 | [diff] [blame] | 659 | build.end_time = time.time() |
James E. Blair | f035866 | 2015-07-20 15:19:12 -0700 | [diff] [blame] | 660 | # Note, as soon as the result is set, other threads may act |
| 661 | # upon this, even though the event hasn't been fully |
| 662 | # processed. Ensure that any other data from the event (eg, |
| 663 | # timing) is recorded before setting the result. |
| 664 | build.result = result |
James E. Blair | 23ec1ba | 2013-01-04 18:06:10 -0800 | [diff] [blame] | 665 | try: |
James E. Blair | 66eeebf | 2013-07-27 17:44:32 -0700 | [diff] [blame] | 666 | if statsd and build.pipeline: |
| 667 | jobname = build.job.name.replace('.', '_') |
Timothy Chavez | b233208 | 2015-08-07 20:08:04 -0500 | [diff] [blame] | 668 | key = 'zuul.pipeline.%s.all_jobs' % build.pipeline.name |
Bruno Tavares | f564b28 | 2015-10-15 15:20:51 -0300 | [diff] [blame] | 669 | statsd.incr(key) |
Timothy Chavez | b233208 | 2015-08-07 20:08:04 -0500 | [diff] [blame] | 670 | for label in build.node_labels: |
| 671 | # Jenkins includes the node name in its list of labels, so |
| 672 | # we filter it out here, since that is not statistically |
| 673 | # interesting. |
| 674 | if label == build.node_name: |
| 675 | continue |
| 676 | dt = int((build.start_time - build.launch_time) * 1000) |
James E. Blair | 50aacbc | 2015-11-17 14:09:59 -0800 | [diff] [blame] | 677 | key = 'zuul.pipeline.%s.label.%s.wait_time' % ( |
| 678 | build.pipeline.name, label) |
Timothy Chavez | b233208 | 2015-08-07 20:08:04 -0500 | [diff] [blame] | 679 | statsd.timing(key, dt) |
James E. Blair | 66eeebf | 2013-07-27 17:44:32 -0700 | [diff] [blame] | 680 | key = 'zuul.pipeline.%s.job.%s.%s' % (build.pipeline.name, |
| 681 | jobname, build.result) |
James E. Blair | 23ec1ba | 2013-01-04 18:06:10 -0800 | [diff] [blame] | 682 | if build.result in ['SUCCESS', 'FAILURE'] and build.start_time: |
| 683 | dt = int((build.end_time - build.start_time) * 1000) |
| 684 | statsd.timing(key, dt) |
| 685 | statsd.incr(key) |
James E. Blair | 50aacbc | 2015-11-17 14:09:59 -0800 | [diff] [blame] | 686 | |
| 687 | key = 'zuul.pipeline.%s.job.%s.wait_time' % ( |
| 688 | build.pipeline.name, jobname) |
| 689 | dt = int((build.start_time - build.launch_time) * 1000) |
| 690 | statsd.timing(key, dt) |
James E. Blair | 23ec1ba | 2013-01-04 18:06:10 -0800 | [diff] [blame] | 691 | except: |
| 692 | self.log.exception("Exception reporting runtime stats") |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 693 | event = BuildCompletedEvent(build) |
| 694 | self.result_event_queue.put(event) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 695 | self.wake_event.set() |
James E. Blair | f62d428 | 2012-12-31 17:01:50 -0800 | [diff] [blame] | 696 | self.log.debug("Done adding complete event for build: %s" % build) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 697 | |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 698 | def onMergeCompleted(self, build_set, zuul_url, merged, updated, commit): |
| 699 | self.log.debug("Adding merge complete event for build set: %s" % |
| 700 | build_set) |
| 701 | event = MergeCompletedEvent(build_set, zuul_url, |
| 702 | merged, updated, commit) |
| 703 | self.result_event_queue.put(event) |
| 704 | self.wake_event.set() |
| 705 | |
James E. Blair | e9d45c3 | 2012-05-31 09:56:45 -0700 | [diff] [blame] | 706 | def reconfigure(self, config): |
James E. Blair | 5d5bc2b | 2012-07-06 10:24:01 -0700 | [diff] [blame] | 707 | self.log.debug("Prepare to reconfigure") |
James E. Blair | 468c851 | 2013-12-06 13:27:19 -0800 | [diff] [blame] | 708 | event = ReconfigureEvent(config) |
| 709 | self.management_event_queue.put(event) |
James E. Blair | e9d45c3 | 2012-05-31 09:56:45 -0700 | [diff] [blame] | 710 | self.wake_event.set() |
| 711 | self.log.debug("Waiting for reconfiguration") |
James E. Blair | 468c851 | 2013-12-06 13:27:19 -0800 | [diff] [blame] | 712 | event.wait() |
James E. Blair | e9d45c3 | 2012-05-31 09:56:45 -0700 | [diff] [blame] | 713 | self.log.debug("Reconfiguration complete") |
Sergey Lukjanov | 5d0438d | 2013-12-24 03:36:39 +0400 | [diff] [blame] | 714 | self.last_reconfigured = int(time.time()) |
James E. Blair | e9d45c3 | 2012-05-31 09:56:45 -0700 | [diff] [blame] | 715 | |
James E. Blair | 36658cf | 2013-12-06 17:53:48 -0800 | [diff] [blame] | 716 | def promote(self, pipeline_name, change_ids): |
| 717 | event = PromoteEvent(pipeline_name, change_ids) |
| 718 | self.management_event_queue.put(event) |
| 719 | self.wake_event.set() |
| 720 | self.log.debug("Waiting for promotion") |
| 721 | event.wait() |
| 722 | self.log.debug("Promotion complete") |
| 723 | |
James E. Blair | d27a96d | 2014-07-10 13:25:13 -0700 | [diff] [blame] | 724 | def enqueue(self, trigger_event): |
| 725 | event = EnqueueEvent(trigger_event) |
| 726 | self.management_event_queue.put(event) |
| 727 | self.wake_event.set() |
| 728 | self.log.debug("Waiting for enqueue") |
| 729 | event.wait() |
| 730 | self.log.debug("Enqueue complete") |
| 731 | |
James E. Blair | 5d5bc2b | 2012-07-06 10:24:01 -0700 | [diff] [blame] | 732 | def exit(self): |
| 733 | self.log.debug("Prepare to exit") |
| 734 | self._pause = True |
| 735 | self._exit = True |
| 736 | self.wake_event.set() |
| 737 | self.log.debug("Waiting for exit") |
| 738 | |
| 739 | def _get_queue_pickle_file(self): |
James E. Blair | 5a95c86 | 2012-07-09 15:11:17 -0700 | [diff] [blame] | 740 | if self.config.has_option('zuul', 'state_dir'): |
| 741 | state_dir = os.path.expanduser(self.config.get('zuul', |
| 742 | 'state_dir')) |
| 743 | else: |
| 744 | state_dir = '/var/lib/zuul' |
James E. Blair | 5d5bc2b | 2012-07-06 10:24:01 -0700 | [diff] [blame] | 745 | return os.path.join(state_dir, 'queue.pickle') |
| 746 | |
James E. Blair | ce8a213 | 2016-05-19 15:21:52 -0700 | [diff] [blame] | 747 | def _get_time_database_dir(self): |
| 748 | if self.config.has_option('zuul', 'state_dir'): |
| 749 | state_dir = os.path.expanduser(self.config.get('zuul', |
| 750 | 'state_dir')) |
| 751 | else: |
| 752 | state_dir = '/var/lib/zuul' |
| 753 | d = os.path.join(state_dir, 'times') |
| 754 | if not os.path.exists(d): |
| 755 | os.mkdir(d) |
| 756 | return d |
| 757 | |
James E. Blair | 5d5bc2b | 2012-07-06 10:24:01 -0700 | [diff] [blame] | 758 | def _save_queue(self): |
| 759 | pickle_file = self._get_queue_pickle_file() |
| 760 | events = [] |
| 761 | while not self.trigger_event_queue.empty(): |
| 762 | events.append(self.trigger_event_queue.get()) |
| 763 | self.log.debug("Queue length is %s" % len(events)) |
| 764 | if events: |
| 765 | self.log.debug("Saving queue") |
| 766 | pickle.dump(events, open(pickle_file, 'wb')) |
| 767 | |
| 768 | def _load_queue(self): |
| 769 | pickle_file = self._get_queue_pickle_file() |
| 770 | if os.path.exists(pickle_file): |
| 771 | self.log.debug("Loading queue") |
| 772 | events = pickle.load(open(pickle_file, 'rb')) |
| 773 | self.log.debug("Queue length is %s" % len(events)) |
| 774 | for event in events: |
| 775 | self.trigger_event_queue.put(event) |
| 776 | else: |
| 777 | self.log.debug("No queue file found") |
| 778 | |
| 779 | def _delete_queue(self): |
| 780 | pickle_file = self._get_queue_pickle_file() |
| 781 | if os.path.exists(pickle_file): |
| 782 | self.log.debug("Deleting saved queue") |
| 783 | os.unlink(pickle_file) |
| 784 | |
| 785 | def resume(self): |
| 786 | try: |
| 787 | self._load_queue() |
| 788 | except: |
| 789 | self.log.exception("Unable to load queue") |
| 790 | try: |
| 791 | self._delete_queue() |
| 792 | except: |
| 793 | self.log.exception("Unable to delete saved queue") |
| 794 | self.log.debug("Resuming queue processing") |
| 795 | self.wake_event.set() |
| 796 | |
| 797 | def _doPauseEvent(self): |
| 798 | if self._exit: |
| 799 | self.log.debug("Exiting") |
| 800 | self._save_queue() |
| 801 | os._exit(0) |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 802 | |
James E. Blair | 468c851 | 2013-12-06 13:27:19 -0800 | [diff] [blame] | 803 | def _doReconfigureEvent(self, event): |
| 804 | # This is called in the scheduler loop after another thread submits |
| 805 | # a request |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 806 | self.layout_lock.acquire() |
James E. Blair | 468c851 | 2013-12-06 13:27:19 -0800 | [diff] [blame] | 807 | self.config = event.config |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 808 | try: |
James E. Blair | 5d5bc2b | 2012-07-06 10:24:01 -0700 | [diff] [blame] | 809 | self.log.debug("Performing reconfiguration") |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 810 | self._unloadDrivers() |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 811 | layout = self._parseConfig( |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 812 | self.config.get('zuul', 'layout_config'), self.connections) |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 813 | for name, new_pipeline in layout.pipelines.items(): |
| 814 | old_pipeline = self.layout.pipelines.get(name) |
| 815 | if not old_pipeline: |
| 816 | if self.layout.pipelines: |
| 817 | # Don't emit this warning on startup |
| 818 | self.log.warning("No old pipeline matching %s found " |
| 819 | "when reconfiguring" % name) |
| 820 | continue |
James E. Blair | dad5225 | 2014-02-07 16:59:17 -0800 | [diff] [blame] | 821 | self.log.debug("Re-enqueueing changes for pipeline %s" % name) |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 822 | items_to_remove = [] |
James E. Blair | 400e8fd | 2015-07-30 17:44:45 -0700 | [diff] [blame] | 823 | builds_to_cancel = [] |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 824 | last_head = None |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 825 | for shared_queue in old_pipeline.queues: |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 826 | for item in shared_queue.queue: |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 827 | if not item.item_ahead: |
| 828 | last_head = item |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 829 | item.item_ahead = None |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 830 | item.items_behind = [] |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 831 | item.pipeline = None |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 832 | item.queue = None |
Evgeny Antyshev | 0deaaad | 2015-08-03 20:22:56 +0000 | [diff] [blame] | 833 | project_name = item.change.project.name |
| 834 | item.change.project = layout.projects.get(project_name) |
| 835 | if not item.change.project: |
| 836 | self.log.debug("Project %s not defined, " |
| 837 | "re-instantiating as foreign" % |
| 838 | project_name) |
| 839 | project = Project(project_name, foreign=True) |
| 840 | layout.projects[project_name] = project |
| 841 | item.change.project = project |
James E. Blair | fe707d1 | 2015-08-05 15:18:15 -0700 | [diff] [blame] | 842 | item_jobs = new_pipeline.getJobs(item) |
James E. Blair | dad5225 | 2014-02-07 16:59:17 -0800 | [diff] [blame] | 843 | for build in item.current_build_set.getBuilds(): |
James E. Blair | 6b07794 | 2014-02-07 17:45:55 -0800 | [diff] [blame] | 844 | job = layout.jobs.get(build.job.name) |
James E. Blair | fe707d1 | 2015-08-05 15:18:15 -0700 | [diff] [blame] | 845 | if job and job in item_jobs: |
James E. Blair | 6b07794 | 2014-02-07 17:45:55 -0800 | [diff] [blame] | 846 | build.job = job |
| 847 | else: |
James E. Blair | 400e8fd | 2015-07-30 17:44:45 -0700 | [diff] [blame] | 848 | item.removeBuild(build) |
| 849 | builds_to_cancel.append(build) |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 850 | if not new_pipeline.manager.reEnqueueItem(item, |
| 851 | last_head): |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 852 | items_to_remove.append(item) |
James E. Blair | 6b07794 | 2014-02-07 17:45:55 -0800 | [diff] [blame] | 853 | for item in items_to_remove: |
| 854 | for build in item.current_build_set.getBuilds(): |
James E. Blair | 400e8fd | 2015-07-30 17:44:45 -0700 | [diff] [blame] | 855 | builds_to_cancel.append(build) |
| 856 | for build in builds_to_cancel: |
James E. Blair | 6b07794 | 2014-02-07 17:45:55 -0800 | [diff] [blame] | 857 | self.log.warning( |
| 858 | "Canceling build %s during reconfiguration" % (build,)) |
James E. Blair | dad5225 | 2014-02-07 16:59:17 -0800 | [diff] [blame] | 859 | try: |
| 860 | self.launcher.cancel(build) |
| 861 | except Exception: |
| 862 | self.log.exception( |
| 863 | "Exception while canceling build %s " |
| 864 | "for change %s" % (build, item.change)) |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 865 | self.layout = layout |
Joshua Hesketh | 4bd7da3 | 2016-02-17 20:58:47 +1100 | [diff] [blame] | 866 | self.maintainConnectionCache() |
James E. Blair | 63bb0ef | 2013-07-29 17:14:51 -0700 | [diff] [blame] | 867 | for trigger in self.triggers.values(): |
| 868 | trigger.postConfig() |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 869 | for pipeline in self.layout.pipelines.values(): |
| 870 | pipeline.source.postConfig() |
Joshua Hesketh | de95865 | 2015-11-10 19:19:50 +1100 | [diff] [blame] | 871 | for action in self._reporter_actions.values(): |
| 872 | for reporter in pipeline.__getattribute__(action): |
| 873 | reporter.postConfig() |
James E. Blair | 3cb1070 | 2013-08-24 08:56:03 -0700 | [diff] [blame] | 874 | if statsd: |
| 875 | try: |
| 876 | for pipeline in self.layout.pipelines.values(): |
| 877 | items = len(pipeline.getAllItems()) |
| 878 | # stats.gauges.zuul.pipeline.NAME.current_changes |
| 879 | key = 'zuul.pipeline.%s' % pipeline.name |
| 880 | statsd.gauge(key + '.current_changes', items) |
| 881 | except Exception: |
| 882 | self.log.exception("Exception reporting initial " |
| 883 | "pipeline stats:") |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 884 | finally: |
| 885 | self.layout_lock.release() |
James E. Blair | e9d45c3 | 2012-05-31 09:56:45 -0700 | [diff] [blame] | 886 | |
James E. Blair | 36658cf | 2013-12-06 17:53:48 -0800 | [diff] [blame] | 887 | def _doPromoteEvent(self, event): |
| 888 | pipeline = self.layout.pipelines[event.pipeline_name] |
| 889 | change_ids = [c.split(',') for c in event.change_ids] |
| 890 | items_to_enqueue = [] |
| 891 | change_queue = None |
| 892 | for shared_queue in pipeline.queues: |
| 893 | if change_queue: |
| 894 | break |
| 895 | for item in shared_queue.queue: |
| 896 | if (item.change.number == change_ids[0][0] and |
Joshua Hesketh | 29d99b7 | 2014-08-19 16:27:42 +1000 | [diff] [blame] | 897 | item.change.patchset == change_ids[0][1]): |
James E. Blair | 36658cf | 2013-12-06 17:53:48 -0800 | [diff] [blame] | 898 | change_queue = shared_queue |
| 899 | break |
| 900 | if not change_queue: |
| 901 | raise Exception("Unable to find shared change queue for %s" % |
| 902 | event.change_ids[0]) |
| 903 | for number, patchset in change_ids: |
| 904 | found = False |
| 905 | for item in change_queue.queue: |
| 906 | if (item.change.number == number and |
Joshua Hesketh | 29d99b7 | 2014-08-19 16:27:42 +1000 | [diff] [blame] | 907 | item.change.patchset == patchset): |
James E. Blair | 36658cf | 2013-12-06 17:53:48 -0800 | [diff] [blame] | 908 | found = True |
| 909 | items_to_enqueue.append(item) |
| 910 | break |
| 911 | if not found: |
| 912 | raise Exception("Unable to find %s,%s in queue %s" % |
| 913 | (number, patchset, change_queue)) |
| 914 | for item in change_queue.queue[:]: |
| 915 | if item not in items_to_enqueue: |
| 916 | items_to_enqueue.append(item) |
| 917 | pipeline.manager.cancelJobs(item) |
| 918 | pipeline.manager.dequeueItem(item) |
| 919 | for item in items_to_enqueue: |
Sean Dague | f39b9ca | 2014-01-10 21:34:35 -0500 | [diff] [blame] | 920 | pipeline.manager.addChange( |
| 921 | item.change, |
| 922 | enqueue_time=item.enqueue_time, |
James E. Blair | f9ab884 | 2014-07-10 13:12:07 -0700 | [diff] [blame] | 923 | quiet=True, |
| 924 | ignore_requirements=True) |
James E. Blair | 36658cf | 2013-12-06 17:53:48 -0800 | [diff] [blame] | 925 | |
James E. Blair | d27a96d | 2014-07-10 13:25:13 -0700 | [diff] [blame] | 926 | def _doEnqueueEvent(self, event): |
| 927 | project = self.layout.projects.get(event.project_name) |
| 928 | pipeline = self.layout.pipelines[event.forced_pipeline] |
James E. Blair | c0dedf8 | 2014-08-06 09:37:52 -0700 | [diff] [blame] | 929 | change = pipeline.source.getChange(event, project) |
James E. Blair | d27a96d | 2014-07-10 13:25:13 -0700 | [diff] [blame] | 930 | self.log.debug("Event %s for change %s was directly assigned " |
| 931 | "to pipeline %s" % (event, change, self)) |
| 932 | self.log.info("Adding %s, %s to %s" % |
| 933 | (project, change, pipeline)) |
| 934 | pipeline.manager.addChange(change, ignore_requirements=True) |
| 935 | |
James E. Blair | e9d45c3 | 2012-05-31 09:56:45 -0700 | [diff] [blame] | 936 | def _areAllBuildsComplete(self): |
| 937 | self.log.debug("Checking if all builds are complete") |
| 938 | waiting = False |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 939 | if self.merger.areMergesOutstanding(): |
| 940 | waiting = True |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 941 | for pipeline in self.layout.pipelines.values(): |
James E. Blair | 6b07794 | 2014-02-07 17:45:55 -0800 | [diff] [blame] | 942 | for item in pipeline.getAllItems(): |
| 943 | for build in item.current_build_set.getBuilds(): |
| 944 | if build.result is None: |
| 945 | self.log.debug("%s waiting on %s" % |
| 946 | (pipeline.manager, build)) |
| 947 | waiting = True |
James E. Blair | e9d45c3 | 2012-05-31 09:56:45 -0700 | [diff] [blame] | 948 | if not waiting: |
| 949 | self.log.debug("All builds are complete") |
| 950 | return True |
| 951 | self.log.debug("All builds are not complete") |
| 952 | return False |
| 953 | |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 954 | def run(self): |
James E. Blair | 71e9412 | 2012-12-24 17:53:08 -0800 | [diff] [blame] | 955 | if statsd: |
| 956 | self.log.debug("Statsd enabled") |
| 957 | else: |
| 958 | self.log.debug("Statsd disabled because python statsd " |
| 959 | "package not found") |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 960 | while True: |
| 961 | self.log.debug("Run handler sleeping") |
| 962 | self.wake_event.wait() |
| 963 | self.wake_event.clear() |
James E. Blair | b0fcae4 | 2012-07-17 11:12:10 -0700 | [diff] [blame] | 964 | if self._stopped: |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 965 | self.log.debug("Run handler stopping") |
James E. Blair | b0fcae4 | 2012-07-17 11:12:10 -0700 | [diff] [blame] | 966 | return |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 967 | self.log.debug("Run handler awake") |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 968 | self.run_handler_lock.acquire() |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 969 | try: |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 970 | while not self.management_event_queue.empty(): |
James E. Blair | 468c851 | 2013-12-06 13:27:19 -0800 | [diff] [blame] | 971 | self.process_management_queue() |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 972 | |
James E. Blair | 263fba9 | 2013-02-27 13:07:19 -0800 | [diff] [blame] | 973 | # Give result events priority -- they let us stop builds, |
| 974 | # whereas trigger evensts cause us to launch builds. |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 975 | while not self.result_event_queue.empty(): |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 976 | self.process_result_queue() |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 977 | |
| 978 | if not self._pause: |
| 979 | while not self.trigger_event_queue.empty(): |
James E. Blair | 263fba9 | 2013-02-27 13:07:19 -0800 | [diff] [blame] | 980 | self.process_event_queue() |
James E. Blair | e9d45c3 | 2012-05-31 09:56:45 -0700 | [diff] [blame] | 981 | |
James E. Blair | 5d5bc2b | 2012-07-06 10:24:01 -0700 | [diff] [blame] | 982 | if self._pause and self._areAllBuildsComplete(): |
| 983 | self._doPauseEvent() |
James E. Blair | e9d45c3 | 2012-05-31 09:56:45 -0700 | [diff] [blame] | 984 | |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 985 | for pipeline in self.layout.pipelines.values(): |
| 986 | while pipeline.manager.processQueue(): |
| 987 | pass |
James E. Blair | 0e933c5 | 2013-07-11 10:18:52 -0700 | [diff] [blame] | 988 | |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 989 | except Exception: |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 990 | self.log.exception("Exception in run handler:") |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 991 | # There may still be more events to process |
| 992 | self.wake_event.set() |
| 993 | finally: |
| 994 | self.run_handler_lock.release() |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 995 | |
Joshua Hesketh | 4bd7da3 | 2016-02-17 20:58:47 +1100 | [diff] [blame] | 996 | def maintainConnectionCache(self): |
James E. Blair | 0e933c5 | 2013-07-11 10:18:52 -0700 | [diff] [blame] | 997 | relevant = set() |
| 998 | for pipeline in self.layout.pipelines.values(): |
Joshua Hesketh | 4bd7da3 | 2016-02-17 20:58:47 +1100 | [diff] [blame] | 999 | self.log.debug("Gather relevant cache items for: %s" % pipeline) |
James E. Blair | 0e933c5 | 2013-07-11 10:18:52 -0700 | [diff] [blame] | 1000 | for item in pipeline.getAllItems(): |
| 1001 | relevant.add(item.change) |
| 1002 | relevant.update(item.change.getRelatedChanges()) |
Joshua Hesketh | 4bd7da3 | 2016-02-17 20:58:47 +1100 | [diff] [blame] | 1003 | for connection in self.connections.values(): |
| 1004 | connection.maintainCache(relevant) |
| 1005 | self.log.debug( |
| 1006 | "End maintain connection cache for: %s" % connection) |
| 1007 | self.log.debug("Connection cache size: %s" % len(relevant)) |
James E. Blair | 0e933c5 | 2013-07-11 10:18:52 -0700 | [diff] [blame] | 1008 | |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1009 | def process_event_queue(self): |
| 1010 | self.log.debug("Fetching trigger event") |
| 1011 | event = self.trigger_event_queue.get() |
| 1012 | self.log.debug("Processing trigger event %s" % event) |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 1013 | try: |
| 1014 | project = self.layout.projects.get(event.project_name) |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 1015 | |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 1016 | for pipeline in self.layout.pipelines.values(): |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 1017 | # Get the change even if the project is unknown to us for the |
| 1018 | # use of updating the cache if there is another change |
| 1019 | # depending on this foreign one. |
| 1020 | try: |
| 1021 | change = pipeline.source.getChange(event, project) |
| 1022 | except exceptions.ChangeNotFound as e: |
| 1023 | self.log.debug("Unable to get change %s from source %s. " |
| 1024 | "(most likely looking for a change from " |
| 1025 | "another connection trigger)", |
| 1026 | e.change, pipeline.source) |
| 1027 | continue |
| 1028 | if not project or project.foreign: |
| 1029 | self.log.debug("Project %s not found" % event.project_name) |
| 1030 | continue |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 1031 | if event.type == 'patchset-created': |
| 1032 | pipeline.manager.removeOldVersionsOfChange(change) |
Antoine Musso | bd86a31 | 2014-01-08 14:51:33 +0100 | [diff] [blame] | 1033 | elif event.type == 'change-abandoned': |
| 1034 | pipeline.manager.removeAbandonedChange(change) |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 1035 | if pipeline.manager.eventMatches(event, change): |
| 1036 | self.log.info("Adding %s, %s to %s" % |
| 1037 | (project, change, pipeline)) |
| 1038 | pipeline.manager.addChange(change) |
| 1039 | finally: |
James E. Blair | ff79197 | 2013-01-09 11:45:43 -0800 | [diff] [blame] | 1040 | self.trigger_event_queue.task_done() |
James E. Blair | 1e8dd89 | 2012-05-30 09:15:05 -0700 | [diff] [blame] | 1041 | |
James E. Blair | 468c851 | 2013-12-06 13:27:19 -0800 | [diff] [blame] | 1042 | def process_management_queue(self): |
| 1043 | self.log.debug("Fetching management event") |
| 1044 | event = self.management_event_queue.get() |
| 1045 | self.log.debug("Processing management event %s" % event) |
James E. Blair | 36658cf | 2013-12-06 17:53:48 -0800 | [diff] [blame] | 1046 | try: |
| 1047 | if isinstance(event, ReconfigureEvent): |
| 1048 | self._doReconfigureEvent(event) |
| 1049 | elif isinstance(event, PromoteEvent): |
| 1050 | self._doPromoteEvent(event) |
James E. Blair | d27a96d | 2014-07-10 13:25:13 -0700 | [diff] [blame] | 1051 | elif isinstance(event, EnqueueEvent): |
| 1052 | self._doEnqueueEvent(event.trigger_event) |
James E. Blair | 36658cf | 2013-12-06 17:53:48 -0800 | [diff] [blame] | 1053 | else: |
| 1054 | self.log.error("Unable to handle event %s" % event) |
| 1055 | event.done() |
| 1056 | except Exception as e: |
| 1057 | event.exception(e, sys.exc_info()[2]) |
James E. Blair | 468c851 | 2013-12-06 13:27:19 -0800 | [diff] [blame] | 1058 | self.management_event_queue.task_done() |
| 1059 | |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1060 | def process_result_queue(self): |
| 1061 | self.log.debug("Fetching result event") |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 1062 | event = self.result_event_queue.get() |
| 1063 | self.log.debug("Processing result event %s" % event) |
| 1064 | try: |
| 1065 | if isinstance(event, BuildStartedEvent): |
| 1066 | self._doBuildStartedEvent(event) |
| 1067 | elif isinstance(event, BuildCompletedEvent): |
| 1068 | self._doBuildCompletedEvent(event) |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 1069 | elif isinstance(event, MergeCompletedEvent): |
| 1070 | self._doMergeCompletedEvent(event) |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 1071 | else: |
| 1072 | self.log.error("Unable to handle event %s" % event) |
| 1073 | finally: |
| 1074 | self.result_event_queue.task_done() |
| 1075 | |
| 1076 | def _doBuildStartedEvent(self, event): |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 1077 | build = event.build |
| 1078 | if build.build_set is not build.build_set.item.current_build_set: |
| 1079 | self.log.warning("Build %s is not in the current build set" % |
| 1080 | (build,)) |
| 1081 | return |
| 1082 | pipeline = build.build_set.item.pipeline |
| 1083 | if not pipeline: |
| 1084 | self.log.warning("Build %s is not associated with a pipeline" % |
| 1085 | (build,)) |
| 1086 | return |
James E. Blair | ce8a213 | 2016-05-19 15:21:52 -0700 | [diff] [blame] | 1087 | try: |
| 1088 | build.estimated_time = float(self.time_database.getEstimatedTime( |
| 1089 | build.job.name)) |
| 1090 | except Exception: |
| 1091 | self.log.exception("Exception estimating build time:") |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 1092 | pipeline.manager.onBuildStarted(event.build) |
James E. Blair | a84f0e4 | 2014-02-06 07:09:22 -0800 | [diff] [blame] | 1093 | |
| 1094 | def _doBuildCompletedEvent(self, event): |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 1095 | build = event.build |
| 1096 | if build.build_set is not build.build_set.item.current_build_set: |
| 1097 | self.log.warning("Build %s is not in the current build set" % |
| 1098 | (build,)) |
| 1099 | return |
| 1100 | pipeline = build.build_set.item.pipeline |
| 1101 | if not pipeline: |
| 1102 | self.log.warning("Build %s is not associated with a pipeline" % |
| 1103 | (build,)) |
| 1104 | return |
James E. Blair | ce8a213 | 2016-05-19 15:21:52 -0700 | [diff] [blame] | 1105 | if build.end_time and build.start_time and build.result: |
| 1106 | duration = build.end_time - build.start_time |
| 1107 | try: |
| 1108 | self.time_database.update(build.job.name, duration, build.result) |
| 1109 | except Exception: |
| 1110 | self.log.exception("Exception recording build time:") |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 1111 | pipeline.manager.onBuildCompleted(event.build) |
| 1112 | |
| 1113 | def _doMergeCompletedEvent(self, event): |
| 1114 | build_set = event.build_set |
| 1115 | if build_set is not build_set.item.current_build_set: |
| 1116 | self.log.warning("Build set %s is not current" % (build_set,)) |
| 1117 | return |
| 1118 | pipeline = build_set.item.pipeline |
| 1119 | if not pipeline: |
| 1120 | self.log.warning("Build set %s is not associated with a pipeline" % |
| 1121 | (build_set,)) |
| 1122 | return |
| 1123 | pipeline.manager.onMergeCompleted(event) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1124 | |
James E. Blair | 8dbd56a | 2012-12-22 10:55:10 -0800 | [diff] [blame] | 1125 | def formatStatusJSON(self): |
James E. Blair | b7273ef | 2016-04-19 08:58:51 -0700 | [diff] [blame] | 1126 | if self.config.has_option('zuul', 'url_pattern'): |
| 1127 | url_pattern = self.config.get('zuul', 'url_pattern') |
| 1128 | else: |
| 1129 | url_pattern = None |
| 1130 | |
James E. Blair | 8dbd56a | 2012-12-22 10:55:10 -0800 | [diff] [blame] | 1131 | data = {} |
Sergey Lukjanov | 5ba961b | 2013-12-27 01:21:04 +0400 | [diff] [blame] | 1132 | |
| 1133 | data['zuul_version'] = self.zuul_version |
| 1134 | |
James E. Blair | 8dbd56a | 2012-12-22 10:55:10 -0800 | [diff] [blame] | 1135 | if self._pause: |
| 1136 | ret = '<p><b>Queue only mode:</b> preparing to ' |
James E. Blair | 8dbd56a | 2012-12-22 10:55:10 -0800 | [diff] [blame] | 1137 | if self._exit: |
| 1138 | ret += 'exit' |
| 1139 | ret += ', queue length: %s' % self.trigger_event_queue.qsize() |
| 1140 | ret += '</p>' |
| 1141 | data['message'] = ret |
| 1142 | |
James E. Blair | fb682cc | 2013-02-26 15:23:27 -0800 | [diff] [blame] | 1143 | data['trigger_event_queue'] = {} |
| 1144 | data['trigger_event_queue']['length'] = \ |
| 1145 | self.trigger_event_queue.qsize() |
| 1146 | data['result_event_queue'] = {} |
| 1147 | data['result_event_queue']['length'] = \ |
| 1148 | self.result_event_queue.qsize() |
| 1149 | |
Sergey Lukjanov | 5d0438d | 2013-12-24 03:36:39 +0400 | [diff] [blame] | 1150 | if self.last_reconfigured: |
| 1151 | data['last_reconfigured'] = self.last_reconfigured * 1000 |
| 1152 | |
James E. Blair | 8dbd56a | 2012-12-22 10:55:10 -0800 | [diff] [blame] | 1153 | pipelines = [] |
| 1154 | data['pipelines'] = pipelines |
Alex Gaynor | fda4c35 | 2014-06-04 11:15:26 -0700 | [diff] [blame] | 1155 | for pipeline in self.layout.pipelines.values(): |
James E. Blair | b7273ef | 2016-04-19 08:58:51 -0700 | [diff] [blame] | 1156 | pipelines.append(pipeline.formatStatusJSON(url_pattern)) |
James E. Blair | 8dbd56a | 2012-12-22 10:55:10 -0800 | [diff] [blame] | 1157 | return json.dumps(data) |
| 1158 | |
James E. Blair | 1e8dd89 | 2012-05-30 09:15:05 -0700 | [diff] [blame] | 1159 | |
James E. Blair | 4aea70c | 2012-07-26 14:23:24 -0700 | [diff] [blame] | 1160 | class BasePipelineManager(object): |
| 1161 | log = logging.getLogger("zuul.BasePipelineManager") |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1162 | |
James E. Blair | 4aea70c | 2012-07-26 14:23:24 -0700 | [diff] [blame] | 1163 | def __init__(self, sched, pipeline): |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1164 | self.sched = sched |
James E. Blair | 4aea70c | 2012-07-26 14:23:24 -0700 | [diff] [blame] | 1165 | self.pipeline = pipeline |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1166 | self.event_filters = [] |
James E. Blair | 11041d2 | 2014-05-02 14:49:53 -0700 | [diff] [blame] | 1167 | self.changeish_filters = [] |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1168 | |
| 1169 | def __str__(self): |
James E. Blair | 93cc8d4 | 2012-08-07 10:46:51 -0700 | [diff] [blame] | 1170 | return "<%s %s>" % (self.__class__.__name__, self.pipeline.name) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1171 | |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 1172 | def _postConfig(self, layout): |
James E. Blair | 4aea70c | 2012-07-26 14:23:24 -0700 | [diff] [blame] | 1173 | self.log.info("Configured Pipeline Manager %s" % self.pipeline.name) |
James E. Blair | c0dedf8 | 2014-08-06 09:37:52 -0700 | [diff] [blame] | 1174 | self.log.info(" Source: %s" % self.pipeline.source) |
James E. Blair | 11041d2 | 2014-05-02 14:49:53 -0700 | [diff] [blame] | 1175 | self.log.info(" Requirements:") |
| 1176 | for f in self.changeish_filters: |
| 1177 | self.log.info(" %s" % f) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1178 | self.log.info(" Events:") |
| 1179 | for e in self.event_filters: |
| 1180 | self.log.info(" %s" % e) |
| 1181 | self.log.info(" Projects:") |
James E. Blair | 1e8dd89 | 2012-05-30 09:15:05 -0700 | [diff] [blame] | 1182 | |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1183 | def log_jobs(tree, indent=0): |
James E. Blair | 1e8dd89 | 2012-05-30 09:15:05 -0700 | [diff] [blame] | 1184 | istr = ' ' + ' ' * indent |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1185 | if tree.job: |
| 1186 | efilters = '' |
James E. Blair | e421a23 | 2012-07-25 16:59:21 -0700 | [diff] [blame] | 1187 | for b in tree.job._branches: |
| 1188 | efilters += str(b) |
James E. Blair | 70c7158 | 2013-03-06 08:50:50 -0800 | [diff] [blame] | 1189 | for f in tree.job._files: |
| 1190 | efilters += str(f) |
Maru Newby | 3fe5f85 | 2015-01-13 04:22:14 +0000 | [diff] [blame] | 1191 | if tree.job.skip_if_matcher: |
| 1192 | efilters += str(tree.job.skip_if_matcher) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1193 | if efilters: |
James E. Blair | 1e8dd89 | 2012-05-30 09:15:05 -0700 | [diff] [blame] | 1194 | efilters = ' ' + efilters |
James E. Blair | af17a97 | 2016-02-03 15:07:18 -0800 | [diff] [blame] | 1195 | tags = [] |
James E. Blair | 222d498 | 2012-07-16 09:31:19 -0700 | [diff] [blame] | 1196 | if tree.job.hold_following_changes: |
James E. Blair | af17a97 | 2016-02-03 15:07:18 -0800 | [diff] [blame] | 1197 | tags.append('[hold]') |
James E. Blair | 4ec821f | 2012-08-23 15:28:28 -0700 | [diff] [blame] | 1198 | if not tree.job.voting: |
James E. Blair | af17a97 | 2016-02-03 15:07:18 -0800 | [diff] [blame] | 1199 | tags.append('[nonvoting]') |
| 1200 | if tree.job.mutex: |
| 1201 | tags.append('[mutex: %s]' % tree.job.mutex) |
| 1202 | tags = ' '.join(tags) |
| 1203 | self.log.info("%s%s%s %s" % (istr, repr(tree.job), |
| 1204 | efilters, tags)) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1205 | for x in tree.job_trees: |
James E. Blair | 1e8dd89 | 2012-05-30 09:15:05 -0700 | [diff] [blame] | 1206 | log_jobs(x, indent + 2) |
| 1207 | |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 1208 | for p in layout.projects.values(): |
James E. Blair | 4aea70c | 2012-07-26 14:23:24 -0700 | [diff] [blame] | 1209 | tree = self.pipeline.getJobTree(p) |
| 1210 | if tree: |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1211 | self.log.info(" %s" % p) |
James E. Blair | 4aea70c | 2012-07-26 14:23:24 -0700 | [diff] [blame] | 1212 | log_jobs(tree) |
Joshua Hesketh | 1879cf7 | 2013-08-19 14:13:15 +1000 | [diff] [blame] | 1213 | self.log.info(" On start:") |
| 1214 | self.log.info(" %s" % self.pipeline.start_actions) |
| 1215 | self.log.info(" On success:") |
| 1216 | self.log.info(" %s" % self.pipeline.success_actions) |
| 1217 | self.log.info(" On failure:") |
| 1218 | self.log.info(" %s" % self.pipeline.failure_actions) |
Joshua Hesketh | b717977 | 2014-01-30 23:30:46 +1100 | [diff] [blame] | 1219 | self.log.info(" On merge-failure:") |
| 1220 | self.log.info(" %s" % self.pipeline.merge_failure_actions) |
Joshua Hesketh | 89e829d | 2015-02-10 16:29:45 +1100 | [diff] [blame] | 1221 | self.log.info(" When disabled:") |
| 1222 | self.log.info(" %s" % self.pipeline.disabled_actions) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1223 | |
James E. Blair | e421a23 | 2012-07-25 16:59:21 -0700 | [diff] [blame] | 1224 | def getSubmitAllowNeeds(self): |
| 1225 | # Get a list of code review labels that are allowed to be |
| 1226 | # "needed" in the submit records for a change, with respect |
| 1227 | # to this queue. In other words, the list of review labels |
| 1228 | # this queue itself is likely to set before submitting. |
Joshua Hesketh | 1879cf7 | 2013-08-19 14:13:15 +1000 | [diff] [blame] | 1229 | allow_needs = set() |
| 1230 | for action_reporter in self.pipeline.success_actions: |
| 1231 | allow_needs.update(action_reporter.getSubmitAllowNeeds()) |
| 1232 | return allow_needs |
James E. Blair | e421a23 | 2012-07-25 16:59:21 -0700 | [diff] [blame] | 1233 | |
James E. Blair | c053d02 | 2014-01-22 14:57:33 -0800 | [diff] [blame] | 1234 | def eventMatches(self, event, change): |
James E. Blair | ad28e91 | 2013-11-27 10:43:22 -0800 | [diff] [blame] | 1235 | if event.forced_pipeline: |
| 1236 | if event.forced_pipeline == self.pipeline.name: |
James E. Blair | 1b26531 | 2014-06-24 09:35:21 -0700 | [diff] [blame] | 1237 | self.log.debug("Event %s for change %s was directly assigned " |
| 1238 | "to pipeline %s" % (event, change, self)) |
James E. Blair | ad28e91 | 2013-11-27 10:43:22 -0800 | [diff] [blame] | 1239 | return True |
| 1240 | else: |
| 1241 | return False |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1242 | for ef in self.event_filters: |
James E. Blair | c053d02 | 2014-01-22 14:57:33 -0800 | [diff] [blame] | 1243 | if ef.matches(event, change): |
James E. Blair | 1b26531 | 2014-06-24 09:35:21 -0700 | [diff] [blame] | 1244 | self.log.debug("Event %s for change %s matched %s " |
| 1245 | "in pipeline %s" % (event, change, ef, self)) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1246 | return True |
| 1247 | return False |
| 1248 | |
James E. Blair | dbfe1cd | 2015-02-07 11:41:19 -0800 | [diff] [blame] | 1249 | def isChangeAlreadyInPipeline(self, change): |
| 1250 | # Checks live items in the pipeline |
| 1251 | for item in self.pipeline.getAllItems(): |
| 1252 | if item.live and change.equals(item.change): |
| 1253 | return True |
| 1254 | return False |
| 1255 | |
| 1256 | def isChangeAlreadyInQueue(self, change, change_queue): |
| 1257 | # Checks any item in the specified change queue |
| 1258 | for item in change_queue.queue: |
| 1259 | if change.equals(item.change): |
James E. Blair | 0dc8ba9 | 2012-07-16 14:23:52 -0700 | [diff] [blame] | 1260 | return True |
| 1261 | return False |
| 1262 | |
Joshua Hesketh | 385d11e | 2015-09-14 14:50:01 -0600 | [diff] [blame] | 1263 | def reportStart(self, item): |
Joshua Hesketh | 89e829d | 2015-02-10 16:29:45 +1100 | [diff] [blame] | 1264 | if not self.pipeline._disabled: |
| 1265 | try: |
Joshua Hesketh | 385d11e | 2015-09-14 14:50:01 -0600 | [diff] [blame] | 1266 | self.log.info("Reporting start, action %s item %s" % |
| 1267 | (self.pipeline.start_actions, item)) |
Joshua Hesketh | 89e829d | 2015-02-10 16:29:45 +1100 | [diff] [blame] | 1268 | ret = self.sendReport(self.pipeline.start_actions, |
Joshua Hesketh | 385d11e | 2015-09-14 14:50:01 -0600 | [diff] [blame] | 1269 | self.pipeline.source, item) |
Joshua Hesketh | 89e829d | 2015-02-10 16:29:45 +1100 | [diff] [blame] | 1270 | if ret: |
Joshua Hesketh | 385d11e | 2015-09-14 14:50:01 -0600 | [diff] [blame] | 1271 | self.log.error("Reporting item start %s received: %s" % |
| 1272 | (item, ret)) |
Joshua Hesketh | 89e829d | 2015-02-10 16:29:45 +1100 | [diff] [blame] | 1273 | except: |
| 1274 | self.log.exception("Exception while reporting start:") |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 1275 | |
Joshua Hesketh | 385d11e | 2015-09-14 14:50:01 -0600 | [diff] [blame] | 1276 | def sendReport(self, action_reporters, source, item, |
| 1277 | message=None): |
Joshua Hesketh | 1879cf7 | 2013-08-19 14:13:15 +1000 | [diff] [blame] | 1278 | """Sends the built message off to configured reporters. |
| 1279 | |
Joshua Hesketh | 385d11e | 2015-09-14 14:50:01 -0600 | [diff] [blame] | 1280 | Takes the action_reporters, item, message and extra options and |
Joshua Hesketh | 1879cf7 | 2013-08-19 14:13:15 +1000 | [diff] [blame] | 1281 | sends them to the pluggable reporters. |
| 1282 | """ |
| 1283 | report_errors = [] |
| 1284 | if len(action_reporters) > 0: |
Joshua Hesketh | de95865 | 2015-11-10 19:19:50 +1100 | [diff] [blame] | 1285 | for reporter in action_reporters: |
Joshua Hesketh | 385d11e | 2015-09-14 14:50:01 -0600 | [diff] [blame] | 1286 | ret = reporter.report(source, self.pipeline, item) |
Joshua Hesketh | 1879cf7 | 2013-08-19 14:13:15 +1000 | [diff] [blame] | 1287 | if ret: |
| 1288 | report_errors.append(ret) |
| 1289 | if len(report_errors) == 0: |
| 1290 | return |
| 1291 | return report_errors |
| 1292 | |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 1293 | def isChangeReadyToBeEnqueued(self, change): |
| 1294 | return True |
| 1295 | |
James E. Blair | 5ee2425 | 2014-12-30 10:12:29 -0800 | [diff] [blame] | 1296 | def enqueueChangesAhead(self, change, quiet, ignore_requirements, |
| 1297 | change_queue): |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 1298 | return True |
| 1299 | |
James E. Blair | 5ee2425 | 2014-12-30 10:12:29 -0800 | [diff] [blame] | 1300 | def enqueueChangesBehind(self, change, quiet, ignore_requirements, |
| 1301 | change_queue): |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 1302 | return True |
| 1303 | |
James E. Blair | dbfe1cd | 2015-02-07 11:41:19 -0800 | [diff] [blame] | 1304 | def checkForChangesNeededBy(self, change, change_queue): |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1305 | return True |
| 1306 | |
James E. Blair | 6965a4b | 2014-12-16 17:19:04 -0800 | [diff] [blame] | 1307 | def getFailingDependentItems(self, item): |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1308 | return None |
| 1309 | |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1310 | def getDependentItems(self, item): |
| 1311 | orig_item = item |
| 1312 | items = [] |
| 1313 | while item.item_ahead: |
| 1314 | items.append(item.item_ahead) |
| 1315 | item = item.item_ahead |
| 1316 | self.log.info("Change %s depends on changes %s" % |
| 1317 | (orig_item.change, |
| 1318 | [x.change for x in items])) |
| 1319 | return items |
| 1320 | |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1321 | def getItemForChange(self, change): |
| 1322 | for item in self.pipeline.getAllItems(): |
| 1323 | if item.change.equals(change): |
| 1324 | return item |
| 1325 | return None |
| 1326 | |
James E. Blair | 2fa5096 | 2013-01-30 21:50:41 -0800 | [diff] [blame] | 1327 | def findOldVersionOfChangeAlreadyInQueue(self, change): |
James E. Blair | ba43736 | 2015-02-07 11:41:52 -0800 | [diff] [blame] | 1328 | for item in self.pipeline.getAllItems(): |
| 1329 | if not item.live: |
| 1330 | continue |
| 1331 | if change.isUpdateOf(item.change): |
| 1332 | return item |
James E. Blair | 2fa5096 | 2013-01-30 21:50:41 -0800 | [diff] [blame] | 1333 | return None |
| 1334 | |
| 1335 | def removeOldVersionsOfChange(self, change): |
| 1336 | if not self.pipeline.dequeue_on_new_patchset: |
| 1337 | return |
James E. Blair | ba43736 | 2015-02-07 11:41:52 -0800 | [diff] [blame] | 1338 | old_item = self.findOldVersionOfChangeAlreadyInQueue(change) |
| 1339 | if old_item: |
James E. Blair | 2fa5096 | 2013-01-30 21:50:41 -0800 | [diff] [blame] | 1340 | self.log.debug("Change %s is a new version of %s, removing %s" % |
James E. Blair | ba43736 | 2015-02-07 11:41:52 -0800 | [diff] [blame] | 1341 | (change, old_item.change, old_item)) |
| 1342 | self.removeItem(old_item) |
James E. Blair | 2fa5096 | 2013-01-30 21:50:41 -0800 | [diff] [blame] | 1343 | |
Antoine Musso | bd86a31 | 2014-01-08 14:51:33 +0100 | [diff] [blame] | 1344 | def removeAbandonedChange(self, change): |
| 1345 | self.log.debug("Change %s abandoned, removing." % change) |
James E. Blair | ba43736 | 2015-02-07 11:41:52 -0800 | [diff] [blame] | 1346 | for item in self.pipeline.getAllItems(): |
| 1347 | if not item.live: |
| 1348 | continue |
| 1349 | if item.change.equals(change): |
| 1350 | self.removeItem(item) |
Antoine Musso | bd86a31 | 2014-01-08 14:51:33 +0100 | [diff] [blame] | 1351 | |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 1352 | def reEnqueueItem(self, item, last_head): |
James E. Blair | 0577cd6 | 2015-02-07 11:42:12 -0800 | [diff] [blame] | 1353 | with self.getChangeQueue(item.change, last_head.queue) as change_queue: |
| 1354 | if change_queue: |
| 1355 | self.log.debug("Re-enqueing change %s in queue %s" % |
| 1356 | (item.change, change_queue)) |
| 1357 | change_queue.enqueueItem(item) |
James E. Blair | 6bc782d | 2015-07-17 16:20:21 -0700 | [diff] [blame] | 1358 | |
| 1359 | # Re-set build results in case any new jobs have been |
| 1360 | # added to the tree. |
| 1361 | for build in item.current_build_set.getBuilds(): |
| 1362 | if build.result: |
| 1363 | self.pipeline.setResult(item, build) |
| 1364 | # Similarly, reset the item state. |
| 1365 | if item.current_build_set.unable_to_merge: |
| 1366 | self.pipeline.setUnableToMerge(item) |
| 1367 | if item.dequeued_needing_change: |
| 1368 | self.pipeline.setDequeuedNeedingChange(item) |
| 1369 | |
James E. Blair | 0577cd6 | 2015-02-07 11:42:12 -0800 | [diff] [blame] | 1370 | self.reportStats(item) |
| 1371 | return True |
| 1372 | else: |
| 1373 | self.log.error("Unable to find change queue for project %s" % |
| 1374 | item.change.project) |
| 1375 | return False |
James E. Blair | cdccd97 | 2013-07-01 12:10:22 -0700 | [diff] [blame] | 1376 | |
James E. Blair | f9ab884 | 2014-07-10 13:12:07 -0700 | [diff] [blame] | 1377 | def addChange(self, change, quiet=False, enqueue_time=None, |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 1378 | ignore_requirements=False, live=True, |
| 1379 | change_queue=None): |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 1380 | self.log.debug("Considering adding change %s" % change) |
James E. Blair | dbfe1cd | 2015-02-07 11:41:19 -0800 | [diff] [blame] | 1381 | |
| 1382 | # If we are adding a live change, check if it's a live item |
| 1383 | # anywhere in the pipeline. Otherwise, we will perform the |
| 1384 | # duplicate check below on the specific change_queue. |
| 1385 | if live and self.isChangeAlreadyInPipeline(change): |
| 1386 | self.log.debug("Change %s is already in pipeline, " |
| 1387 | "ignoring" % change) |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 1388 | return True |
James E. Blair | 692c6b3 | 2012-07-17 11:16:35 -0700 | [diff] [blame] | 1389 | |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 1390 | if not self.isChangeReadyToBeEnqueued(change): |
| 1391 | self.log.debug("Change %s is not ready to be enqueued, ignoring" % |
| 1392 | change) |
| 1393 | return False |
| 1394 | |
James E. Blair | f9ab884 | 2014-07-10 13:12:07 -0700 | [diff] [blame] | 1395 | if not ignore_requirements: |
| 1396 | for f in self.changeish_filters: |
| 1397 | if not f.matches(change): |
| 1398 | self.log.debug("Change %s does not match pipeline " |
| 1399 | "requirement %s" % (change, f)) |
| 1400 | return False |
James E. Blair | 11041d2 | 2014-05-02 14:49:53 -0700 | [diff] [blame] | 1401 | |
James E. Blair | 0577cd6 | 2015-02-07 11:42:12 -0800 | [diff] [blame] | 1402 | with self.getChangeQueue(change, change_queue) as change_queue: |
James E. Blair | 5ee2425 | 2014-12-30 10:12:29 -0800 | [diff] [blame] | 1403 | if not change_queue: |
| 1404 | self.log.debug("Unable to find change queue for " |
| 1405 | "change %s in project %s" % |
| 1406 | (change, change.project)) |
| 1407 | return False |
| 1408 | |
James E. Blair | 0577cd6 | 2015-02-07 11:42:12 -0800 | [diff] [blame] | 1409 | if not self.enqueueChangesAhead(change, quiet, ignore_requirements, |
| 1410 | change_queue): |
| 1411 | self.log.debug("Failed to enqueue changes " |
| 1412 | "ahead of %s" % change) |
| 1413 | return False |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 1414 | |
James E. Blair | 0577cd6 | 2015-02-07 11:42:12 -0800 | [diff] [blame] | 1415 | if self.isChangeAlreadyInQueue(change, change_queue): |
| 1416 | self.log.debug("Change %s is already in queue, " |
| 1417 | "ignoring" % change) |
| 1418 | return True |
| 1419 | |
| 1420 | self.log.debug("Adding change %s to queue %s" % |
| 1421 | (change, change_queue)) |
James E. Blair | 0577cd6 | 2015-02-07 11:42:12 -0800 | [diff] [blame] | 1422 | item = change_queue.enqueueChange(change) |
| 1423 | if enqueue_time: |
| 1424 | item.enqueue_time = enqueue_time |
| 1425 | item.live = live |
| 1426 | self.reportStats(item) |
Joshua Hesketh | 385d11e | 2015-09-14 14:50:01 -0600 | [diff] [blame] | 1427 | if not quiet: |
| 1428 | if len(self.pipeline.start_actions) > 0: |
| 1429 | self.reportStart(item) |
James E. Blair | 0577cd6 | 2015-02-07 11:42:12 -0800 | [diff] [blame] | 1430 | self.enqueueChangesBehind(change, quiet, ignore_requirements, |
| 1431 | change_queue) |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 1432 | for trigger in self.sched.triggers.values(): |
| 1433 | trigger.onChangeEnqueued(item.change, self.pipeline) |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 1434 | return True |
| 1435 | |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1436 | def dequeueItem(self, item): |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1437 | self.log.debug("Removing change %s from queue" % item.change) |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 1438 | item.queue.dequeueItem(item) |
James E. Blair | 2fa5096 | 2013-01-30 21:50:41 -0800 | [diff] [blame] | 1439 | |
James E. Blair | ba43736 | 2015-02-07 11:41:52 -0800 | [diff] [blame] | 1440 | def removeItem(self, item): |
| 1441 | # Remove an item from the queue, probably because it has been |
Alex Gaynor | 813d39b | 2014-05-17 16:17:16 -0700 | [diff] [blame] | 1442 | # superseded by another change. |
James E. Blair | ba43736 | 2015-02-07 11:41:52 -0800 | [diff] [blame] | 1443 | self.log.debug("Canceling builds behind change: %s " |
| 1444 | "because it is being removed." % item.change) |
| 1445 | self.cancelJobs(item) |
| 1446 | self.dequeueItem(item) |
| 1447 | self.reportStats(item) |
James E. Blair | 2fa5096 | 2013-01-30 21:50:41 -0800 | [diff] [blame] | 1448 | |
James E. Blair | ac2c324 | 2014-01-24 13:38:51 -0800 | [diff] [blame] | 1449 | def _makeMergerItem(self, item): |
| 1450 | # Create a dictionary with all info about the item needed by |
| 1451 | # the merger. |
Clark Boylan | 4c6566b | 2014-03-10 11:02:01 -0700 | [diff] [blame] | 1452 | number = None |
| 1453 | patchset = None |
| 1454 | oldrev = None |
| 1455 | newrev = None |
| 1456 | if hasattr(item.change, 'number'): |
| 1457 | number = item.change.number |
| 1458 | patchset = item.change.patchset |
| 1459 | elif hasattr(item.change, 'newrev'): |
| 1460 | oldrev = item.change.oldrev |
| 1461 | newrev = item.change.newrev |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 1462 | connection_name = self.pipeline.source.connection.connection_name |
James E. Blair | ac2c324 | 2014-01-24 13:38:51 -0800 | [diff] [blame] | 1463 | return dict(project=item.change.project.name, |
James E. Blair | c0dedf8 | 2014-08-06 09:37:52 -0700 | [diff] [blame] | 1464 | url=self.pipeline.source.getGitUrl( |
James E. Blair | ac2c324 | 2014-01-24 13:38:51 -0800 | [diff] [blame] | 1465 | item.change.project), |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 1466 | connection_name=connection_name, |
James E. Blair | ac2c324 | 2014-01-24 13:38:51 -0800 | [diff] [blame] | 1467 | merge_mode=item.change.project.merge_mode, |
| 1468 | refspec=item.change.refspec, |
| 1469 | branch=item.change.branch, |
| 1470 | ref=item.current_build_set.ref, |
Clark Boylan | 4c6566b | 2014-03-10 11:02:01 -0700 | [diff] [blame] | 1471 | number=number, |
| 1472 | patchset=patchset, |
| 1473 | oldrev=oldrev, |
| 1474 | newrev=newrev, |
James E. Blair | ac2c324 | 2014-01-24 13:38:51 -0800 | [diff] [blame] | 1475 | ) |
| 1476 | |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1477 | def prepareRef(self, item): |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 1478 | # Returns True if the ref is ready, false otherwise |
| 1479 | build_set = item.current_build_set |
| 1480 | if build_set.merge_state == build_set.COMPLETE: |
| 1481 | return True |
| 1482 | if build_set.merge_state == build_set.PENDING: |
| 1483 | return False |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 1484 | ref = build_set.ref |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1485 | if hasattr(item.change, 'refspec') and not ref: |
| 1486 | self.log.debug("Preparing ref for: %s" % item.change) |
| 1487 | item.current_build_set.setConfiguration() |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1488 | dependent_items = self.getDependentItems(item) |
| 1489 | dependent_items.reverse() |
| 1490 | all_items = dependent_items + [item] |
James E. Blair | ac2c324 | 2014-01-24 13:38:51 -0800 | [diff] [blame] | 1491 | merger_items = map(self._makeMergerItem, all_items) |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 1492 | self.sched.merger.mergeChanges(merger_items, |
James E. Blair | e9a8184 | 2014-09-24 13:37:45 -0700 | [diff] [blame] | 1493 | item.current_build_set, |
| 1494 | self.pipeline.precedence) |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 1495 | else: |
| 1496 | self.log.debug("Preparing update repo for: %s" % item.change) |
James E. Blair | c0dedf8 | 2014-08-06 09:37:52 -0700 | [diff] [blame] | 1497 | url = self.pipeline.source.getGitUrl(item.change.project) |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 1498 | self.sched.merger.updateRepo(item.change.project.name, |
James E. Blair | e9a8184 | 2014-09-24 13:37:45 -0700 | [diff] [blame] | 1499 | url, build_set, |
| 1500 | self.pipeline.precedence) |
Antoine Musso | fe20ccf | 2014-10-16 15:32:14 +0200 | [diff] [blame] | 1501 | # merge:merge has been emitted properly: |
| 1502 | build_set.merge_state = build_set.PENDING |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1503 | return False |
| 1504 | |
| 1505 | def _launchJobs(self, item, jobs): |
| 1506 | self.log.debug("Launching jobs for change %s" % item.change) |
| 1507 | dependent_items = self.getDependentItems(item) |
| 1508 | for job in jobs: |
| 1509 | self.log.debug("Found job %s for change %s" % (job, item.change)) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1510 | try: |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1511 | build = self.sched.launcher.launch(job, item, |
| 1512 | self.pipeline, |
| 1513 | dependent_items) |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1514 | self.log.debug("Adding build %s of job %s to item %s" % |
| 1515 | (build, job, item)) |
| 1516 | item.addBuild(build) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1517 | except: |
Zhongyue Luo | 1c860d7 | 2012-07-19 11:03:56 +0800 | [diff] [blame] | 1518 | self.log.exception("Exception while launching job %s " |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1519 | "for change %s:" % (job, item.change)) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1520 | |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1521 | def launchJobs(self, item): |
James E. Blair | af17a97 | 2016-02-03 15:07:18 -0800 | [diff] [blame] | 1522 | jobs = self.pipeline.findJobsToRun(item, self.sched.mutex) |
James E. Blair | daabed2 | 2012-08-15 15:38:57 -0700 | [diff] [blame] | 1523 | if jobs: |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1524 | self._launchJobs(item, jobs) |
| 1525 | |
| 1526 | def cancelJobs(self, item, prime=True): |
| 1527 | self.log.debug("Cancel jobs for change %s" % item.change) |
| 1528 | canceled = False |
James E. Blair | 6b07794 | 2014-02-07 17:45:55 -0800 | [diff] [blame] | 1529 | old_build_set = item.current_build_set |
James E. Blair | 36658cf | 2013-12-06 17:53:48 -0800 | [diff] [blame] | 1530 | if prime and item.current_build_set.ref: |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1531 | item.resetAllBuilds() |
James E. Blair | 6b07794 | 2014-02-07 17:45:55 -0800 | [diff] [blame] | 1532 | for build in old_build_set.getBuilds(): |
| 1533 | try: |
| 1534 | self.sched.launcher.cancel(build) |
| 1535 | except: |
| 1536 | self.log.exception("Exception while canceling build %s " |
| 1537 | "for change %s" % (build, item.change)) |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1538 | build.result = 'CANCELED' |
James E. Blair | 6b07794 | 2014-02-07 17:45:55 -0800 | [diff] [blame] | 1539 | canceled = True |
James E. Blair | 82a42d1 | 2015-07-20 13:50:09 -0700 | [diff] [blame] | 1540 | self.updateBuildDescriptions(old_build_set) |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1541 | for item_behind in item.items_behind: |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1542 | self.log.debug("Canceling jobs for change %s, behind change %s" % |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1543 | (item_behind.change, item.change)) |
| 1544 | if self.cancelJobs(item_behind, prime=prime): |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1545 | canceled = True |
| 1546 | return canceled |
| 1547 | |
Dmitry Teselkin | 116fef3 | 2015-04-20 14:32:14 +0300 | [diff] [blame] | 1548 | def _processOneItem(self, item, nnfi): |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1549 | changed = False |
| 1550 | item_ahead = item.item_ahead |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 1551 | if item_ahead and (not item_ahead.live): |
| 1552 | item_ahead = None |
| 1553 | change_queue = item.queue |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1554 | failing_reasons = [] # Reasons this item is failing |
| 1555 | |
James E. Blair | dbfe1cd | 2015-02-07 11:41:19 -0800 | [diff] [blame] | 1556 | if self.checkForChangesNeededBy(item.change, change_queue) is not True: |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1557 | # It's not okay to enqueue this change, we should remove it. |
| 1558 | self.log.info("Dequeuing change %s because " |
| 1559 | "it can no longer merge" % item.change) |
| 1560 | self.cancelJobs(item) |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1561 | self.dequeueItem(item) |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1562 | self.pipeline.setDequeuedNeedingChange(item) |
James E. Blair | f8b42fb | 2015-02-18 09:23:36 -0800 | [diff] [blame] | 1563 | if item.live: |
| 1564 | try: |
| 1565 | self.reportItem(item) |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 1566 | except exceptions.MergeFailure: |
James E. Blair | f8b42fb | 2015-02-18 09:23:36 -0800 | [diff] [blame] | 1567 | pass |
Dmitry Teselkin | 116fef3 | 2015-04-20 14:32:14 +0300 | [diff] [blame] | 1568 | return (True, nnfi) |
James E. Blair | 6965a4b | 2014-12-16 17:19:04 -0800 | [diff] [blame] | 1569 | dep_items = self.getFailingDependentItems(item) |
Clark Boylan | af2476f | 2014-01-23 14:47:36 -0800 | [diff] [blame] | 1570 | actionable = change_queue.isActionable(item) |
| 1571 | item.active = actionable |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 1572 | ready = False |
James E. Blair | 6965a4b | 2014-12-16 17:19:04 -0800 | [diff] [blame] | 1573 | if dep_items: |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1574 | failing_reasons.append('a needed change is failing') |
| 1575 | self.cancelJobs(item, prime=False) |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1576 | else: |
James E. Blair | fef7163 | 2013-09-23 11:15:47 -0700 | [diff] [blame] | 1577 | item_ahead_merged = False |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 1578 | if (item_ahead and item_ahead.change.is_merged): |
James E. Blair | fef7163 | 2013-09-23 11:15:47 -0700 | [diff] [blame] | 1579 | item_ahead_merged = True |
| 1580 | if (item_ahead != nnfi and not item_ahead_merged): |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1581 | # Our current base is different than what we expected, |
| 1582 | # and it's not because our current base merged. Something |
| 1583 | # ahead must have failed. |
| 1584 | self.log.info("Resetting builds for change %s because the " |
| 1585 | "item ahead, %s, is not the nearest non-failing " |
| 1586 | "item, %s" % (item.change, item_ahead, nnfi)) |
| 1587 | change_queue.moveItem(item, nnfi) |
| 1588 | changed = True |
| 1589 | self.cancelJobs(item) |
Clark Boylan | af2476f | 2014-01-23 14:47:36 -0800 | [diff] [blame] | 1590 | if actionable: |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 1591 | ready = self.prepareRef(item) |
Clark Boylan | 3d2f7a7 | 2014-01-23 11:07:42 -0800 | [diff] [blame] | 1592 | if item.current_build_set.unable_to_merge: |
| 1593 | failing_reasons.append("it has a merge conflict") |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 1594 | ready = False |
Dmitry Teselkin | 116fef3 | 2015-04-20 14:32:14 +0300 | [diff] [blame] | 1595 | if actionable and ready and self.launchJobs(item): |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1596 | changed = True |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1597 | if self.pipeline.didAnyJobFail(item): |
| 1598 | failing_reasons.append("at least one job failed") |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 1599 | if (not item.live) and (not item.items_behind): |
| 1600 | failing_reasons.append("is a non-live item with no items behind") |
| 1601 | self.dequeueItem(item) |
| 1602 | changed = True |
James E. Blair | ec2e156 | 2015-02-05 10:45:54 -0800 | [diff] [blame] | 1603 | if ((not item_ahead) and self.pipeline.areAllJobsComplete(item) |
| 1604 | and item.live): |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1605 | try: |
| 1606 | self.reportItem(item) |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 1607 | except exceptions.MergeFailure: |
James E. Blair | 062c4fb | 2013-09-26 07:46:00 -0700 | [diff] [blame] | 1608 | failing_reasons.append("it did not merge") |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1609 | for item_behind in item.items_behind: |
| 1610 | self.log.info("Resetting builds for change %s because the " |
| 1611 | "item ahead, %s, failed to merge" % |
| 1612 | (item_behind.change, item)) |
| 1613 | self.cancelJobs(item_behind) |
| 1614 | self.dequeueItem(item) |
| 1615 | changed = True |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 1616 | elif not failing_reasons and item.live: |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1617 | nnfi = item |
| 1618 | item.current_build_set.failing_reasons = failing_reasons |
| 1619 | if failing_reasons: |
| 1620 | self.log.debug("%s is a failing item because %s" % |
| 1621 | (item, failing_reasons)) |
Dmitry Teselkin | 116fef3 | 2015-04-20 14:32:14 +0300 | [diff] [blame] | 1622 | return (changed, nnfi) |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1623 | |
| 1624 | def processQueue(self): |
| 1625 | # Do whatever needs to be done for each change in the queue |
| 1626 | self.log.debug("Starting queue processor: %s" % self.pipeline.name) |
| 1627 | changed = False |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1628 | for queue in self.pipeline.queues: |
| 1629 | queue_changed = False |
| 1630 | nnfi = None # Nearest non-failing item |
Clark Boylan | 3d2f7a7 | 2014-01-23 11:07:42 -0800 | [diff] [blame] | 1631 | for item in queue.queue[:]: |
Dmitry Teselkin | 116fef3 | 2015-04-20 14:32:14 +0300 | [diff] [blame] | 1632 | item_changed, nnfi = self._processOneItem( |
| 1633 | item, nnfi) |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1634 | if item_changed: |
| 1635 | queue_changed = True |
| 1636 | self.reportStats(item) |
| 1637 | if queue_changed: |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1638 | changed = True |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1639 | status = '' |
| 1640 | for item in queue.queue: |
Joshua Hesketh | 85af4e9 | 2014-02-21 08:28:58 -0800 | [diff] [blame] | 1641 | status += item.formatStatus() |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 1642 | if status: |
| 1643 | self.log.debug("Queue %s status is now:\n %s" % |
| 1644 | (queue.name, status)) |
James E. Blair | fadc6e1 | 2013-08-21 18:23:15 -0700 | [diff] [blame] | 1645 | self.log.debug("Finished queue processor: %s (changed: %s)" % |
| 1646 | (self.pipeline.name, changed)) |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1647 | return changed |
James E. Blair | daabed2 | 2012-08-15 15:38:57 -0700 | [diff] [blame] | 1648 | |
James E. Blair | 11700c3 | 2012-07-05 17:50:05 -0700 | [diff] [blame] | 1649 | def updateBuildDescriptions(self, build_set): |
| 1650 | for build in build_set.getBuilds(): |
Richard Hedlind | 13cb40c | 2015-12-18 13:45:58 -0700 | [diff] [blame] | 1651 | try: |
| 1652 | desc = self.formatDescription(build) |
| 1653 | self.sched.launcher.setBuildDescription(build, desc) |
| 1654 | except: |
| 1655 | # Log the failure and let loop continue |
| 1656 | self.log.error("Failed to update description for build %s" % |
| 1657 | (build)) |
James E. Blair | 11700c3 | 2012-07-05 17:50:05 -0700 | [diff] [blame] | 1658 | |
| 1659 | if build_set.previous_build_set: |
| 1660 | for build in build_set.previous_build_set.getBuilds(): |
Richard Hedlind | 13cb40c | 2015-12-18 13:45:58 -0700 | [diff] [blame] | 1661 | try: |
| 1662 | desc = self.formatDescription(build) |
| 1663 | self.sched.launcher.setBuildDescription(build, desc) |
| 1664 | except: |
| 1665 | # Log the failure and let loop continue |
| 1666 | self.log.error("Failed to update description for " |
| 1667 | "build %s in previous build set" % (build)) |
James E. Blair | 11700c3 | 2012-07-05 17:50:05 -0700 | [diff] [blame] | 1668 | |
| 1669 | def onBuildStarted(self, build): |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1670 | self.log.debug("Build %s started" % build) |
James E. Blair | 11700c3 | 2012-07-05 17:50:05 -0700 | [diff] [blame] | 1671 | return True |
| 1672 | |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1673 | def onBuildCompleted(self, build): |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1674 | self.log.debug("Build %s completed" % build) |
James E. Blair | 6b07794 | 2014-02-07 17:45:55 -0800 | [diff] [blame] | 1675 | item = build.build_set.item |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1676 | |
James E. Blair | 6b07794 | 2014-02-07 17:45:55 -0800 | [diff] [blame] | 1677 | self.pipeline.setResult(item, build) |
James E. Blair | af17a97 | 2016-02-03 15:07:18 -0800 | [diff] [blame] | 1678 | self.sched.mutex.release(item, build.job) |
James E. Blair | 6b07794 | 2014-02-07 17:45:55 -0800 | [diff] [blame] | 1679 | self.log.debug("Item %s status is now:\n %s" % |
Joshua Hesketh | 85af4e9 | 2014-02-21 08:28:58 -0800 | [diff] [blame] | 1680 | (item, item.formatStatus())) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1681 | return True |
| 1682 | |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 1683 | def onMergeCompleted(self, event): |
| 1684 | build_set = event.build_set |
| 1685 | item = build_set.item |
| 1686 | build_set.merge_state = build_set.COMPLETE |
| 1687 | build_set.zuul_url = event.zuul_url |
| 1688 | if event.merged: |
| 1689 | build_set.commit = event.commit |
| 1690 | elif event.updated: |
Evgeny Antyshev | b078bb4 | 2016-01-25 14:44:21 +0000 | [diff] [blame] | 1691 | if not isinstance(item.change, NullChange): |
Yolanda Robla | 276996c | 2015-11-10 10:41:18 +0100 | [diff] [blame] | 1692 | build_set.commit = item.change.newrev |
Evgeny Antyshev | b078bb4 | 2016-01-25 14:44:21 +0000 | [diff] [blame] | 1693 | if not build_set.commit and not isinstance(item.change, NullChange): |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 1694 | self.log.info("Unable to merge change %s" % item.change) |
Joshua Hesketh | b717977 | 2014-01-30 23:30:46 +1100 | [diff] [blame] | 1695 | self.pipeline.setUnableToMerge(item) |
James E. Blair | 4076e2b | 2014-01-28 12:42:20 -0800 | [diff] [blame] | 1696 | |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1697 | def reportItem(self, item): |
Clark Boylan | f7dc4da | 2014-07-28 10:12:25 -0700 | [diff] [blame] | 1698 | if not item.reported: |
| 1699 | # _reportItem() returns True if it failed to report. |
| 1700 | item.reported = not self._reportItem(item) |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1701 | if self.changes_merge: |
| 1702 | succeeded = self.pipeline.didAllJobsSucceed(item) |
Clark Boylan | f7dc4da | 2014-07-28 10:12:25 -0700 | [diff] [blame] | 1703 | merged = item.reported |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1704 | if merged: |
James E. Blair | c0dedf8 | 2014-08-06 09:37:52 -0700 | [diff] [blame] | 1705 | merged = self.pipeline.source.isMerged(item.change, |
| 1706 | item.change.branch) |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1707 | self.log.info("Reported change %s status: all-succeeded: %s, " |
| 1708 | "merged: %s" % (item.change, succeeded, merged)) |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 1709 | change_queue = item.queue |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1710 | if not (succeeded and merged): |
| 1711 | self.log.debug("Reported change %s failed tests or failed " |
| 1712 | "to merge" % (item.change)) |
James E. Blair | 4a035d9 | 2014-01-23 13:10:48 -0800 | [diff] [blame] | 1713 | change_queue.decreaseWindowSize() |
Clark Boylan | 7603a37 | 2014-01-21 11:43:20 -0800 | [diff] [blame] | 1714 | self.log.debug("%s window size decreased to %s" % |
James E. Blair | 4a035d9 | 2014-01-23 13:10:48 -0800 | [diff] [blame] | 1715 | (change_queue, change_queue.window)) |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 1716 | raise exceptions.MergeFailure( |
| 1717 | "Change %s failed to merge" % item.change) |
Clark Boylan | 7603a37 | 2014-01-21 11:43:20 -0800 | [diff] [blame] | 1718 | else: |
James E. Blair | 4a035d9 | 2014-01-23 13:10:48 -0800 | [diff] [blame] | 1719 | change_queue.increaseWindowSize() |
Clark Boylan | 7603a37 | 2014-01-21 11:43:20 -0800 | [diff] [blame] | 1720 | self.log.debug("%s window size increased to %s" % |
James E. Blair | 4a035d9 | 2014-01-23 13:10:48 -0800 | [diff] [blame] | 1721 | (change_queue, change_queue.window)) |
Joshua Hesketh | 352264b | 2015-08-11 23:42:08 +1000 | [diff] [blame] | 1722 | |
| 1723 | for trigger in self.sched.triggers.values(): |
| 1724 | trigger.onChangeMerged(item.change, self.pipeline.source) |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 1725 | |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1726 | def _reportItem(self, item): |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1727 | self.log.debug("Reporting change %s" % item.change) |
James E. Blair | b98fcdb | 2013-08-26 18:23:09 -0700 | [diff] [blame] | 1728 | ret = True # Means error as returned by trigger.report |
Evgeny Antyshev | 88db9cb | 2015-06-04 12:51:40 +0000 | [diff] [blame] | 1729 | if not self.pipeline.getJobs(item): |
| 1730 | # We don't send empty reports with +1, |
| 1731 | # and the same for -1's (merge failures or transient errors) |
| 1732 | # as they cannot be followed by +1's |
| 1733 | self.log.debug("No jobs for change %s" % item.change) |
| 1734 | actions = [] |
| 1735 | elif self.pipeline.didAllJobsSucceed(item): |
Joshua Hesketh | b717977 | 2014-01-30 23:30:46 +1100 | [diff] [blame] | 1736 | self.log.debug("success %s" % (self.pipeline.success_actions)) |
Joshua Hesketh | 1879cf7 | 2013-08-19 14:13:15 +1000 | [diff] [blame] | 1737 | actions = self.pipeline.success_actions |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1738 | item.setReportedResult('SUCCESS') |
Joshua Hesketh | 89e829d | 2015-02-10 16:29:45 +1100 | [diff] [blame] | 1739 | self.pipeline._consecutive_failures = 0 |
Joshua Hesketh | b717977 | 2014-01-30 23:30:46 +1100 | [diff] [blame] | 1740 | elif not self.pipeline.didMergerSucceed(item): |
| 1741 | actions = self.pipeline.merge_failure_actions |
| 1742 | item.setReportedResult('MERGER_FAILURE') |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1743 | else: |
Joshua Hesketh | 1879cf7 | 2013-08-19 14:13:15 +1000 | [diff] [blame] | 1744 | actions = self.pipeline.failure_actions |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1745 | item.setReportedResult('FAILURE') |
Joshua Hesketh | 89e829d | 2015-02-10 16:29:45 +1100 | [diff] [blame] | 1746 | self.pipeline._consecutive_failures += 1 |
| 1747 | if self.pipeline._disabled: |
| 1748 | actions = self.pipeline.disabled_actions |
| 1749 | # Check here if we should disable so that we only use the disabled |
| 1750 | # reporters /after/ the last disable_at failure is still reported as |
| 1751 | # normal. |
| 1752 | if (self.pipeline.disable_at and not self.pipeline._disabled and |
| 1753 | self.pipeline._consecutive_failures >= self.pipeline.disable_at): |
| 1754 | self.pipeline._disabled = True |
James E. Blair | e591020 | 2013-12-27 09:50:31 -0800 | [diff] [blame] | 1755 | if actions: |
James E. Blair | e591020 | 2013-12-27 09:50:31 -0800 | [diff] [blame] | 1756 | try: |
Joshua Hesketh | 385d11e | 2015-09-14 14:50:01 -0600 | [diff] [blame] | 1757 | self.log.info("Reporting item %s, actions: %s" % |
| 1758 | (item, actions)) |
| 1759 | ret = self.sendReport(actions, self.pipeline.source, item) |
James E. Blair | e591020 | 2013-12-27 09:50:31 -0800 | [diff] [blame] | 1760 | if ret: |
Joshua Hesketh | 385d11e | 2015-09-14 14:50:01 -0600 | [diff] [blame] | 1761 | self.log.error("Reporting item %s received: %s" % |
| 1762 | (item, ret)) |
James E. Blair | e591020 | 2013-12-27 09:50:31 -0800 | [diff] [blame] | 1763 | except: |
| 1764 | self.log.exception("Exception while reporting:") |
| 1765 | item.setReportedResult('ERROR') |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1766 | self.updateBuildDescriptions(item.current_build_set) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 1767 | return ret |
| 1768 | |
James E. Blair | 8b0d4c4 | 2012-08-23 16:03:05 -0700 | [diff] [blame] | 1769 | def formatDescription(self, build): |
| 1770 | concurrent_changes = '' |
| 1771 | concurrent_builds = '' |
| 1772 | other_builds = '' |
| 1773 | |
| 1774 | for change in build.build_set.other_changes: |
| 1775 | concurrent_changes += '<li><a href="{change.url}">\ |
| 1776 | {change.number},{change.patchset}</a></li>'.format( |
| 1777 | change=change) |
| 1778 | |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1779 | change = build.build_set.item.change |
James E. Blair | 8b0d4c4 | 2012-08-23 16:03:05 -0700 | [diff] [blame] | 1780 | |
| 1781 | for build in build.build_set.getBuilds(): |
Ori Livneh | 7191ee8 | 2013-05-02 19:13:53 -0700 | [diff] [blame] | 1782 | if build.url: |
James E. Blair | 8b0d4c4 | 2012-08-23 16:03:05 -0700 | [diff] [blame] | 1783 | concurrent_builds += """\ |
| 1784 | <li> |
Ori Livneh | 7191ee8 | 2013-05-02 19:13:53 -0700 | [diff] [blame] | 1785 | <a href="{build.url}"> |
James E. Blair | 8b0d4c4 | 2012-08-23 16:03:05 -0700 | [diff] [blame] | 1786 | {build.job.name} #{build.number}</a>: {build.result} |
| 1787 | </li> |
| 1788 | """.format(build=build) |
| 1789 | else: |
| 1790 | concurrent_builds += """\ |
| 1791 | <li> |
| 1792 | {build.job.name}: {build.result} |
| 1793 | </li>""".format(build=build) |
| 1794 | |
| 1795 | if build.build_set.previous_build_set: |
| 1796 | other_build = build.build_set.previous_build_set.getBuild( |
| 1797 | build.job.name) |
| 1798 | if other_build: |
| 1799 | other_builds += """\ |
| 1800 | <li> |
Ori Livneh | 7191ee8 | 2013-05-02 19:13:53 -0700 | [diff] [blame] | 1801 | Preceded by: <a href="{build.url}"> |
James E. Blair | 8b0d4c4 | 2012-08-23 16:03:05 -0700 | [diff] [blame] | 1802 | {build.job.name} #{build.number}</a> |
| 1803 | </li> |
| 1804 | """.format(build=other_build) |
| 1805 | |
| 1806 | if build.build_set.next_build_set: |
| 1807 | other_build = build.build_set.next_build_set.getBuild( |
| 1808 | build.job.name) |
| 1809 | if other_build: |
| 1810 | other_builds += """\ |
| 1811 | <li> |
Ori Livneh | 7191ee8 | 2013-05-02 19:13:53 -0700 | [diff] [blame] | 1812 | Succeeded by: <a href="{build.url}"> |
James E. Blair | 8b0d4c4 | 2012-08-23 16:03:05 -0700 | [diff] [blame] | 1813 | {build.job.name} #{build.number}</a> |
| 1814 | </li> |
| 1815 | """.format(build=other_build) |
| 1816 | |
| 1817 | result = build.build_set.result |
| 1818 | |
| 1819 | if hasattr(change, 'number'): |
| 1820 | ret = """\ |
| 1821 | <p> |
| 1822 | Triggered by change: |
| 1823 | <a href="{change.url}">{change.number},{change.patchset}</a><br/> |
| 1824 | Branch: <b>{change.branch}</b><br/> |
| 1825 | Pipeline: <b>{self.pipeline.name}</b> |
| 1826 | </p>""" |
James E. Blair | 63bb0ef | 2013-07-29 17:14:51 -0700 | [diff] [blame] | 1827 | elif hasattr(change, 'ref'): |
James E. Blair | 8b0d4c4 | 2012-08-23 16:03:05 -0700 | [diff] [blame] | 1828 | ret = """\ |
| 1829 | <p> |
| 1830 | Triggered by reference: |
| 1831 | {change.ref}</a><br/> |
| 1832 | Old revision: <b>{change.oldrev}</b><br/> |
| 1833 | New revision: <b>{change.newrev}</b><br/> |
| 1834 | Pipeline: <b>{self.pipeline.name}</b> |
| 1835 | </p>""" |
James E. Blair | 63bb0ef | 2013-07-29 17:14:51 -0700 | [diff] [blame] | 1836 | else: |
| 1837 | ret = "" |
James E. Blair | 8b0d4c4 | 2012-08-23 16:03:05 -0700 | [diff] [blame] | 1838 | |
| 1839 | if concurrent_changes: |
| 1840 | ret += """\ |
| 1841 | <p> |
| 1842 | Other changes tested concurrently with this change: |
| 1843 | <ul>{concurrent_changes}</ul> |
| 1844 | </p> |
| 1845 | """ |
| 1846 | if concurrent_builds: |
| 1847 | ret += """\ |
| 1848 | <p> |
| 1849 | All builds for this change set: |
| 1850 | <ul>{concurrent_builds}</ul> |
| 1851 | </p> |
| 1852 | """ |
| 1853 | |
| 1854 | if other_builds: |
| 1855 | ret += """\ |
| 1856 | <p> |
| 1857 | Other build sets for this change: |
| 1858 | <ul>{other_builds}</ul> |
| 1859 | </p> |
| 1860 | """ |
| 1861 | if result: |
| 1862 | ret += """\ |
| 1863 | <p> |
| 1864 | Reported result: <b>{result}</b> |
| 1865 | </p> |
| 1866 | """ |
| 1867 | |
| 1868 | ret = ret.format(**locals()) |
James E. Blair | 268d934 | 2012-06-13 18:24:29 -0700 | [diff] [blame] | 1869 | return ret |
| 1870 | |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1871 | def reportStats(self, item): |
James E. Blair | 8fa1697 | 2013-01-15 16:57:20 -0800 | [diff] [blame] | 1872 | if not statsd: |
| 1873 | return |
| 1874 | try: |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1875 | # Update the gauge on enqueue and dequeue, but timers only |
James E. Blair | 8fa1697 | 2013-01-15 16:57:20 -0800 | [diff] [blame] | 1876 | # when dequeing. |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1877 | if item.dequeue_time: |
| 1878 | dt = int((item.dequeue_time - item.enqueue_time) * 1000) |
James E. Blair | 8fa1697 | 2013-01-15 16:57:20 -0800 | [diff] [blame] | 1879 | else: |
| 1880 | dt = None |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1881 | items = len(self.pipeline.getAllItems()) |
James E. Blair | 8fa1697 | 2013-01-15 16:57:20 -0800 | [diff] [blame] | 1882 | |
| 1883 | # stats.timers.zuul.pipeline.NAME.resident_time |
| 1884 | # stats_counts.zuul.pipeline.NAME.total_changes |
| 1885 | # stats.gauges.zuul.pipeline.NAME.current_changes |
| 1886 | key = 'zuul.pipeline.%s' % self.pipeline.name |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1887 | statsd.gauge(key + '.current_changes', items) |
James E. Blair | 8fa1697 | 2013-01-15 16:57:20 -0800 | [diff] [blame] | 1888 | if dt: |
| 1889 | statsd.timing(key + '.resident_time', dt) |
| 1890 | statsd.incr(key + '.total_changes') |
| 1891 | |
| 1892 | # stats.timers.zuul.pipeline.NAME.ORG.PROJECT.resident_time |
| 1893 | # stats_counts.zuul.pipeline.NAME.ORG.PROJECT.total_changes |
James E. Blair | fee8d65 | 2013-06-07 08:57:52 -0700 | [diff] [blame] | 1894 | project_name = item.change.project.name.replace('/', '.') |
James E. Blair | 8fa1697 | 2013-01-15 16:57:20 -0800 | [diff] [blame] | 1895 | key += '.%s' % project_name |
| 1896 | if dt: |
| 1897 | statsd.timing(key + '.resident_time', dt) |
| 1898 | statsd.incr(key + '.total_changes') |
| 1899 | except: |
| 1900 | self.log.exception("Exception reporting pipeline stats") |
| 1901 | |
James E. Blair | 1e8dd89 | 2012-05-30 09:15:05 -0700 | [diff] [blame] | 1902 | |
James E. Blair | 0577cd6 | 2015-02-07 11:42:12 -0800 | [diff] [blame] | 1903 | class DynamicChangeQueueContextManager(object): |
| 1904 | def __init__(self, change_queue): |
| 1905 | self.change_queue = change_queue |
| 1906 | |
| 1907 | def __enter__(self): |
| 1908 | return self.change_queue |
| 1909 | |
| 1910 | def __exit__(self, etype, value, tb): |
| 1911 | if self.change_queue and not self.change_queue.queue: |
| 1912 | self.change_queue.pipeline.removeQueue(self.change_queue.queue) |
| 1913 | |
| 1914 | |
James E. Blair | 4aea70c | 2012-07-26 14:23:24 -0700 | [diff] [blame] | 1915 | class IndependentPipelineManager(BasePipelineManager): |
| 1916 | log = logging.getLogger("zuul.IndependentPipelineManager") |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 1917 | changes_merge = False |
| 1918 | |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 1919 | def _postConfig(self, layout): |
| 1920 | super(IndependentPipelineManager, self)._postConfig(layout) |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 1921 | |
James E. Blair | 0577cd6 | 2015-02-07 11:42:12 -0800 | [diff] [blame] | 1922 | def getChangeQueue(self, change, existing=None): |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 1923 | # creates a new change queue for every change |
James E. Blair | 0577cd6 | 2015-02-07 11:42:12 -0800 | [diff] [blame] | 1924 | if existing: |
| 1925 | return DynamicChangeQueueContextManager(existing) |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 1926 | if change.project not in self.pipeline.getProjects(): |
Evgeny Antyshev | 0deaaad | 2015-08-03 20:22:56 +0000 | [diff] [blame] | 1927 | self.pipeline.addProject(change.project) |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 1928 | change_queue = ChangeQueue(self.pipeline) |
| 1929 | change_queue.addProject(change.project) |
| 1930 | self.pipeline.addQueue(change_queue) |
Evgeny Antyshev | 0deaaad | 2015-08-03 20:22:56 +0000 | [diff] [blame] | 1931 | self.log.debug("Dynamically created queue %s", change_queue) |
James E. Blair | 0577cd6 | 2015-02-07 11:42:12 -0800 | [diff] [blame] | 1932 | return DynamicChangeQueueContextManager(change_queue) |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 1933 | |
| 1934 | def enqueueChangesAhead(self, change, quiet, ignore_requirements, |
| 1935 | change_queue): |
James E. Blair | dbfe1cd | 2015-02-07 11:41:19 -0800 | [diff] [blame] | 1936 | ret = self.checkForChangesNeededBy(change, change_queue) |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 1937 | if ret in [True, False]: |
| 1938 | return ret |
| 1939 | self.log.debug(" Changes %s must be merged ahead of %s" % |
| 1940 | (ret, change)) |
| 1941 | for needed_change in ret: |
| 1942 | # This differs from the dependent pipeline by enqueuing |
| 1943 | # changes ahead as "not live", that is, not intended to |
| 1944 | # have jobs run. Also, pipeline requirements are always |
| 1945 | # ignored (which is safe because the changes are not |
| 1946 | # live). |
| 1947 | r = self.addChange(needed_change, quiet=True, |
| 1948 | ignore_requirements=True, |
| 1949 | live=False, change_queue=change_queue) |
| 1950 | if not r: |
| 1951 | return False |
| 1952 | return True |
| 1953 | |
James E. Blair | dbfe1cd | 2015-02-07 11:41:19 -0800 | [diff] [blame] | 1954 | def checkForChangesNeededBy(self, change, change_queue): |
James E. Blair | 17dd677 | 2015-02-09 14:45:18 -0800 | [diff] [blame] | 1955 | if self.pipeline.ignore_dependencies: |
| 1956 | return True |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 1957 | self.log.debug("Checking for changes needed by %s:" % change) |
| 1958 | # Return true if okay to proceed enqueing this change, |
| 1959 | # false if the change should not be enqueued. |
| 1960 | if not hasattr(change, 'needs_changes'): |
| 1961 | self.log.debug(" Changeish does not support dependencies") |
| 1962 | return True |
| 1963 | if not change.needs_changes: |
| 1964 | self.log.debug(" No changes needed") |
| 1965 | return True |
| 1966 | changes_needed = [] |
| 1967 | for needed_change in change.needs_changes: |
| 1968 | self.log.debug(" Change %s needs change %s:" % ( |
| 1969 | change, needed_change)) |
| 1970 | if needed_change.is_merged: |
| 1971 | self.log.debug(" Needed change is merged") |
| 1972 | continue |
James E. Blair | dbfe1cd | 2015-02-07 11:41:19 -0800 | [diff] [blame] | 1973 | if self.isChangeAlreadyInQueue(needed_change, change_queue): |
James E. Blair | bfb8e04 | 2014-12-30 17:01:44 -0800 | [diff] [blame] | 1974 | self.log.debug(" Needed change is already ahead in the queue") |
| 1975 | continue |
| 1976 | self.log.debug(" Change %s is needed" % needed_change) |
| 1977 | if needed_change not in changes_needed: |
| 1978 | changes_needed.append(needed_change) |
| 1979 | continue |
| 1980 | # This differs from the dependent pipeline check in not |
| 1981 | # verifying that the dependent change is mergable. |
| 1982 | if changes_needed: |
| 1983 | return changes_needed |
| 1984 | return True |
| 1985 | |
| 1986 | def dequeueItem(self, item): |
| 1987 | super(IndependentPipelineManager, self).dequeueItem(item) |
| 1988 | # An independent pipeline manager dynamically removes empty |
| 1989 | # queues |
| 1990 | if not item.queue.queue: |
| 1991 | self.pipeline.removeQueue(item.queue) |
James E. Blair | 5ee2425 | 2014-12-30 10:12:29 -0800 | [diff] [blame] | 1992 | |
James E. Blair | 1e8dd89 | 2012-05-30 09:15:05 -0700 | [diff] [blame] | 1993 | |
James E. Blair | 0577cd6 | 2015-02-07 11:42:12 -0800 | [diff] [blame] | 1994 | class StaticChangeQueueContextManager(object): |
| 1995 | def __init__(self, change_queue): |
| 1996 | self.change_queue = change_queue |
| 1997 | |
| 1998 | def __enter__(self): |
| 1999 | return self.change_queue |
| 2000 | |
| 2001 | def __exit__(self, etype, value, tb): |
| 2002 | pass |
| 2003 | |
| 2004 | |
James E. Blair | 4aea70c | 2012-07-26 14:23:24 -0700 | [diff] [blame] | 2005 | class DependentPipelineManager(BasePipelineManager): |
| 2006 | log = logging.getLogger("zuul.DependentPipelineManager") |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 2007 | changes_merge = True |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 2008 | |
| 2009 | def __init__(self, *args, **kwargs): |
James E. Blair | 4aea70c | 2012-07-26 14:23:24 -0700 | [diff] [blame] | 2010 | super(DependentPipelineManager, self).__init__(*args, **kwargs) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 2011 | |
James E. Blair | eff8816 | 2013-07-01 12:44:14 -0400 | [diff] [blame] | 2012 | def _postConfig(self, layout): |
| 2013 | super(DependentPipelineManager, self)._postConfig(layout) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 2014 | self.buildChangeQueues() |
| 2015 | |
| 2016 | def buildChangeQueues(self): |
| 2017 | self.log.debug("Building shared change queues") |
| 2018 | change_queues = [] |
| 2019 | |
James E. Blair | 4aea70c | 2012-07-26 14:23:24 -0700 | [diff] [blame] | 2020 | for project in self.pipeline.getProjects(): |
Clark Boylan | 7603a37 | 2014-01-21 11:43:20 -0800 | [diff] [blame] | 2021 | change_queue = ChangeQueue( |
| 2022 | self.pipeline, |
| 2023 | window=self.pipeline.window, |
| 2024 | window_floor=self.pipeline.window_floor, |
| 2025 | window_increase_type=self.pipeline.window_increase_type, |
| 2026 | window_increase_factor=self.pipeline.window_increase_factor, |
| 2027 | window_decrease_type=self.pipeline.window_decrease_type, |
| 2028 | window_decrease_factor=self.pipeline.window_decrease_factor) |
James E. Blair | 4aea70c | 2012-07-26 14:23:24 -0700 | [diff] [blame] | 2029 | change_queue.addProject(project) |
| 2030 | change_queues.append(change_queue) |
| 2031 | self.log.debug("Created queue: %s" % change_queue) |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 2032 | |
James E. Blair | c3d428e | 2013-12-03 15:06:48 -0800 | [diff] [blame] | 2033 | # Iterate over all queues trying to combine them, and keep doing |
| 2034 | # so until they can not be combined further. |
| 2035 | last_change_queues = change_queues |
| 2036 | while True: |
| 2037 | new_change_queues = self.combineChangeQueues(last_change_queues) |
| 2038 | if len(last_change_queues) == len(new_change_queues): |
| 2039 | break |
| 2040 | last_change_queues = new_change_queues |
| 2041 | |
| 2042 | self.log.info(" Shared change queues:") |
| 2043 | for queue in new_change_queues: |
| 2044 | self.pipeline.addQueue(queue) |
James E. Blair | c8a1e05 | 2014-02-25 09:29:26 -0800 | [diff] [blame] | 2045 | self.log.info(" %s containing %s" % ( |
| 2046 | queue, queue.generated_name)) |
James E. Blair | c3d428e | 2013-12-03 15:06:48 -0800 | [diff] [blame] | 2047 | |
| 2048 | def combineChangeQueues(self, change_queues): |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 2049 | self.log.debug("Combining shared queues") |
| 2050 | new_change_queues = [] |
| 2051 | for a in change_queues: |
| 2052 | merged_a = False |
| 2053 | for b in new_change_queues: |
| 2054 | if not a.getJobs().isdisjoint(b.getJobs()): |
| 2055 | self.log.debug("Merging queue %s into %s" % (a, b)) |
| 2056 | b.mergeChangeQueue(a) |
| 2057 | merged_a = True |
| 2058 | break # this breaks out of 'for b' and continues 'for a' |
| 2059 | if not merged_a: |
| 2060 | self.log.debug("Keeping queue %s" % (a)) |
| 2061 | new_change_queues.append(a) |
James E. Blair | c3d428e | 2013-12-03 15:06:48 -0800 | [diff] [blame] | 2062 | return new_change_queues |
James E. Blair | ee74361 | 2012-05-29 14:49:32 -0700 | [diff] [blame] | 2063 | |
James E. Blair | 0577cd6 | 2015-02-07 11:42:12 -0800 | [diff] [blame] | 2064 | def getChangeQueue(self, change, existing=None): |
| 2065 | if existing: |
| 2066 | return StaticChangeQueueContextManager(existing) |
| 2067 | return StaticChangeQueueContextManager( |
| 2068 | self.pipeline.getQueue(change.project)) |
James E. Blair | 5ee2425 | 2014-12-30 10:12:29 -0800 | [diff] [blame] | 2069 | |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 2070 | def isChangeReadyToBeEnqueued(self, change): |
James E. Blair | c0dedf8 | 2014-08-06 09:37:52 -0700 | [diff] [blame] | 2071 | if not self.pipeline.source.canMerge(change, |
| 2072 | self.getSubmitAllowNeeds()): |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 2073 | self.log.debug("Change %s can not merge, ignoring" % change) |
| 2074 | return False |
| 2075 | return True |
James E. Blair | 1e8dd89 | 2012-05-30 09:15:05 -0700 | [diff] [blame] | 2076 | |
James E. Blair | 5ee2425 | 2014-12-30 10:12:29 -0800 | [diff] [blame] | 2077 | def enqueueChangesBehind(self, change, quiet, ignore_requirements, |
| 2078 | change_queue): |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 2079 | to_enqueue = [] |
| 2080 | self.log.debug("Checking for changes needing %s:" % change) |
| 2081 | if not hasattr(change, 'needed_by_changes'): |
| 2082 | self.log.debug(" Changeish does not support dependencies") |
| 2083 | return |
James E. Blair | 5ee2425 | 2014-12-30 10:12:29 -0800 | [diff] [blame] | 2084 | for other_change in change.needed_by_changes: |
James E. Blair | 0577cd6 | 2015-02-07 11:42:12 -0800 | [diff] [blame] | 2085 | with self.getChangeQueue(other_change) as other_change_queue: |
| 2086 | if other_change_queue != change_queue: |
| 2087 | self.log.debug(" Change %s in project %s can not be " |
| 2088 | "enqueued in the target queue %s" % |
| 2089 | (other_change, other_change.project, |
| 2090 | change_queue)) |
| 2091 | continue |
James E. Blair | 5ee2425 | 2014-12-30 10:12:29 -0800 | [diff] [blame] | 2092 | if self.pipeline.source.canMerge(other_change, |
James E. Blair | c0dedf8 | 2014-08-06 09:37:52 -0700 | [diff] [blame] | 2093 | self.getSubmitAllowNeeds()): |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 2094 | self.log.debug(" Change %s needs %s and is ready to merge" % |
James E. Blair | 5ee2425 | 2014-12-30 10:12:29 -0800 | [diff] [blame] | 2095 | (other_change, change)) |
| 2096 | to_enqueue.append(other_change) |
| 2097 | |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 2098 | if not to_enqueue: |
| 2099 | self.log.debug(" No changes need %s" % change) |
| 2100 | |
| 2101 | for other_change in to_enqueue: |
James E. Blair | f9ab884 | 2014-07-10 13:12:07 -0700 | [diff] [blame] | 2102 | self.addChange(other_change, quiet=quiet, |
James E. Blair | 5ee2425 | 2014-12-30 10:12:29 -0800 | [diff] [blame] | 2103 | ignore_requirements=ignore_requirements, |
| 2104 | change_queue=change_queue) |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 2105 | |
James E. Blair | 5ee2425 | 2014-12-30 10:12:29 -0800 | [diff] [blame] | 2106 | def enqueueChangesAhead(self, change, quiet, ignore_requirements, |
| 2107 | change_queue): |
James E. Blair | dbfe1cd | 2015-02-07 11:41:19 -0800 | [diff] [blame] | 2108 | ret = self.checkForChangesNeededBy(change, change_queue) |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 2109 | if ret in [True, False]: |
| 2110 | return ret |
James E. Blair | 5ee2425 | 2014-12-30 10:12:29 -0800 | [diff] [blame] | 2111 | self.log.debug(" Changes %s must be merged ahead of %s" % |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 2112 | (ret, change)) |
James E. Blair | 6965a4b | 2014-12-16 17:19:04 -0800 | [diff] [blame] | 2113 | for needed_change in ret: |
| 2114 | r = self.addChange(needed_change, quiet=quiet, |
James E. Blair | 5ee2425 | 2014-12-30 10:12:29 -0800 | [diff] [blame] | 2115 | ignore_requirements=ignore_requirements, |
| 2116 | change_queue=change_queue) |
James E. Blair | 6965a4b | 2014-12-16 17:19:04 -0800 | [diff] [blame] | 2117 | if not r: |
| 2118 | return False |
| 2119 | return True |
James E. Blair | e048707 | 2012-08-29 17:38:31 -0700 | [diff] [blame] | 2120 | |
James E. Blair | dbfe1cd | 2015-02-07 11:41:19 -0800 | [diff] [blame] | 2121 | def checkForChangesNeededBy(self, change, change_queue): |
James E. Blair | e421a23 | 2012-07-25 16:59:21 -0700 | [diff] [blame] | 2122 | self.log.debug("Checking for changes needed by %s:" % change) |
| 2123 | # Return true if okay to proceed enqueing this change, |
| 2124 | # false if the change should not be enqueued. |
James E. Blair | 6965a4b | 2014-12-16 17:19:04 -0800 | [diff] [blame] | 2125 | if not hasattr(change, 'needs_changes'): |
James E. Blair | 4aea70c | 2012-07-26 14:23:24 -0700 | [diff] [blame] | 2126 | self.log.debug(" Changeish does not support dependencies") |
| 2127 | return True |
James E. Blair | 6965a4b | 2014-12-16 17:19:04 -0800 | [diff] [blame] | 2128 | if not change.needs_changes: |
James E. Blair | e421a23 | 2012-07-25 16:59:21 -0700 | [diff] [blame] | 2129 | self.log.debug(" No changes needed") |
| 2130 | return True |
James E. Blair | 6965a4b | 2014-12-16 17:19:04 -0800 | [diff] [blame] | 2131 | changes_needed = [] |
James E. Blair | 0577cd6 | 2015-02-07 11:42:12 -0800 | [diff] [blame] | 2132 | # Ignore supplied change_queue |
| 2133 | with self.getChangeQueue(change) as change_queue: |
| 2134 | for needed_change in change.needs_changes: |
| 2135 | self.log.debug(" Change %s needs change %s:" % ( |
| 2136 | change, needed_change)) |
| 2137 | if needed_change.is_merged: |
| 2138 | self.log.debug(" Needed change is merged") |
James E. Blair | 6965a4b | 2014-12-16 17:19:04 -0800 | [diff] [blame] | 2139 | continue |
James E. Blair | 0577cd6 | 2015-02-07 11:42:12 -0800 | [diff] [blame] | 2140 | with self.getChangeQueue(needed_change) as needed_change_queue: |
| 2141 | if needed_change_queue != change_queue: |
| 2142 | self.log.debug(" Change %s in project %s does not " |
| 2143 | "share a change queue with %s " |
| 2144 | "in project %s" % |
| 2145 | (needed_change, needed_change.project, |
| 2146 | change, change.project)) |
| 2147 | return False |
| 2148 | if not needed_change.is_current_patchset: |
| 2149 | self.log.debug(" Needed change is not the " |
| 2150 | "current patchset") |
| 2151 | return False |
| 2152 | if self.isChangeAlreadyInQueue(needed_change, change_queue): |
| 2153 | self.log.debug(" Needed change is already ahead " |
| 2154 | "in the queue") |
| 2155 | continue |
| 2156 | if self.pipeline.source.canMerge(needed_change, |
| 2157 | self.getSubmitAllowNeeds()): |
| 2158 | self.log.debug(" Change %s is needed" % needed_change) |
| 2159 | if needed_change not in changes_needed: |
| 2160 | changes_needed.append(needed_change) |
| 2161 | continue |
| 2162 | # The needed change can't be merged. |
| 2163 | self.log.debug(" Change %s is needed but can not be merged" % |
| 2164 | needed_change) |
| 2165 | return False |
James E. Blair | 6965a4b | 2014-12-16 17:19:04 -0800 | [diff] [blame] | 2166 | if changes_needed: |
| 2167 | return changes_needed |
| 2168 | return True |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 2169 | |
James E. Blair | 6965a4b | 2014-12-16 17:19:04 -0800 | [diff] [blame] | 2170 | def getFailingDependentItems(self, item): |
| 2171 | if not hasattr(item.change, 'needs_changes'): |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 2172 | return None |
James E. Blair | 6965a4b | 2014-12-16 17:19:04 -0800 | [diff] [blame] | 2173 | if not item.change.needs_changes: |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 2174 | return None |
James E. Blair | 6965a4b | 2014-12-16 17:19:04 -0800 | [diff] [blame] | 2175 | failing_items = set() |
| 2176 | for needed_change in item.change.needs_changes: |
| 2177 | needed_item = self.getItemForChange(needed_change) |
| 2178 | if not needed_item: |
| 2179 | continue |
| 2180 | if needed_item.current_build_set.failing_reasons: |
| 2181 | failing_items.add(needed_item) |
| 2182 | if failing_items: |
| 2183 | return failing_items |
James E. Blair | 972e3c7 | 2013-08-29 12:04:55 -0700 | [diff] [blame] | 2184 | return None |