From fec1a8be922a10b05194f3b4f28bae84f420d376 Mon Sep 17 00:00:00 2001
From: mprahl <mprahl@redhat.com>
Date: Dec 18 2018 22:17:44 +0000
Subject: [PATCH 1/2] Fix a log statement in the poller


---

diff --git a/module_build_service/scheduler/producer.py b/module_build_service/scheduler/producer.py
index 62f06bb..3a9c8bf 100644
--- a/module_build_service/scheduler/producer.py
+++ b/module_build_service/scheduler/producer.py
@@ -202,8 +202,7 @@ class MBSProducer(PollingProducer):
         """
         log.info('Looking for module builds stuck in the %s state', state)
         builds = models.ModuleBuild.by_state(session, state)
-        log.info(' {0!r} module builds in the %s state...'
-                 .format(len(builds), state))
+        log.info(' %r module builds in the %s state...', len(builds), state)
         now = datetime.utcnow()
         time_modified_threshold = timedelta(minutes=older_than_minutes)
         for build in builds:

From e237676c7e5ea62803980980e7a7d9d136ee28ac Mon Sep 17 00:00:00 2001
From: mprahl <mprahl@redhat.com>
Date: Dec 19 2018 14:06:59 +0000
Subject: [PATCH 2/2] Don't overwrite the state parameter in _nudge_module_builds_in_state


---

diff --git a/module_build_service/scheduler/producer.py b/module_build_service/scheduler/producer.py
index 3a9c8bf..0aa68bf 100644
--- a/module_build_service/scheduler/producer.py
+++ b/module_build_service/scheduler/producer.py
@@ -194,15 +194,15 @@ class MBSProducer(PollingProducer):
                         log.info('      * {0} components in batch {1}'
                                  .format(n, i))
 
-    def _nudge_module_builds_in_state(self, session, state, older_than_minutes):
+    def _nudge_module_builds_in_state(self, session, state_name, older_than_minutes):
         """
         Finds all the module builds in the `state` with `time_modified` older
         than `older_than_minutes` and adds fake MBSModule message to the
         work queue.
         """
-        log.info('Looking for module builds stuck in the %s state', state)
-        builds = models.ModuleBuild.by_state(session, state)
-        log.info(' %r module builds in the %s state...', len(builds), state)
+        log.info('Looking for module builds stuck in the %s state', state_name)
+        builds = models.ModuleBuild.by_state(session, state_name)
+        log.info(' %r module builds in the %s state...', len(builds), state_name)
         now = datetime.utcnow()
         time_modified_threshold = timedelta(minutes=older_than_minutes)
         for build in builds:
@@ -216,7 +216,7 @@ class MBSProducer(PollingProducer):
             session.commit()
 
             # Fake a message to kickstart the build anew in the consumer
-            state = module_build_service.models.BUILD_STATES[state]
+            state = module_build_service.models.BUILD_STATES[state_name]
             msg = module_build_service.messaging.MBSModule(
                 'nudge_module_builds_fake_message', build.id, state)
             log.info("  Scheduling faked event %r" % msg)