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

Updates deconz to the latest stable version: 2.25.3 #3432

Merged
merged 4 commits into from
Feb 9, 2024

Conversation

francescopeloi
Copy link
Contributor

@francescopeloi francescopeloi commented Jan 23, 2024

Updates deconz to the latest stable version: 2.25.3

Note: this hasn't been tested locally by me. I am trying to help with what I can, so at least do the code change.

  • Phoscon/deconz says that 2.24.3 includes a fix for this deconz addon to work (source)
  • Problems that were experienced before are discussed in deconz forum (source) and on the previous PR (source)
  • I think the other PR can be closed, as it's upgrading to a broken version.

@francescopeloi francescopeloi marked this pull request as draft January 23, 2024 15:23
@francescopeloi
Copy link
Contributor Author

I made it a draft to make it clearer that this needs to be locally tested first.

@manup
Copy link
Contributor

manup commented Jan 23, 2024

Hi if it is any help I did test the v2.24.3 version with my external add-on repository: https://github.com/manup/ha_addons_test which was also used to fix the issues mentioned in the changelog.

@francescopeloi francescopeloi marked this pull request as ready for review January 23, 2024 20:58
@francescopeloi
Copy link
Contributor Author

thanks @manup , I put it out of draft state then

@frenck frenck self-assigned this Jan 23, 2024
@frenck frenck self-requested a review January 23, 2024 21:02
@frenck
Copy link
Member

frenck commented Jan 23, 2024

Assigned myself to run a suite of tests.

@francescopeloi
Copy link
Contributor Author

@frenck , to avoid the risk of wasting your time, it might be worth for this thread to get resolved before you start your testing.

In a nutshell:

  • 1 user is experiencing slow network and high cpu usage with deconz v2.24.2 and v2.25.1 (and not with 2.23.2)
  • 1 user is experiencing the same with deconz v2.24.3 (and not with 2.23.2)
  • it seems that the problem these users are experiencing is with versions > 2.23.2
  • one user says it's all good with him
  • deconz people have been tagged and hopefully will respond

@manup maybe you can help here, Mimiix tagged @de_employees a couple of times on the thread

Just a suggestion of course, up to you.

@bcutter bcutter mentioned this pull request Jan 29, 2024
@francescopeloi
Copy link
Contributor Author

some good developments: https://forum.phoscon.de/t/current-deconz-2-24-3-2-25-1-slow-in-response-after-time/4517/48

I'll wait for a new version from @manup and update this PR accordingly. Putting the PR back to draft for now.

@francescopeloi francescopeloi marked this pull request as draft February 1, 2024 10:59
@dutch-erik
Copy link

Latest stable deconz release v2.25.3

@francescopeloi francescopeloi changed the title Updates deconz to the latest stable version: 2.24.3 Updates deconz to the latest stable version: 2.25.3 Feb 7, 2024
@francescopeloi
Copy link
Contributor Author

As @dutch-erik says, there's a new deconz version which fixes the problems I've highlighted in my previous comment.

@frenck when you get the time this is now ready to be tested, thanks for your help with this.

Putting the PR out of draft.

@francescopeloi francescopeloi marked this pull request as ready for review February 7, 2024 22:16
@frenck
Copy link
Member

frenck commented Feb 9, 2024

I am picking this up right now. Sorry for the delays, it has been hectic :)

Copy link
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Completed testing on amd64 & aarch64 architectures.

Confirmed the add-on worked well. Ingress seems to have no issues or errors, and no oddities in the startup and other logs seen.

Nice! ❤️

../Frenck

PS: If you see this and think: Wait, this is merged and I don't see the new version yet? That is possible. It seems the build takes quite some time right now (1,5 hours on my ARM test devices installing from source). So it may take a few hours after I merge this PR before it becomes available.

Edit: Available for all platforms now.

@frenck frenck merged commit 955493d into home-assistant:master Feb 9, 2024
10 checks passed
@DanielHabenicht
Copy link

Installed and works like a charm with the new IKEA sensors. Thanks for your effort!

@lasry1
Copy link

lasry1 commented Feb 9, 2024

It works perfectly! Thank you!

@MasterEvarior
Copy link

It works beautifully, thanks!

@rexbron
Copy link

rexbron commented Feb 16, 2024

Hi,

I am having an issue with 6.23.0 where it fails to start with an Exec format error:
#3470

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants