vbatts/quay
Archived
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

buildreporter: does not execute in a coroutine!

Browse source
This commit is contained in:
Jimmy Zelinskie 2015-02-18 17:11:45 -05:00
parent 0d38e0b00b
commit 9ab3554226
2 changed files with 6 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
buildman/jobutil/buildreporter.py
View file

@ -11,7 +11,6 @@ class BuildReporter(object):
def report_completion_status(self, status):
"""
Method to invoke the recording of build's completion status to a metric service.
This must _not_ block because it is assumed to run in an event loop.
"""
raise NotImplementedError
@ -35,6 +34,9 @@ class CloudWatchBuildReporter(BuildReporter):
self._failed_name = failed_name
self._incompleted_name = incompleted_name
def _send_to_queue(self, *args, **kwargs):
self._queue.put((args, kwargs))
def report_completion_status(self, status):
if status == BuildJobResult.COMPLETE:
status_name = self._completed_name
@ -45,7 +47,7 @@ class CloudWatchBuildReporter(BuildReporter):
else:
return
yield From(self._queue.put(self._namespace_name, status_name, 1, unit='Count'))
self._send_to_queue(self._namespace_name, status_name, 1, unit='Count')
class BuildMetrics(object):

7
util/cloudwatch.py
View file

@ -1,6 +1,5 @@
import logging
import boto
import thread
from Queue import Queue
from threading import Thread
@ -12,10 +11,8 @@ def get_queue(app):
"""
Returns a queue to the CloudWatchSender. If a queue/sender do not exist, creates them.
"""
access_key = app.config.get('CLOUDWATCH_AWS_ACCESS_KEY')
secret_key = app.config.get('CLOUDWATCH_AWS_SECRET_KEY')
if None in (access_key, secret_key):
raise TypeError
access_key = app.config['CLOUDWATCH_AWS_ACCESS_KEY']
secret_key = app.config['CLOUDWATCH_AWS_SECRET_KEY']
queue = Queue()
sender = CloudWatchSender(queue, access_key, secret_key)