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

Make the Qnap sensor more resilient if server is not reachable (#16445)

Browse Source 
* add CONF_ALLOW_UNREACHABLE option, retry connection if allowed

* fix linting errors

* Removing the config option, just using PlatformNotReady
This commit is contained in:
mrosseel 2018-09-10 16:19:17 +02:00 committed by Fabian Affolter
parent e96635b5c1
commit f858938ada
1 changed files with 3 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
homeassistant/components/sensor/qnap.py
View File

@ -15,6 +15,7 @@ from homeassistant.const import (
CONF_HOST, CONF_USERNAME, CONF_PASSWORD, CONF_PORT, CONF_SSL, ATTR_NAME, CONF_HOST, CONF_USERNAME, CONF_PASSWORD, CONF_PORT, CONF_SSL, ATTR_NAME,
CONF_VERIFY_SSL, CONF_TIMEOUT, CONF_MONITORED_CONDITIONS, TEMP_CELSIUS) CONF_VERIFY_SSL, CONF_TIMEOUT, CONF_MONITORED_CONDITIONS, TEMP_CELSIUS)
from homeassistant.util import Throttle from homeassistant.util import Throttle
from homeassistant.exceptions import PlatformNotReady
import homeassistant.helpers.config_validation as cv import homeassistant.helpers.config_validation as cv
REQUIREMENTS = ['qnapstats==0.2.7'] REQUIREMENTS = ['qnapstats==0.2.7']
@ -107,14 +108,9 @@ def setup_platform(hass, config, add_entities, discovery_info=None):
api = QNAPStatsAPI(config) api = QNAPStatsAPI(config)
api.update() api.update()
# QNAP is not available
if not api.data: if not api.data:
hass.components.persistent_notification.create( raise PlatformNotReady
'Error: Failed to set up QNAP sensor.<br />'
'Check the logs for additional information. '
'You will need to restart hass after fixing.',
title=NOTIFICATION_TITLE,
notification_id=NOTIFICATION_ID)
return False
sensors = [] sensors = []