diff --git a/tests/test_scheduler.py b/tests/test_scheduler.py
index f698d590df..a953ffb008 100755
--- a/tests/test_scheduler.py
+++ b/tests/test_scheduler.py
@@ -635,7 +635,7 @@ class TestScheduler(ZuulTestCase):
         self.fake_gerrit.addEvent(B.addApproval('APRV', 1))
         self.waitUntilSettled()
 
-        self.log.debug("len %s " % self.gerrit._change_cache.keys())
+        self.log.debug("len %s" % self.gerrit._change_cache.keys())
         # there should still be changes in the cache
         self.assertNotEqual(len(self.gerrit._change_cache.keys()), 0)
 
diff --git a/zuul/launcher/gearman.py b/zuul/launcher/gearman.py
index b0d8546bbd..ada9c707d4 100644
--- a/zuul/launcher/gearman.py
+++ b/zuul/launcher/gearman.py
@@ -436,7 +436,7 @@ class Gearman(object):
 
     def onWorkStatus(self, job):
         data = getJobData(job)
-        self.log.debug("Build %s update %s " % (job, data))
+        self.log.debug("Build %s update %s" % (job, data))
         build = self.builds.get(job.unique)
         if build:
             # Allow URL to be updated
diff --git a/zuul/merger/merger.py b/zuul/merger/merger.py
index f150771fdc..9e1f4c1784 100644
--- a/zuul/merger/merger.py
+++ b/zuul/merger/merger.py
@@ -132,14 +132,14 @@ class Repo(object):
 
     def createZuulRef(self, ref, commit='HEAD'):
         repo = self.createRepoObject()
-        self.log.debug("CreateZuulRef %s at %s " % (ref, commit))
+        self.log.debug("CreateZuulRef %s at %s" % (ref, commit))
         ref = ZuulReference.create(repo, ref, commit)
         return ref.commit
 
     def push(self, local, remote):
         repo = self.createRepoObject()
-        self.log.debug("Pushing %s:%s to %s " % (local, remote,
-                                                 self.remote_url))
+        self.log.debug("Pushing %s:%s to %s" % (local, remote,
+                                                self.remote_url))
         repo.remotes.origin.push('%s:%s' % (local, remote))
 
     def update(self):