home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
69.79k stars 28.93k forks source link

Target IP - Gree AC #107991

Open vantech14 opened 5 months ago

vantech14 commented 5 months ago

The problem

Unable to enter IP address of Gree AC. Integration tries to use some method of auto discovery which doesn’t support IPs on different subnets.

What version of Home Assistant Core has the issue?

core-2024.1.3

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

Free

Link to integration documentation on our website

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

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Nodered integration here supports IP entry for connection. HA integration should ideally have the same. 

https://flows.nodered.org/node/node-red-contrib-gree-hvac

Additional information

No response

home-assistant[bot] commented 5 months ago

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

Code owner commands Code owners of `gree` 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 gree` Removes the current integration label and assignees on the issue, add the integration domain after the command. - `@home-assistant add-label needs-more-information` Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue. - `@home-assistant remove-label needs-more-information` Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


gree documentation gree source (message by IssueLinks)

bartowl commented 4 months ago

+1 from me too, have gree ac on another subnet and autodiscovery does not work. There is a custom climate component for free in HACS, I’ll give it a try.. would however like to have at least yaml example for the official one.

chal086 commented 4 months ago

+1 same here, I removed the Devices and now the autoscan can't find them now, they are on a different subnet.

rapi3 commented 4 months ago

+1 Any idea maybe we can edit the IP from some config file after we move the AC to different subnet ? edit: I found custom integration that work properly with gree on another LAN segment and no need for AC to call home to report user actions.

viiliainen commented 3 months ago

+1 I have Docker based HA and it has it's own IP subnet that is bridged to my general ip space. Naturally my devices are not found because it searches HA subnet. Autoscan should be more clever would say. Or ask for IP.

v-rejected commented 3 months ago

+1

I have Docker based HA

2024-03-20 20:38:24.441 DEBUG (MainThread) [homeassistant.components.websocket_api.http.connection] [139718651448640] Vladas from 10.32.17.5 (Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/122.0.0.0 Safari/537.36): Sending b'{"id":47,"type":"result","success":true,"result":{"domain":"gree","name":"Gree Climate","codeowners":["@cmroche"],"config_flow":true,"dependencies":["network"],"documentation":"https://www.home-assistant.io/integrations/gree","iot_class":"local_polling","loggers":["greeclimate"],"requirements":["greeclimate==1.4.1"],"is_built_in":true}}' 2024-03-20 20:38:26.304 INFO (MainThread) [greeclimate.discovery] Scanning for Gree devices ... 2024-03-20 20:38:26.305 DEBUG (MainThread) [greeclimate.discovery] Listening for devices on 172.27.255.255 2024-03-20 20:38:26.306 DEBUG (MainThread) [greeclimate.network] Sending packet:

I see this is also covered here but was not resolved:

106043

95374

spacerokk commented 3 months ago

I also run HA on Docker and am running into the same problem.

v-rejected commented 3 months ago

I found this #100731 and tried it, it works

bartowl commented 3 months ago

I wonder why this PR is still in draft, it looks complete at first sight...

vantech14 commented 3 months ago

I found this #100731 and tried it, it works

How do you import this into home assistant over the official integration?

Thanks.

v-rejected commented 3 months ago

I found this #100731 and tried it, it works

How do you import this into home assistant over the official integration?

Thanks.

docker cp __init__.py homeassistant-test:/usr/src/homeassistant/homeassistant/components/gree/__init__.py docker cp config_flow.py homeassistant-test:/usr/src/homeassistant/homeassistant/components/gree/config_flow.py docker cp strings.json homeassistant-test:/usr/src/homeassistant/homeassistant/components/gree/strings.json restart HA

vantech14 commented 3 months ago

I found this #100731 and tried it, it works

How do you import this into home assistant over the official integration?

Thanks.

`

docker cp init.py homeassistant-test:/usr/src/homeassistant/homeassistant/components/gree/init.py

docker cp config_flow.py homeassistant-test:/usr/src/homeassistant/homeassistant/components/gree/config_flow.py

docker cp strings.json homeassistant-test:/usr/src/homeassistant/homeassistant/components/gree/strings.json

`

restart HA

Thanks but I'm using HAOS. In my case the ACs are in a different location and connected via a VPN. I have a HACS integration working so the connectivity is fine but want to use the office integration.

spacerokk commented 3 months ago

I found this #100731 and tried it, it works

How do you import this into home assistant over the official integration? Thanks.

docker cp __init__.py homeassistant-test:/usr/src/homeassistant/homeassistant/components/gree/__init__.py docker cp config_flow.py homeassistant-test:/usr/src/homeassistant/homeassistant/components/gree/config_flow.py docker cp strings.json homeassistant-test:/usr/src/homeassistant/homeassistant/components/gree/strings.json restart HA

Thanks! Does this have to be done every time you update the HA docker version? Wish there was an easier way.

viiliainen commented 3 months ago

Worked 2/3 of my devices. Same modell all of them and controllable through cloud. Need to study that one device why it won't connect. My docker won't recognize those commands but copying files manually in Bash worked also OK.

issue-triage-workflows[bot] commented 2 weeks ago

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.

rapi3 commented 2 weeks ago

I ported original HA Gree Climate integration with patch from kspearrin to HA - OS as custom integration, have a look: https://github.com/rapi3/HA-OS-Gree

can be installed from HACS - custom repository - integration