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

Hayward Omnilogic integration does not work #54781

Closed
fulviofreitas opened this issue Aug 17, 2021 · 14 comments
Closed

Hayward Omnilogic integration does not work #54781

fulviofreitas opened this issue Aug 17, 2021 · 14 comments

Comments

@fulviofreitas
Copy link

The problem

After the add integration steps (username/password), the Omnilogic integration does not load/work.

What is version of Home Assistant Core has the issue?

core-2021.8.7

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Hayward Omnilogic

Link to integration documentation on our website

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

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Logger: homeassistant.config_entries
Source: components/omnilogic/__init__.py:41
First occurred: 2:04:59 PM (1 occurrences)
Last logged: 2:04:59 PM

Error setting up entry Omnilogic for omnilogic
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 293, in async_setup
    result = await component.async_setup_entry(hass, self)  # type: ignore
  File "/usr/src/homeassistant/homeassistant/components/omnilogic/__init__.py", line 41, in async_setup_entry
    await api.get_telemetry_data()
  File "/usr/local/lib/python3.9/site-packages/omnilogic/__init__.py", line 1008, in get_telemetry_data
    site_telem = self.telemetry_to_json(telem, config_item, self.alarms_to_json(this_alarm))
  File "/usr/local/lib/python3.9/site-packages/omnilogic/__init__.py", line 846, in telemetry_to_json
    if type(bow_item["Chlorinator"]["Operation"]) == dict:
KeyError: 'Operation'

Additional information

No response

@probot-home-assistant
Copy link

omnilogic documentation
omnilogic source
(message by IssueLinks)

@probot-home-assistant
Copy link

Hey there @Oliver84, @djtimca, @gentoosu, mind taking a look at this issue as it has been labeled with an integration (omnilogic) you are listed as a code owner for? Thanks!
(message by CodeOwnersMention)

@djtimca
Copy link
Contributor

djtimca commented Sep 7, 2021

Sorry for the delay @fulviofreitas. Are you still having this issue, and if so can we jump on an external communication to get some additional data from your system to see what is different?

@fulviofreitas
Copy link
Author

Hey @djtimca, I'm still having this issue...

@djtimca
Copy link
Contributor

djtimca commented Sep 7, 2021

In order for us to troubleshoot I will need to get a look at the telemetry data that is returned from your pool setup. We have an online Python environment where you can do that: https://replit.com/@djtimca/OmniTesting#main.py

If you hit Run at the top it will ask you to log in with your Omnilogic credentials and will save a file out just with the telemetry from your pool. It does not save or store any of your personal information / passwords etc.

Thanks!

@fulviofreitas
Copy link
Author

@djtimca , I just hit the run and added my credentials. Below I received this output:

Unclosed client session
client_session: <aiohttp.client.ClientSession object at 0x7f01ef149640>
Unclosed connector
connections: ['[(<aiohttp.client_proto.ResponseHandler object at 0x7f01ef1588e0>, 14275.027281255)]']
connector: <aiohttp.connector.TCPConnector object at 0x7f01ef149670>
repl process died unexpectedly: 

@fulviofreitas
Copy link
Author

@djtimca Do you have any guess what can be this issue?

@djtimca
Copy link
Contributor

djtimca commented Oct 24, 2021

Oh shoot - sorry @fulviofreitas I didn't see your response until now. Unfortunately it looks like it didn't save any data from your system. Can you try again? I just tested and it is returning telemetry from my system. Unfortunately the error above isn't helpful either so I'm not sure what happened.

@fulviofreitas
Copy link
Author

fulviofreitas commented Oct 24, 2021

@djtimca ,
That was the result of the test, after add my credentials (user/pass):

Username: fufreitas@<>.com
Password: <SECRET-DATA>
An error occurred: 'Operation'
Press enter to clear.
Unclosed client session
client_session: <aiohttp.client.ClientSession object at 0x7f9baad14670>
Unclosed connector
connections: ['[(<aiohttp.client_proto.ResponseHandler object at 0x7f9baad265e0>, 48036.318899851)]']
connector: <aiohttp.connector.TCPConnector object at 0x7f9baad146a0>
repl process died unexpectedly: 

@djtimca
Copy link
Contributor

djtimca commented Oct 24, 2021

I'm not sure - based on that result your pool isn't even returning even basic telemetry from the API. I can't replicate that issue with any of the Omnilogic credentials that I have.

To confirm, this is a Hayward Omnilogic system, not a OmniHub, OmniPL, or ProLogic system correct? (This integration will only work with the OmniLogic system).

@fulviofreitas
Copy link
Author

That is really weird. The system that I have it's the Omnilogic System

image

@github-actions
Copy link

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Jan 22, 2022
@fulviofreitas
Copy link
Author

@djtimca Any idea what it can be?
Looks like an error when the client tries to parse the XML file over the Operation block

@github-actions github-actions bot removed the stale label Jan 27, 2022
@github-actions
Copy link

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Apr 27, 2022
@github-actions github-actions bot closed this as completed May 4, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Jun 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants