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

109.4 update causes wiserheatingapi to get deleted #105

Closed
scooper1 opened this issue May 6, 2020 · 24 comments
Closed

109.4 update causes wiserheatingapi to get deleted #105

scooper1 opened this issue May 6, 2020 · 24 comments
Assignees

Comments

@scooper1
Copy link

scooper1 commented May 6, 2020

wiser api still being deleted and needing reinstall after update to 109.4
this is still happening after the comma fix

@asantaga
Copy link
Owner

asantaga commented May 6, 2020

I have no idea.. It works perfectly on my instance and Adam on https://community.home-assistant.io/t/drayton-wiser-home-assistant-integration/80965/117 says it now works fine (after the comma fix)...

I'll create a clean virtual environment and test it out tomorrow... So from memory, you need to log using portainer (so your using docker) and manually pip install the library right?

@scooper1
Copy link
Author

scooper1 commented May 6, 2020

yes normal docker install
i have send this behaviour both machines I have access to
no other libraries (not checked that there are libraries) that are installed via hacs have this problems - i don't know how libraries are handled during a update but really would not expect them to get deleted unless the complete docker image is being replaced ?

@msp1974
Copy link
Collaborator

msp1974 commented May 6, 2020

Not seeing either. Just updated to 109.4 and all was fine.

@chris-github1975
Copy link

chris-github1975 commented May 7, 2020 via email

@scooper1
Copy link
Author

scooper1 commented May 7, 2020

Maybe just a interaction on my systems
Any idea what would cause library to get deleted during a supervisor update ?

@asantaga
Copy link
Owner

asantaga commented May 7, 2020

I have no idea.,. Im assuming your using the official (if it thats what we call it) home assistant docker image from docker hub right?

If so then perhaps delete the container, any assigned volumes and then recreate the container.. That said when u upgrade the container would be blitzed anyway..

@asantaga asantaga changed the title 109.4 update 109.4 update causes wiserheatingapi to get deleted May 7, 2020
@scooper1
Copy link
Author

scooper1 commented May 7, 2020

yes using official image
I accept that problem is something strange on my systems
just trying to understand how it happens to be able to fix it
just happened again on 109.5 update

@asantaga
Copy link
Owner

asantaga commented May 7, 2020

lets sync up again in a couple of days, happy to hold a zoom with you and help debug this.. im very curious to why this is happening..

@scooper1
Copy link
Author

scooper1 commented May 7, 2020

Ok I would be up for that.
Normally use discord but I try to get zoom account
Thanks

@scooper1
Copy link
Author

scooper1 commented May 8, 2020

I think my hacs install might be a bit corrupted around wiser component as yesterday it was giving a title with no spaces between words (ok now) and i think today the readme.md is not showing as formatted texted instead in gives a raw link

is the anyway to rebuild the hacs database without - might try deleting the wiser component and re-installing

should the config\deps folder contain a copy of the wiserheatingapi or a link
mine is empty
documentation implies that this contains a copy of any dependencies required after docker container restart - this may just be out of date info

@msp1974
Copy link
Collaborator

msp1974 commented May 8, 2020

Only thing i can think of worth looking at is

https://community.home-assistant.io/t/ha-0-103-0-update-broke-hacs-requirements-for-hacs-not-found-aiofiles-0-4-0/154896/9

I don't know if your other HACs integrations need to download any dependancies.

Other people on various similar issues have suggested taking a snapshot, delete and reinstall docker env and then restore shapshot. Never done this so dont know how much it does or does not keep...on you!

@asantaga
Copy link
Owner

asantaga commented May 8, 2020

Hey @scooper1 , I have a zoom account, so zoom isnt a problem..

Not sure you need to create a snapshot of the docker env. If you configured it correctly all your config is mapped to a volume on your filesystem.. (ie the -v flag)

You should be able to delete the docker container, delete the image (just in case) and then recreate the container.. thats the beauty of docker

@asantaga
Copy link
Owner

asantaga commented May 8, 2020

oh, and yeah I would definitively also delete the wiser directory in custom_components

@scooper1
Copy link
Author

scooper1 commented May 8, 2020

the docker container appears to be recreated on each upgrade
it appears to me that hacs does a scan of the manifest files and reinstalls the requirements
but on my system the it misses the wiserheatingapi
so I have deleted the wiser custom components and then reinstalled via hacs then done the 109.6 upgrade that deletes the hassioassistant container and recreates it
still same problem I had to pull wiserheatingapi manually

@asantaga
Copy link
Owner

Sorry bank holiday got me..

i have no clue..

Can we try one more thing..

Can you create a "NEW" HA container, pointing to a new config directory, ie a different -v mapping..

Lets see if it happens there too, if it works then there is something with the HA config we need de-debug,.

@asantaga asantaga self-assigned this May 11, 2020
@scooper1
Copy link
Author

I am not sure how to do a new container
but turn on debugging for hacs
uninstall wiser restarted and then installed wiser restarted
at no point did I get any debug messages about installing the wiserheatingapi
tried a few other like breaking changes - still did not see any checking for requirements
thinking that my hacs is broken from the data I have in the config directory

@asantaga
Copy link
Owner

Hey, let me know when we can do a zoom and I'll have a nose.. I do think somethings wrong with HA but worth being sure. Ping me an email , see any of the files, and we'll sort this out offline

@scooper1
Copy link
Author

the problem maybe because I have a growing ha system and its tripped this problem
on update
https://community.home-assistant.io/t/ha-0-103-0-update-broke-hacs-requirements-for-hacs-not-found-aiofiles-0-4-0/154896

@scooper1
Copy link
Author

cant find a private email account
I set up a temporary test user that you should be able to see my ha server via duckdns address - but I only want to send login details to private email address - not this public platform

@scooper1
Copy link
Author

this problem seems to fixed in 1.10b0
beta installed without needing to pull wiserheatingapi
did notice some new log entries
2020-05-14 01:56:07 WARNING (MainThread) [homeassistant.components.switch] SwitchDevice is deprecated, modify WiserSwitch to extend SwitchEntity
2020-05-14 01:56:07 WARNING (MainThread) [homeassistant.components.switch] SwitchDevice is deprecated, modify WiserSmartPlug to extend SwitchEntity
2020-05-14 01:56:07 WARNING (MainThread) [homeassistant.components.climate] ClimateDevice is deprecated, modify WiserRoom to extend ClimateEntity

@msp1974
Copy link
Collaborator

msp1974 commented May 14, 2020

Have been made aware of this on my jag integration. However, these new entity types are only available in v0.110 onwards which seems a little premature to give depreciation warnings in the logs. We will need to work out how we deploy these changes as people who have not upgraded to v0.110 will not be able to use it.

@msp1974
Copy link
Collaborator

msp1974 commented May 14, 2020

Probably a good reason to get this into the main repo which we keep stopping Angelo doing with requests for fixes/upgrades. Me included! ;-)

@asantaga
Copy link
Owner

asantaga commented May 14, 2020

Joy!!!

0.110 is due out may 20, i'll wait till then , then fix the warnings

@asantaga
Copy link
Owner

closing this one, opening new issue

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

No branches or pull requests

4 participants