diff options
| author | Zuul <zuul@review.opendev.org> | 2020-05-16 15:27:24 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2020-05-16 15:27:24 +0000 |
| commit | 41225744e67c8106a21a709a622fda4df5cce93f (patch) | |
| tree | e95566243f940c18b60a8ad56d0c6aa58ae7eab5 /tests/base.py | |
| parent | ea1d3b7d72268d906ed981d5421ee2b218332aab (diff) | |
| parent | d6beb77397f596dbd333975d31b6f3af02ea8f6d (diff) | |
| download | zuul-41225744e67c8106a21a709a622fda4df5cce93f.tar.gz | |
Merge "Enhance some logging messages"
Diffstat (limited to 'tests/base.py')
| -rw-r--r-- | tests/base.py | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/tests/base.py b/tests/base.py index ba5e68342..6d369133d 100644 --- a/tests/base.py +++ b/tests/base.py @@ -2685,8 +2685,11 @@ class RecordingExecutorServer(zuul.executor.server.ExecutorServer): """ builds = self.running_builds[:] - self.log.debug("Releasing build %s (%s)" % (regex, - len(self.running_builds))) + if len(builds) == 0: + self.log.debug('No running builds to release') + return + + self.log.debug("Releasing build %s (%s)" % (regex, len(builds))) for build in builds: if not regex or re.match(regex, build.name): self.log.debug("Releasing build %s" % @@ -2696,7 +2699,7 @@ class RecordingExecutorServer(zuul.executor.server.ExecutorServer): self.log.debug("Not releasing build %s" % (build.parameters['zuul']['build'])) self.log.debug("Done releasing builds %s (%s)" % - (regex, len(self.running_builds))) + (regex, len(builds))) def executeJob(self, job): build = FakeBuild(self, job) @@ -4687,6 +4690,10 @@ class ZuulTestCase(BaseTestCase): completed. """ + if not self.history: + self.log.debug("Build history: no builds ran") + return + self.log.debug("Build history:") for build in self.history: self.log.debug(build) |