zuul/nodepool
Code Issues Proposed changes

Merge "Change TaskManager from is-a to has-a thread"

Browse Source
This commit is contained in:
Zuul 2018-07-13 14:32:25 +00:00 committed by Gerrit Code Review
commit b11516ae24
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
nodepool/task_manager.py
View File

@ -62,11 +62,11 @@ class Task(object):
self.exception(e, sys.exc_info()[2])
class TaskManager(threading.Thread):
class TaskManager(object):
log = logging.getLogger("nodepool.TaskManager")
def __init__(self, client, name, rate):
super(TaskManager, self).__init__(name=name)
super(TaskManager, self).__init__()
self.daemon = True
self.queue = queue.Queue()
self._running = True
@ -74,14 +74,22 @@ class TaskManager(threading.Thread):
self.rate = float(rate)
self._client = None
self.statsd = stats.get_client()
self._thread = threading.Thread(name=name, target=self.run)
self._thread.daemon = True
def setClient(self, client):
self._client = client
def start(self):
self._thread.start()
def stop(self):
self._running = False
self.queue.put(None)
def join(self):
self._thread.join()
def run(self):
last_ts = 0
try: