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 Hydrawise integration - various errors #97317

Closed
Crashman1983 opened this issue Jul 27, 2023 · 3 comments · Fixed by #97333
Closed

New Hydrawise integration - various errors #97317

Crashman1983 opened this issue Jul 27, 2023 · 3 comments · Fixed by #97333

Comments

@Crashman1983
Copy link

The problem

Installed 2023.8.0b0 and the renewed Hydrawise integration isn't working.

  • all sensors showing "unknown"
  • no switch working

Various errors in log file, see attachment.

What version of Home Assistant Core has the issue?

core-2023.8.0b0

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

Hunter Hydrawise

Link to integration documentation on our website

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

Diagnostics information

home-assistant_2023-07-27T06-17-20.429Z.log

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@home-assistant
Copy link

Hey there @dknowles2, @ptcryan, mind taking a look at this issue as it has been labeled with an integration (hydrawise) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of hydrawise can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign hydrawise Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


hydrawise documentation
hydrawise source
(message by IssueLinks)

@frenck frenck changed the title 2023.8.0b0: new Hydrawise integration - various errors New Hydrawise integration - various errors Jul 27, 2023
@frenck frenck added this to the 2023.8.0 milestone Jul 27, 2023
@dknowles2
Copy link
Contributor

#97333 should fix this.

I also sent #97334 to pick up dknowles2/pydrawise#54 in the pydrawise library, which was causing all entities to show as unavailable.

@Crashman1983
Copy link
Author

Crashman1983 commented Jul 28, 2023

confirmed working! Thx a lot!

I would suggest to reduce API calls if possible, especially at startup:

Logger: homeassistant.components.hydrawise
Source: helpers/update_coordinator.py:262
Integration: Hunter Hydrawise (documentation, issues)
First occurred: 06:43:12 (1 occurrences)
Last logged: 06:43:12

Error requesting hydrawise data: 429 Client Error: Too Many Requests for url: https://api.hydrawise.com/api/v1/customerdetails.php?api_key=5372-B3BE-35AC-F6C8&type=controllers

@github-actions github-actions bot locked and limited conversation to collaborators Aug 27, 2023
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