jeans/core
jeans/core
1
0
Fork 0
mirror of https://github.com/home-assistant/core.git synced 2025-07-08 22:07:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

More graphite hardening work

Browse Source 
This adds verbose debugging which can be turned on to figure out what is
going on. It also adds a broad exception handler in the worker thread
to avoid dying. If you're running this such that stderr doesn't go to a
log, it can be easy to miss the thread's death.

I wrote all this to try to diagnose #1283, which seems to maybe have
healed itself. But since I have it, I figure we might as well keep it
in case we have trouble in the future.
This commit is contained in:
Dan Smith 2016-02-17 07:45:00 -08:00
parent bb37708716
commit cd6780baf4
1 changed files with 27 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
homeassistant/components/graphite.py
View File

@ -49,15 +49,20 @@ class GraphiteFeeder(threading.Thread):
self._prefix = prefix.rstrip('.') self._prefix = prefix.rstrip('.')
self._queue = queue.Queue() self._queue = queue.Queue()
self._quit_object = object() self._quit_object = object()
self._we_started = False
hass.bus.listen_once(EVENT_HOMEASSISTANT_START, hass.bus.listen_once(EVENT_HOMEASSISTANT_START,
self.start_listen) self.start_listen)
hass.bus.listen_once(EVENT_HOMEASSISTANT_STOP, hass.bus.listen_once(EVENT_HOMEASSISTANT_STOP,
self.shutdown) self.shutdown)
hass.bus.listen(EVENT_STATE_CHANGED, self.event_listener) hass.bus.listen(EVENT_STATE_CHANGED, self.event_listener)
_LOGGER.debug('Graphite feeding to %s:%i initialized',
self._host, self._port)
def start_listen(self, event): def start_listen(self, event):
""" Start event-processing thread. """ """ Start event-processing thread. """
_LOGGER.debug('Event processing thread started')
self._we_started = True
self.start() self.start()
def shutdown(self, event): def shutdown(self, event):
@ -67,11 +72,17 @@ class GraphiteFeeder(threading.Thread):
clean up (and no penalty for killing in-process clean up (and no penalty for killing in-process
connections to graphite. connections to graphite.
""" """
_LOGGER.debug('Event processing signaled exit')
self._queue.put(self._quit_object) self._queue.put(self._quit_object)
def event_listener(self, event): def event_listener(self, event):
""" Queue an event for processing. """ """ Queue an event for processing. """
if self.is_alive() or not self._we_started:
_LOGGER.debug('Received event')
self._queue.put(event) self._queue.put(event)
else:
_LOGGER.error('Graphite feeder thread has died, not '
'queuing event!')
def _send_to_graphite(self, data): def _send_to_graphite(self, data):
sock = socket.socket(socket.AF_INET, socket.SOCK_STREAM) sock = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
@ -107,10 +118,23 @@ class GraphiteFeeder(threading.Thread):
while True: while True:
event = self._queue.get() event = self._queue.get()
if event == self._quit_object: if event == self._quit_object:
_LOGGER.debug('Event processing thread stopped')
self._queue.task_done() self._queue.task_done()
return return
elif (event.event_type == EVENT_STATE_CHANGED and elif (event.event_type == EVENT_STATE_CHANGED and
event.data.get('new_state')): event.data.get('new_state')):
_LOGGER.debug('Processing STATE_CHANGED event for %s',
event.data['entity_id'])
try:
self._report_attributes(event.data['entity_id'], self._report_attributes(event.data['entity_id'],
event.data['new_state']) event.data['new_state'])
# pylint: disable=broad-except
except Exception:
# Catch this so we can avoid the thread dying and
# make it visible.
_LOGGER.exception('Failed to process STATE_CHANGED event')
else:
_LOGGER.warning('Processing unexpected event type %s',
event.event_type)
self._queue.task_done() self._queue.task_done()