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

20220802 bump homeassistant 2022 7 7 #5385

Conversation

chickenandpork
Copy link
Member

@chickenandpork chickenandpork commented Aug 7, 2022

Description

This is a simple repack of the latest (at time of branch) HomeAssistant. This seems to work on the few uses I have on my DS1819+ (denverton-7.0) (running DSM 7.0.1-42218 Update 4). Install did stop me until Python-3.10 was installed (which is probably @th0ma7 's work).

Of note, several concerns and blockers have been raised to consider around a HomeAssistant update; I don't specifically fix any of them except half of the two-part "update HA so that " (#5362). Indeed, I'm doing this to unblock Tuya for myself, but I've only just upgraded to this without installing a cloud-poll Tuya yet.

I want to make this available to be able to audit the other open HA issues and see if it helps them at all, or we can update this reports.

I also want to call attention(re #5043) to @th0ma7 and @hgy59 to have them weigh-in on simply releasing a re-pack for beta-review and see if it helps the community without the rust-cross compilation fully available.

Relates to: #5362 #4580
Does not address #4183 #4977 (@mortenlj), #5125
May Unblock: #4189 #4969 #5362

Checklist

  • [ongoing] Build rule all-supported completed successfully
  • New installation of package completed successfully
  • Package upgrade completed successfully (Manually install the package again)
  • Package functionality was tested
  • Any needed documentation is updated/created

Type of change

  • Bug fix
  • New Package
  • Package update
  • Includes small framework changes
  • This change requires a documentation update (e.g. Wiki)

@chickenandpork
Copy link
Member Author

Logs show issues with pyatv - I'd prefer to rebase our world on 2022-07-07 as a beta and layer-on fixes for various issue son top -- I worry that in Spksrc we suffer from perfect being the enemy of good, and prefect PRs being too large to review.

@hgy59
Copy link
Contributor

hgy59 commented Aug 7, 2022

@chickenandpork I doubt that this will ever run. At least the requirements files need to be updated.
I am still longing for someone that helps to build cryptography 36.0.2 wheel that needs setuptools_rust to build.

@chickenandpork
Copy link
Member Author

I appreciate the feedback; mine is currently running but maybe there are undetected errors.

move got a basic Tuya connection (cloud) but no scenes yet to control

@chickenandpork
Copy link
Member Author

I'm looking to follow this PR with an update: Essentially, I want to use another Integration that needs a narrow, very recent dependency in the version-lock files.

@hgy59 I had no problems running this on python-3.9, pre-rust-wheels: minimal work needed to recognize and update cffi. I haven't tested on python-3.10. Can we merge this and move forward to the next update?

A recurring issue with spksrc development is that there seems to be this chase-the-horizon of PRs: hard to get review, and no part of the work is ever merged until it's pristine.

Can we merge this and move forward off it to rebase on current HomeAssistant? Would you rather I rebased on this week's release ?

@hgy59
Copy link
Contributor

hgy59 commented Oct 28, 2022

@hgy59 I had no problems running this on python-3.9, pre-rust-wheels: minimal work needed to recognize and update cffi. I haven't tested on python-3.10. Can we merge this and move forward to the next update?

Can we merge this and move forward off it to rebase on current HomeAssistant? Would you rather I rebased on this week's release ?

@chickenandpork you can go forward with rebase and merge...
I will take a later version (probably 2022.10.5) for the next update. As I run a test suite that loads all integrations to find any missing dependency, it will take some time to get it ready to publish.

@chickenandpork chickenandpork force-pushed the 20220802-bump-homeassistant-2022-7-7 branch from 0a150ac to bfc43dd Compare October 28, 2022 18:25
@chickenandpork
Copy link
Member Author

chickenandpork commented Oct 28, 2022

rebased on e9ee4f0 (current master)

@hgy59
Copy link
Contributor

hgy59 commented Oct 31, 2022

@chickenandpork this does not update homeassistant to 2022.7.7. After installation/update it still shows homeassistant 2021.9.7.
The only update is the python dependency that changed from Python 3.8 to 3.10

I started to update homeassistant to 2022.10.5.

@chickenandpork
Copy link
Member Author

chickenandpork commented Nov 1, 2022

does not update

@hgy59 before this change, I was unable to use the Cloud/SaaS Tuya. I built and installed this update as a clean build, and I was able to install and use the Integration that required new dependencies in a very narrow range.

I found that the build is not always hermetic, but in a clean build, installing the result, I saw a change in behavior, with no other changes made to the DSM involved.

Maybe I'll pull apart the SPK and see what it brings as payload

@daxcore
Copy link

daxcore commented Dec 20, 2022

@hgy59 I tested ur current 22.10.5 version. starts and works fine so far. 👍

but would be nice, if we could update to version 2022.11.0 to get HACS support

You are running Home Assistant version 2022.11.0 or newer.

another point, restarting the service via HA web-interface doesn't work. I have to start the service via synology web-interface all the time.

@chickenandpork chickenandpork deleted the 20220802-bump-homeassistant-2022-7-7 branch December 20, 2022 18:38
@chickenandpork
Copy link
Member Author

@daxcore moved your message to #5530

@daxcore
Copy link

daxcore commented Dec 22, 2022

I'm sorry. home-assistant is a new topic for me.

I realized, it is not necessary to upgrade to 2022.11.0 go get HACS running.
With 2022.10.5 we are just limited upto version 1.28.4.
Since version 1.29.0 HA 2022.11.0 is required.

I have HACS 1.28.4 running on this 2022.10.5 release.

update:
btw: for hacs (at least for the installation) unzip is required.
I just downloaded a pre-compiled version it for aarch64, but that's not, like it should be ;-)

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

Successfully merging this pull request may close these issues.

3 participants