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

new version breaks satel_integra integration #74528

Closed
pinione opened this issue Jul 6, 2022 · 4 comments · Fixed by #74543
Closed

new version breaks satel_integra integration #74528

pinione opened this issue Jul 6, 2022 · 4 comments · Fixed by #74543

Comments

@pinione
Copy link

pinione commented Jul 6, 2022

The problem

After upgrading to the lastest version notification is shown:

Invalid config
The following integrations and platforms could not be set up:

satel_integra (Show logs)
Please check your config and logs.

Logs shows:

2022-07-06 21:43:15 WARNING (MainThread) [satel_integra.satel_integra] Exception during connecting: BaseEventLoop.create_connection() got an unexpected keyword argument 'loop'.
2022-07-06 21:43:15 ERROR (MainThread) [homeassistant.setup] Setup failed for satel_integra: Integration failed to initialize.

satel_integra integration is not loaded.

What version of Home Assistant Core has the issue?

2022.7.0

What was the last working version of Home Assistant Core?

2022.6.7

What type of installation are you running?

Home Assistant OS

Integration causing the issue

satel_integra

Link to integration documentation on our website

https://www.home-assistant.io/integrations/satel_integra/

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

2022-07-06 21:43:15 WARNING (MainThread) [satel_integra.satel_integra] Exception during connecting: BaseEventLoop.create_connection() got an unexpected keyword argument 'loop'.
2022-07-06 21:43:15 ERROR (MainThread) [homeassistant.setup] Setup failed for satel_integra: Integration failed to initialize.

Additional information

No response

@probot-home-assistant
Copy link

satel_integra documentation
satel_integra source
(message by IssueLinks)

@Rijswijker
Copy link

It is already fixed so far i can see but it is not in the 2022.7 release :-(
I tried to copy the code to the custom_components folder, but didn’t get it work.

@xury77
Copy link

xury77 commented Jul 6, 2022

If you use docker you can do as following:
in bash ssh type:
docker ps
Get te container ID and next:
docker exec -it d3f758f20d2f /bin/bash
Don't forget change the container ID (d3f758f20d2f ) for your previously displayed by docker ps
And finally after login to container:
vi /usr/local/lib/python3.10/site-packages/satel_integra/satel_integra.py
and modify line nr 176 from: self._host, self._port, loop=self._loop)
to self._host, self._port)

You need some skills with vi editor

[edited]
You dont have to know Container ID just type
docker exec -it homeassistant /bin/bash

@pinione
Copy link
Author

pinione commented Jul 7, 2022

If you use docker you can do as following: in bash ssh type: docker ps Get te container ID and next: docker exec -it d3f758f20d2f /bin/bash Don't forget change the container ID (d3f758f20d2f ) for your previously displayed by docker ps And finally after login to container: vi /usr/local/lib/python3.10/site-packages/satel_integra/satel_integra.py and modify line nr 176 from: self._host, self._port, loop=self._loop) to self._host, self._port)

You need some skills with vi editor

[edited] You dont have to know Container ID just type docker exec -it homeassistant /bin/bash

Thank you that fixed the problem, well till next reboot at least.

@github-actions github-actions bot locked and limited conversation to collaborators Aug 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants