Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Component will not work after updating to Home Assistant 2021.4 #11

Closed
ahattest opened this issue Apr 8, 2021 · 5 comments
Closed

Component will not work after updating to Home Assistant 2021.4 #11

ahattest opened this issue Apr 8, 2021 · 5 comments

Comments

@ahattest
Copy link

ahattest commented Apr 8, 2021

Error message from Home Assistant

Detected I/O inside the event loop. This is causing stability issues. Please report issue to the custom component author for min_renovasjon doing I/O at custom_components/min_renovasjon/init.py, line 83: response = requests.get(url, headers=header)

@kimrunan
Copy link

kimrunan commented Apr 9, 2021

more info from the error on my setup;
Logger: homeassistant.setup
Source: util/async_.py:139
First occurred: 10:59:00 (1 occurrences)
Last logged: 10:59:00

Error during setup of component min_renovasjon
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/setup.py", line 240, in _async_setup_component
result = await task
File "/config/custom_components/min_renovasjon/init.py", line 48, in async_setup
min_renovasjon = MinRenovasjon(street_name, street_code, house_no, county_id, date_format)
File "/config/custom_components/min_renovasjon/init.py", line 61, in init
self._kalender_list = self._get_calendar_list()
File "/config/custom_components/min_renovasjon/init.py", line 117, in _get_calendar_list
tommekalender, fraksjoner = self._get_from_web_api()
File "/config/custom_components/min_renovasjon/init.py", line 104, in _get_from_web_api
tommekalender = self._get_tommekalender_from_web_api()
File "/config/custom_components/min_renovasjon/init.py", line 83, in _get_tommekalender_from_web_api
response = requests.get(url, headers=header)
File "/usr/local/lib/python3.8/site-packages/requests/api.py", line 76, in get
return request('get', url, params=params, **kwargs)
File "/usr/local/lib/python3.8/site-packages/requests/api.py", line 61, in request
return session.request(method=method, url=url, **kwargs)
File "/usr/local/lib/python3.8/site-packages/requests/sessions.py", line 542, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python3.8/site-packages/requests/sessions.py", line 655, in send
r = adapter.send(request, **kwargs)
File "/usr/local/lib/python3.8/site-packages/requests/adapters.py", line 439, in send
resp = conn.urlopen(
File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 699, in urlopen
httplib_response = self._make_request(
File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 394, in _make_request
conn.request(method, url, **httplib_request_kw)
File "/usr/local/lib/python3.8/site-packages/urllib3/connection.py", line 234, in request
super(HTTPConnection, self).request(method, url, body=body, headers=headers)
File "/usr/local/lib/python3.8/http/client.py", line 1255, in request
self._send_request(method, url, body, headers, encode_chunked)
File "/usr/local/lib/python3.8/http/client.py", line 1266, in send_request
self.putrequest(method, url, **skips)
File "/usr/local/lib/python3.8/site-packages/urllib3/connection.py", line 214, in putrequest
return HTTPConnection.putrequest(self, method, url, *args, **kwargs)
File "/usr/src/homeassistant/homeassistant/util/async
.py", line 150, in protected_loop_func
check_loop()
File "/usr/src/homeassistant/homeassistant/util/async
.py", line 139, in check_loop
raise RuntimeError(
RuntimeError: I/O must be done in the executor; Use await hass.async_add_executor_job() at custom_components/min_renovasjon/init.py, line 83: response = requests.get(url, headers=header)

@ahattest
Copy link
Author

ahattest commented Apr 9, 2021

Tried danielhiversen’s pull request, worked flawlessly, replace line 40 with:
def setup(hass, config):

@Danielhiversen
Copy link

You can also try https://github.com/Danielhiversen/home_assistant_min_renovasjon

@Mengine
Copy link

Mengine commented Apr 30, 2021

fix works here also :)

@eyesoft
Copy link
Owner

eyesoft commented Jan 25, 2022

Fixed

@eyesoft eyesoft closed this as completed Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants