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

Daikin AC are not working after update to 2024.8.1 #123550

Closed
VinceRMP opened this issue Aug 10, 2024 · 75 comments · Fixed by #123623
Closed

Daikin AC are not working after update to 2024.8.1 #123550

VinceRMP opened this issue Aug 10, 2024 · 75 comments · Fixed by #123623

Comments

@VinceRMP
Copy link

The problem

After the update to 2024.8.1 all my AC's are unavailable.

This is reported in the logs:

2024-08-10 20:34:29.753 ERROR (MainThread) [pydaikin.daikin_base] Exception in TaskGroup: 'DaikinBRP069' object has no attribute 'headers' 2024-08-10 20:34:29.754 ERROR (MainThread) [homeassistant.components.daikin] Unexpected error creating device 10.10.3.6 2024-08-10 20:34:29.754 ERROR (MainThread) [pydaikin.daikin_base] Exception in TaskGroup: 'DaikinBRP069' object has no attribute 'headers' 2024-08-10 20:34:29.754 ERROR (MainThread) [homeassistant.components.daikin] Unexpected error creating device 10.10.3.5 2024-08-10 20:34:29.754 ERROR (MainThread) [pydaikin.daikin_base] Exception in TaskGroup: 'DaikinBRP069' object has no attribute 'headers' 2024-08-10 20:34:29.754 ERROR (MainThread) [homeassistant.components.daikin] Unexpected error creating device 10.10.3.7 2024-08-10 20:34:29.754 ERROR (MainThread) [pydaikin.daikin_base] Exception in TaskGroup: 'DaikinBRP069' object has no attribute 'headers' 2024-08-10 20:34:29.754 ERROR (MainThread) [homeassistant.components.daikin] Unexpected error creating device 10.10.3.8

What version of Home Assistant Core has the issue?

core-2024.8.1

What was the last working version of Home Assistant Core?

core-2024.8.0

What type of installation are you running?

Home Assistant OS

Integration causing the issue

daikin

Link to integration documentation on our website

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

Diagnostics information

No response

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 @fredrike, mind taking a look at this issue as it has been labeled with an integration (daikin) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of daikin 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 daikin 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)


daikin documentation
daikin source
(message by IssueLinks)

@stefanoferrario
Copy link

Same issue here

@albertoarmida
Copy link

Same issue here.
Can I help in some way? Do you need logs or anything else?

@Jonssii
Copy link

Jonssii commented Aug 10, 2024

Same here.

"Exception in TaskGroup: 'DaikinBRP069' object has no attribute 'headers'"

@VinceRMP
Copy link
Author

VinceRMP commented Aug 10, 2024

The commit that was done earlier to fix an issue with the newer wifi modules added some code regarding the headers. This probably caused the issue with my (our?) older wifi modules.

#123519

@mike4001
Copy link

same here

this is quiet bad ...

@berglez
Copy link

berglez commented Aug 10, 2024

Same here

Logger: pydaikin.daikin_base
Source: components/daikin/init.py:88
First occurred: 21:46:22 (3 occurrences)
Last logged: 21:50:08

Exception in TaskGroup: 'DaikinBRP069' object has no attribute 'headers'

@rogervdh
Copy link

I have the same issue. I'm using a Daikin BRP069B4x unit.

@Roger954
Copy link

Roger954 commented Aug 10, 2024

Same issue here. Deleted both airco devices from HA and after restart the are auto-discovered. However, I cannot get passed the ‘Failed to connect’ message…

Reverted back to 2024.8.0.

@lysanev
Copy link

lysanev commented Aug 10, 2024

homeassistant.components.daikin.config_flow
components/daikin/config_flow.py:87
Daikin AC

Unexpected error creating device
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/daikin/config_flow.py", line 87, in _create_device
device: Appliance = await DaikinFactory(
^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/pydaikin/factory.py", line 26, in new
await instance.init(*a, **kw)
File "/usr/local/lib/python3.12/site-packages/pydaikin/factory.py", line 52, in init
await self._generated_object.update_status(
File "/usr/local/lib/python3.12/site-packages/pydaikin/daikin_base.py", line 189, in update_status
self.values.update_by_resource(resource, task.result())
^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/tenacity/asyncio/init.py", line 189, in async_wrapped
return await copy(fn, *args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/tenacity/asyncio/init.py", line 111, in call
do = await self.iter(retry_state=retry_state)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/tenacity/asyncio/init.py", line 153, in iter
result = await action(retry_state)
^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/tenacity/_utils.py", line 99, in inner
return call(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/tenacity/init.py", line 398, in
self._add_action_func(lambda rs: rs.outcome.result())
^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/concurrent/futures/_base.py", line 449, in result
return self.__get_result()
^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/concurrent/futures/_base.py", line 401, in __get_result
raise self._exception
File "/usr/local/lib/python3.12/site-packages/tenacity/asyncio/init.py", line 114, in call
result = await fn(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/pydaikin/daikin_base.py", line 138, in _get_resource
headers = self.headers
^^^^^^^^^^^^
AttributeError: 'DaikinBRP069' object has no attribute 'headers'

@robitsrl
Copy link

Same problem with BRP069B4x

@DoMaboul
Copy link

same here...

Enregistreur: pydaikin.daikin_base
Source: components/daikin/init.py:88
Exception in TaskGroup: 'DaikinBRP069' object has no attribute 'headers'

@messerffm
Copy link

Same issue here with my three BRP069B4x. :(

@naked-head
Copy link

Same issue here

@sergiomastrocola
Copy link

same issue

@alpha520098
Copy link

Same issue! - HA updates are always hit and miss

@totolook
Copy link

The cause this PR fredrike/pydaikin#13 the missing the members (headers and ssl_context) in my case in DaikinBRP069 class

@Danifly80
Copy link

Same issue

@tomlut
Copy link

tomlut commented Aug 10, 2024

I don't have a BRP069. I have a BRP072B42. Why does it think I have a BRP069?

EDIT: actually I have both. Just realised this release has broken my ducted system as well as the mini split.

Logger: homeassistant.components.daikin.config_flow
Source: components/daikin/config_flow.py:87
integration: Daikin AC (documentation, issues)
First occurred: 07:38:19 (1 occurrences)
Last logged: 07:38:19

Unexpected error creating device
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/daikin/config_flow.py", line 87, in _create_device
    device: Appliance = await DaikinFactory(
                        ^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/pydaikin/factory.py", line 26, in __new__
    await instance.__init__(*a, **kw)
  File "/usr/local/lib/python3.12/site-packages/pydaikin/factory.py", line 52, in __init__
    await self._generated_object.update_status(
  File "/usr/local/lib/python3.12/site-packages/pydaikin/daikin_base.py", line 189, in update_status
    self.values.update_by_resource(resource, task.result())
                                             ^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/tenacity/asyncio/__init__.py", line 189, in async_wrapped
    return await copy(fn, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/tenacity/asyncio/__init__.py", line 111, in __call__
    do = await self.iter(retry_state=retry_state)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/tenacity/asyncio/__init__.py", line 153, in iter
    result = await action(retry_state)
             ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/tenacity/_utils.py", line 99, in inner
    return call(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/tenacity/__init__.py", line 398, in <lambda>
    self._add_action_func(lambda rs: rs.outcome.result())
                                     ^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/concurrent/futures/_base.py", line 449, in result
    return self.__get_result()
           ^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/concurrent/futures/_base.py", line 401, in __get_result
    raise self._exception
  File "/usr/local/lib/python3.12/site-packages/tenacity/asyncio/__init__.py", line 114, in __call__
    result = await fn(*args, **kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/pydaikin/daikin_base.py", line 138, in _get_resource
    headers = self.headers
              ^^^^^^^^^^^^
AttributeError: 'DaikinBRP069' object has no attribute 'headers'

@DaveOzzie
Copy link

Same problem with my Daikin BRP15B61

@EthanBezz
Copy link

EthanBezz commented Aug 10, 2024

For anyone who knows what they're doing, here's a temporary solution:

sed -i '119a \    headers = None\n    ssl_context = None\n' /usr/local/lib/python3.12/site-packages/pydaikin/daikin_brp069.py

Executing that command against your Home Assistant install will apply the same change as the above Pull Request (fredrike/pydaikin#14). I tested it on my end and it worked for me.

@joao-opo
Copy link

Everyone is coming here to state the same: this change broke the integration.

image

(now I need to find those old remotes... where did I put them???)

@alpha520098
Copy link

Perhaps @EthanBezz you could make a step by step instructions on how you fixed it - you will get legendary status

@EthanBezz
Copy link

Perhaps @EthanBezz you could make a step by step instructions on how you fixed it - you will get legendary status

Sorry but I won't. Simply because there are multiple ways that Home Assistant can be installed and trying to make a guide that caters to each one, for what is a temporary solution, isn't worth it IMO.

Also, I don't want to be blamed for borking anyones install, hence why I said "For anyone who knows what they're doing".

@jee67
Copy link

jee67 commented Aug 12, 2024

I’m using home assistant Core 2024.8.1 on my RPI and face the same issues on all my AC’s. All my climate automations stopped with this bug 😭. Luckily the systems can be also controlled manually without Home Assistant. I will try to downgrade to 2024.7.4 today as I don’t have received the above mentioned 2024.8.0. Anybody know how to receive update 2024.8.0 only? Hope the update will be released fast.

ha core update —version 2024.8.0

@alpha520098
Copy link

@jee67 The owner of the code @fredrike has sent out some information suggesting that it will be fixed with HA next update 8.2 so hold tight with your current version and it will be fixed with the next update.

@mike4001
Copy link

2024.8.2 is due on 16. August

@tomlut
Copy link

tomlut commented Aug 13, 2024

One advantage to having the Daikin integration disabled: My HA restart times have gone from >60 seconds to <20 seconds. 😁

@sunnyiiii
Copy link

Same issue with BRP15B61

@wvkregten
Copy link

Same issue here with the wifi module brp069b41

@arpel
Copy link

arpel commented Aug 13, 2024

To simply mimic the code change solving the issue, you can update the package version for pydaikin yourself :
in /[whatever your HA base dir is]/lib/python3.12/site-packages/homeassistant/components/daikin modify the following in manifest.json :
"requirements": ["pydaikin==2.13.3"] instead of "requirements": ["pydaikin==2.13.3"]
and restart your HA instance, for me it upgraded immediately (I'm running Core, not HAOS).

@VinceRMP
Copy link
Author

To simply mimic the code change solving the issue, you can update the package version for pydaikin yourself :

in /[whatever your HA base dir is]/lib/python3.12/site-packages/homeassistant/components/daikin modify the following in manifest.json :

"requirements": ["pydaikin==2.13.3"] instead of "requirements": ["pydaikin==2.13.3"]

and restart your HA instance, for me it upgraded immediately (I'm running Core, not HAOS).

That PR is currently blocked because there are still possible issues with it. I would suggest to just revert to 2024.8.0 and wait till the approved fix is merged.

@Marcoletto
Copy link

Same

Registratore: pydaikin.daikin_base
Fonte: components/daikin/init.py:88
Prima occorrenza: 17:05:25 (2 occorrenze)
Ultima registrazione: 17:05:25

Exception in TaskGroup: 'DaikinBRP069' object has no attribute 'headers'

@rmarinho
Copy link

Same issue here.

@mdima
Copy link

mdima commented Aug 13, 2024

Same issue here. Hope that 2024.8.2 will be released ASAP... thanks!

@icygame
Copy link

icygame commented Aug 13, 2024

just to add the standard comment...same issue here.

@jurgenweber
Copy link

Hi, just to communicate some GitHub etiquette, saying 'same issue here' is not helpful and just clutters up the issue comments. Please just +1 the original post with a Thumbs up. This is the cleaner way to say, 'same issue here'.

Thanks

@thomas-99
Copy link

Same issue here.

@tomlut
Copy link

tomlut commented Aug 14, 2024

Read the post directly above yours 🤦‍♂️

@nfrankel
Copy link

@joostlek Sorry, but I've updated to the latest HA version (2024.8.1) and I still have the same error. For good measure, I restarted HA and it's still there.

Did I miss something?

@joostlek
Copy link
Member

2024.8.2 will have it

@VinceRMP
Copy link
Author

The fix will be in 2024.8.2 which is not released yet.

@nfrankel
Copy link

Thanks for your quick reply @joostlek & @VinceRMP

@tefod-zz
Copy link

Looking forward to the update! Due the missing automation, my back pain, arthritis, and muscle soreness are getting a workout from this unusual Remote Control Dash training 😶‍🌫️

@DragonQ
Copy link

DragonQ commented Aug 15, 2024

@joostlek Sorry, but I've updated to the latest HA version (2024.8.1) and I still have the same error. For good measure, I restarted HA and it's still there.

Did I miss something?

Looking forward to the update! Due the missing automation, my back pain, arthritis, and muscle soreness are getting a workout from this unusual Remote Control Dash training 😶‍🌫️

Downgrading to 2024.8.0 will also fix the issue described in this thread.

@spaggio80
Copy link

spaggio80 commented Aug 16, 2024

Davvero strano... non si rompe mai niente in Home Assistant :D (scusate il sarcasmo)
Anch'io stesso problema comunque.

@jo8be
Copy link

jo8be commented Aug 16, 2024

2024.8.2 is out and it has fixed the Daikin AC issue for me.

@Diomet
Copy link

Diomet commented Aug 17, 2024

2024.8.2 is out and it has fixed the Daikin AC issue for me.

Affirmative! 2024.8.2 solves the issue.

@fredrike
Copy link
Contributor

Davvero strano... non si rompe mai niente in Home Assistant :D (scusate il sarcasmo) Anch'io stesso problema comunque.

And how much are you paying for HA? The Daikin integration is built by enthusiasts on their spare time (my Daikin unit does not even operate anymore).

Totally unnecessary comment. Show some appreciation instead.

@alpha520098
Copy link

@fredrike well on behalf of everyone else here I'd like to extend a very sincere thank you for your efforts and I happy to report that all is right in the world because I can turn on my a/c.

I certainly appreciate your support

@github-actions github-actions bot locked and limited conversation to collaborators Sep 16, 2024
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.