diff options
author | Antoine Musso <hashar@free.fr> | 2020-01-25 12:28:20 +0100 |
---|---|---|
committer | Antoine Musso <hashar@free.fr> | 2020-04-02 10:45:35 +0200 |
commit | d6beb77397f596dbd333975d31b6f3af02ea8f6d (patch) | |
tree | e23e2d1ce5ec465446c51984543f88f75f720a74 /tests/base.py | |
parent | b9f885e2a7e3fd4c44c53d22815d270ebeb41431 (diff) | |
download | zuul-d6beb77397f596dbd333975d31b6f3af02ea8f6d.tar.gz |
Enhance some logging messages
Change-Id: I257a3592643a716cd88efc255150a603d89d108a
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 aa43ab2e7..527bdf66c 100644 --- a/tests/base.py +++ b/tests/base.py @@ -2659,8 +2659,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" % @@ -2670,7 +2673,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) @@ -4599,6 +4602,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) |