migration broken since last release

[ChrisOboe]

Checklist

• I have updated the integration to the latest version available
• I have checked if the problem is already reported

The problem

Since updating to the lastest version homeassistant shows a migration error.

here is the relevant log:

Traceback (most recent call last):
  File "/nix/store/7z97x7nvmr0kpssp9xa3j058x4nmgx3f-homeassistant-2024.3.3/lib/python3.12/site-packages/homeassistant/config_entries.py", line 770, in async_migrate
    result = await component.async_migrate_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/var/lib/hass/custom_components/valetudo_vacuum_camera/__init__.py", line 211, in async_migrate_entry
    await move_data_to_valetudo_camera()
  File "/var/lib/hass/custom_components/valetudo_vacuum_camera/__init__.py", line 305, in move_data_to_valetudo_camera
    os.makedirs(valetudo_camera_folder)
  File "<frozen os>", line 215, in makedirs
  File "<frozen os>", line 225, in makedirs
OSError: [Errno 30] Read-only file system: '/.storage'

it seems that it tries to write to /.storage (which doesn't exist)
my homeassistant storage dir lives in /var/lib/hass/.storage
all the other plugins and homeassistant itself uses the correct storage dir.

What version of an integration has described problem?

2024.04.1

What was the last working version of an integration?

No response

What vacuum model do you have problems with?

two dreames

Please firmware installed on your Vacuum.

valtudo

What type of platform you use?

Intel NUC (or generic x86_64)

What version of Home Assistant do you use?

2024.4.0

What type of installation are you running?

Home Assistant Core

Camera's settings.

No response

Logs or Errors shown in the HA snapshots (please enable the Debug Mode)

No response

Additional information

No response

[ChrisOboe]

i just checked on my system hass is executed with / as PWD.
as a workarround i can modify my hass systemd service to change the pwd before starting hass.

[sca075]

@ChrisOboe thanks a lot for this.. I will check why it didn't get the path automatically as it should.
On the contained version of HA I had no problem.
The previous version do not use the valetudo_camera folder in .storage.

Probably the issue is that it can't create the folder without psw.

[sca075]

@ChrisOboe does the camera work after you created the folder with the proper password? If not would work on a patch for it.
https://github.com/sca075/valetudo_vacuum_camera/blob/20c46543f24840b98b3aa82098e8761755d733a9/custom_components/valetudo_vacuum_camera/__init__.py#L299-L306

as you can see I'm looking for the current working dir of home assistant and honestly I do not catch how this failed to create the folder on your system.

[sca075]

@ChrisOboe I hope you could figure out a work around and once again thanks for highlight this issue.
when os.getcwd didn't actually got that your instance was working in /var/lib/hass/ is your config folder, I will investigate on what is the best method to avoid this in the future. Apologies for the inconvenience.

[carefulcomputer]

same error. let me know if you need any logs/info from my system. I have HAAS running in a docker container. Can you also please provide a workaround ?

[sca075]

same error. let me know if you need any logs/info from my system. I have HAAS running in a docker container. Can you also please provide a workaround ?

yes please can you provide the logs? I will try to fix this asap.

[carefulcomputer]

not just migration, new install is broken too. I removed old integration and tried to reinstall from scratch and got error.

2024-04-09T04:43:09.802580123Z 2024-04-08 21:43:09.786 ERROR (MainThread) [aiohttp.server] Error handling request
2024-04-09T04:43:09.802772587Z Traceback (most recent call last):
2024-04-09T04:43:09.802814525Z   File "/usr/local/lib/python3.12/site-packages/aiohttp/web_protocol.py", line 452, in _handle_request
2024-04-09T04:43:09.802853088Z     resp = await request_handler(request)
2024-04-09T04:43:09.802890076Z            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.803181241Z   File "/usr/local/lib/python3.12/site-packages/aiohttp/web_app.py", line 543, in _handle
2024-04-09T04:43:09.803245104Z     resp = await handler(request)
2024-04-09T04:43:09.803281467Z            ^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.803316917Z   File "/usr/local/lib/python3.12/site-packages/aiohttp/web_middlewares.py", line 114, in impl
2024-04-09T04:43:09.803354930Z     return await handler(request)
2024-04-09T04:43:09.803389930Z            ^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.803426856Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/components/http/security_filter.py", line 92, in security_filter_middleware
2024-04-09T04:43:09.803464919Z     return await handler(request)
2024-04-09T04:43:09.803500844Z            ^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.803536382Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/components/http/forwarded.py", line 83, in forwarded_middleware
2024-04-09T04:43:09.803573095Z     return await handler(request)
2024-04-09T04:43:09.803681058Z            ^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.803720608Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/components/http/request_context.py", line 26, in request_context_middleware
2024-04-09T04:43:09.803757846Z     return await handler(request)
2024-04-09T04:43:09.803793134Z            ^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.803890898Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/components/http/ban.py", line 88, in ban_middleware
2024-04-09T04:43:09.804034924Z     return await handler(request)
2024-04-09T04:43:09.804071112Z            ^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.804106687Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/components/http/auth.py", line 236, in auth_middleware
2024-04-09T04:43:09.804143237Z     return await handler(request)
2024-04-09T04:43:09.804178488Z            ^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.804231463Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/components/http/headers.py", line 32, in headers_middleware
2024-04-09T04:43:09.804268751Z     response = await handler(request)
2024-04-09T04:43:09.804304139Z                ^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.804339964Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/helpers/http.py", line 73, in handle
2024-04-09T04:43:09.804387002Z     result = await handler(request, **request.match_info)
2024-04-09T04:43:09.804424365Z              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.804468390Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/components/http/decorators.py", line 71, in with_admin
2024-04-09T04:43:09.804516303Z     return await func(self, request, *args, **kwargs)
2024-04-09T04:43:09.804616154Z            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.804676230Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/components/config/config_entries.py", line 222, in post
2024-04-09T04:43:09.804713468Z     return await super().post(request, flow_id)
2024-04-09T04:43:09.804759268Z            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.804796544Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/components/http/data_validator.py", line 73, in wrapper
2024-04-09T04:43:09.804833444Z     result = await method(view, request, data, *args, **kwargs)
2024-04-09T04:43:09.804882282Z              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.804919182Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/helpers/data_entry_flow.py", line 122, in post
2024-04-09T04:43:09.804955733Z     result = await self._flow_mgr.async_configure(flow_id, data)
2024-04-09T04:43:09.805005658Z              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.805043058Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/data_entry_flow.py", line 356, in async_configure
2024-04-09T04:43:09.805079709Z     result = await self._async_configure(flow_id, user_input)
2024-04-09T04:43:09.805128372Z              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.805164785Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/data_entry_flow.py", line 402, in _async_configure
2024-04-09T04:43:09.805259973Z     result = await self._async_handle_step(
2024-04-09T04:43:09.805298198Z              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.805333799Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/data_entry_flow.py", line 506, in _async_handle_step
2024-04-09T04:43:09.805380662Z     result: _FlowResultT = await getattr(flow, method)(user_input)
2024-04-09T04:43:09.805418037Z                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T04:43:09.805454787Z   File "/config/custom_components/valetudo_vacuum_camera/config_flow.py", line 224, in async_step_user
2024-04-09T04:43:09.805505838Z     os.makedirs(storage_path)
2024-04-09T04:43:09.805542426Z   File "<frozen os>", line 215, in makedirs
2024-04-09T04:43:09.805579188Z   File "<frozen os>", line 225, in makedirs
2024-04-09T04:43:09.805627251Z PermissionError: [Errno 13] Permission denied: '/run/s6-rc:s6-rc-init:OlgGKB/servicedirs/svc-homeassistant/.storage'

[sca075]

I figure out that the issue might be caused from os.path.join.

 storage_folder = os.path.join(os.getcwd(), STORAGE_DIR) 
 valetudo_camera_folder = os.path.join(storage_folder, "valetudo_camera")

this lines 300 and 301 in init.py need to be replaced with:

 storage_folder = f"{os.getcwd()}/{STORAGE_DIR}"
 valetudo_camera_folder = f"{storage_folder}/valetudo_camera"

can you give it a try and let me know if it works on yours @carefulcomputer and @ChrisOboe configs?
alternatively you can download the main where I did modify those lines just few minutes ago #1a3e387

[carefulcomputer]

i edited the file, and restarted haas. still getting the error when trying to set the vaccum name in integrations.
do i need to clear some cache or perform any other operations etc ?

2024-04-09T05:58:52.761256281Z     return await func(self, request, *args, **kwargs)
2024-04-09T05:58:52.761342494Z            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T05:58:52.761427320Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/components/config/config_entries.py", line 222, in post
2024-04-09T05:58:52.761515496Z     return await super().post(request, flow_id)
2024-04-09T05:58:52.761600171Z            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T05:58:52.761686322Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/components/http/data_validator.py", line 73, in wrapper
2024-04-09T05:58:52.761775573Z     result = await method(view, request, data, *args, **kwargs)
2024-04-09T05:58:52.761866861Z              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T05:58:52.762006288Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/helpers/data_entry_flow.py", line 122, in post
2024-04-09T05:58:52.762105814Z     result = await self._flow_mgr.async_configure(flow_id, data)
2024-04-09T05:58:52.762192089Z              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T05:58:52.762280153Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/data_entry_flow.py", line 356, in async_configure
2024-04-09T05:58:52.762368679Z     result = await self._async_configure(flow_id, user_input)
2024-04-09T05:58:52.762461942Z              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T05:58:52.762541418Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/data_entry_flow.py", line 402, in _async_configure
2024-04-09T05:58:52.762618306Z     result = await self._async_handle_step(
2024-04-09T05:58:52.762691119Z              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T05:58:52.762764145Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/data_entry_flow.py", line 506, in _async_handle_step
2024-04-09T05:58:52.762839145Z     result: _FlowResultT = await getattr(flow, method)(user_input)
2024-04-09T05:58:52.762913159Z                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-09T05:58:52.763030610Z   File "/config/custom_components/valetudo_vacuum_camera/config_flow.py", line 224, in async_step_user
2024-04-09T05:58:52.763119748Z     os.makedirs(storage_path)
2024-04-09T05:58:52.763195236Z   File "<frozen os>", line 215, in makedirs
2024-04-09T05:58:52.763270025Z   File "<frozen os>", line 225, in makedirs
2024-04-09T05:58:52.763344888Z PermissionError: [Errno 13] Permission denied: '/run/s6-rc:s6-rc-init:OlgGKB/servicedirs/svc-homeassistant/.storage'

[sca075]

@carefulcomputer this is while editing options?? because it do not looks the startup sequence. Can you please download and instal the integration from the main repository, enable the logs for the camera and restart home assistant and provide me the startup sequence please?

edit: hold on, please revert to the 2024.04.0 I think I got to review the way how to get the working directory, some issue with python 12. with os.getcwd probably.

[carefulcomputer]

thanks for looking into it! it is getting late for me. i will try tomorrow.

[carefulcomputer]

any luck ?

[sca075]

I'm working on it, in few minutes I will update the main repository.

[sca075]

@carefulcomputer can you please download the main distribution, I added some try to handle the errors and as well some additional logging and a counter measure in case of failure. It should work now if all goes as planned.
Indeed I would need your kind feedback and I do really thanks you in advance for the cooperation here.

[carefulcomputer]

thank you for new update. must be something really weird. still getting error.

2024-04-10T13:55:21.773107117Z            ^^^^^^^^^^^^^^^^^^^^^^
2024-04-10T13:55:21.773164530Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/components/http/headers.py", line 32, in headers_middleware
2024-04-10T13:55:21.773225318Z     response = await handler(request)
2024-04-10T13:55:21.773279556Z                ^^^^^^^^^^^^^^^^^^^^^^
2024-04-10T13:55:21.773317682Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/helpers/http.py", line 73, in handle
2024-04-10T13:55:21.773356532Z     result = await handler(request, **request.match_info)
2024-04-10T13:55:21.773395757Z              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-10T13:55:21.773477083Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/components/http/decorators.py", line 71, in with_admin
2024-04-10T13:55:21.773544859Z     return await func(self, request, *args, **kwargs)
2024-04-10T13:55:21.773602072Z            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-10T13:55:21.773659772Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/components/config/config_entries.py", line 222, in post
2024-04-10T13:55:21.773718098Z     return await super().post(request, flow_id)
2024-04-10T13:55:21.773775161Z            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-10T13:55:21.773832124Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/components/http/data_validator.py", line 73, in wrapper
2024-04-10T13:55:21.773891937Z     result = await method(view, request, data, *args, **kwargs)
2024-04-10T13:55:21.773950125Z              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-10T13:55:21.774009113Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/helpers/data_entry_flow.py", line 122, in post
2024-04-10T13:55:21.774068563Z     result = await self._flow_mgr.async_configure(flow_id, data)
2024-04-10T13:55:21.774125901Z              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-10T13:55:21.774182627Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/data_entry_flow.py", line 356, in async_configure
2024-04-10T13:55:21.774243665Z     result = await self._async_configure(flow_id, user_input)
2024-04-10T13:55:21.774300478Z              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-10T13:55:21.774358241Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/data_entry_flow.py", line 402, in _async_configure
2024-04-10T13:55:21.774418079Z     result = await self._async_handle_step(
2024-04-10T13:55:21.774475692Z              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-10T13:55:21.774533205Z   File "/usr/local/lib/python3.12/site-packages/homeassistant/data_entry_flow.py", line 506, in _async_handle_step
2024-04-10T13:55:21.774593293Z     result: _FlowResultT = await getattr(flow, method)(user_input)
2024-04-10T13:55:21.774652131Z                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-04-10T13:55:21.774710919Z   File "/config/custom_components/valetudo_vacuum_camera/config_flow.py", line 227, in async_step_user
2024-04-10T13:55:21.774771257Z     os.mkdir(f"{storage_path}/valetudo_camera")
2024-04-10T13:55:21.774830833Z FileNotFoundError: [Errno 2] No such file or directory: '/run/s6-rc:s6-rc-init:OlgGKB/servicedirs/svc-homeassistant/.storage/valetudo_camera'

[sca075]

Okay at this point please revert to the previous version and wait for next one. The camera will be released next week and I will ask to the core guys as per it looks there's some issues with the basic os function. It is anyhow strange that on 3 instances "supervised" I use on my tests I can't reproduce this issue.

[carefulcomputer]

can you please try on a container ? i have been using docker container and that has this problem.

[sca075]

@carefulcomputer my production environment is in a docker container (supervised) on a server running Debian. I normally test also on this the camera, HA 2024.04.02. No problem on it.

You can use the SSH add on home assistant you can go on cd /config/.storage and there .. as you can see the camera already stored in the past the png with your maps, probably a raw file too if not a zip when you had the logs enabled, this is the background reason to create a folder over there.

In the PI4 running the OS I removed both cameras (integration therefore removed from home assistant) removed the folder and then reinstall the cameras. The folder was not created but the integration did setup both vacuums with no error (as it should be). Having although some strange snapshot issue with the rand256 vacuum currently after that but the other one running the original valetudo works as expected (no folder then store the data in .storage only).

I do not catch fully why those error happens and also why the mkdir or makedirs of OS isn't working..
https://github.com/sca075/valetudo_vacuum_camera/blob/c07205c2a300aef93073981e51afd80026208125/custom_components/valetudo_vacuum_camera/config_flow.py#L221-L235

The if should avoid if the directory .storage "do not exist" to create the new folder over there (you can comment those lines and the initial config process must end without error in your case). I need to check again the python 12 docs probably something wrong on this.. this requires time.. so if you want please go back to the know working version in your case or.. simply create the folder in .storage so that this works also for you.

[carefulcomputer]

is .storage folder supposed to map to a physical folder in home assistant ?

Here is what I did..

1. i shelled into haas docker image.
2. went to /run/s6-rc:s6-rc-init:gEioCK/servicedirs/svc-homeassistant folder
3. there was no .storage folder. so created .storage/valetudo_camera
4. also create a test file to see if it would show up somewhere in mapped docker volume on my machine
5. exited out of docker shell.
6. searched for test file in haas docker volume, but couldn't find it.

However the integration started working.

so it may not be the issue with folder creation, but rather mapping ?
if this integration is supposed to use haas's mounted /config volume, why is it trying to use /run/s6-rc:s6-rc-init:gEioCK/servicedirs/svc-homeassistant/.storage ? should it not use /config/.storage ?

[sca075]

2. went to /run/s6-rc:s6-rc-init:gEioCK/servicedirs/svc-homeassistant/ folder

@carefulcomputer thanks for the update and your time on this.

Interesting.. typically for HA contained is the root /config/.storage and core is /var/lib/hass/.storage in the file explorer you will not see it as it is a reserved folder for HA (os .folder is hidden). Both of you were running the camera so far I understand and there was no issue os.getcwd() function that is used to determinate the working dir of the HA instance (python wise) .. anyhow this is quite an interesting point too #115.. please have a look at it.

Happy to know that both of you are up and running.
@ChrisOboe is too so far I understand you had permission issues and you gave it.. as work around.. and it worked.
@carefulcomputer strange path.. I do not know if this.. correct but if you are in SSH (using the add on) just type cd /config/.storage and ls ..

I can add just some additional logging anyhow from coding point of view I do not see any "bug".. on the several tests I made as in #115 I noticed too some environmental changes during my test..

[sca075]

Moved to discussions for tracking.. I do not close this because it is a potential issue.. thanks you both once again.

[carefulcomputer]

when go inside the container and check /config, it is pointing to right volume mount (external folder where /config is mounted in container). But for some reason os.getcwd() is getting /run/s6-rc:s6-rc-init:gEioCK/servicedirs/

[sca075]

okay we need to investigate this happen, I repeat there was no issue on my installations but we keep it under observation.

[sca075]

hass.config.path(STORAGE_DIR) is the correct way to found the storage folder I will release the fix in the next release ;)

[sca075]

@ChrisOboe and @carefulcomputer
thanks once again for the help and hightailing this issue. The fix has been release on 2024.04.02. Hope you will enjoy the camera and if you wold like it, please star it ;)

[carefulcomputer]

Thank you ! works great now. appreciate you looking into this.

[phrozen77]

Hey,

i just updated to Home Assistant 2024.5.1 (coming from 2024.4.2), updated all HACS integrations (can i somehow check which version i was coming from, because i'm pretty sure i've skipped at least 3-5 versions before updating Valetudo Vacuum Camera to 2024.05.0).

Anyhow, i also seem to face a similar bug - at least, the error is referring to the migration - hence why i post it in here (i may, of course, open a separate issue if you tell me to):

2024-05-05 07:03:54.304 ERROR (MainThread) [homeassistant.config_entries] Error migrating entry <vacuumname> Camera for valetudo_vacuum_camera
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 880, in async_migrate
result = await component.async_migrate_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/valetudo_vacuum_camera/__init__.py", line 181, in async_migrate_entry
new_options = await update_options(old_options, tmp_option)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/valetudo_vacuum_camera/common.py", line 170, in update_options
updated_options[key] = bk_options[key]
~~~~~~~~~~^^^^^
KeyError: 'aspect_ratio'

I'd appreciate it if you could look into that :)

[sca075]

@phrozen77 will do and come back to you as soon.

[sca075]

@phrozen77 mmm, you are correct there is a small issue, will work on this migration problem, but actually as per I do not know what version of the camera you did run before updating it.. would you be so kind to remove the camera, reboot HA, setup the camera again from 0. I would advice you write down the settings of the camera before you do this.

The problem is that in the init.py the config is set to match the value stored in HA. The camera current version of the config_flow is 3.1 and the init.py point directly to this migration process.

I did change the code to check for earlier versions with <= instead of == and also add a try check on the update_options to raise a warning instead to crash and return the old options (will need to do some check for it).

I will publish a fix for this issue in the 2024.05.1 scheduled for later today, so you might wait till later (meaning please downgrade the camera for the time be or just as I wrote earlier re-setup it).
Really sorry for inconvenience.
Sandro

[phrozen77]

but actually as per I do not know what version of the camera you did run before updating it.

Actually, i could check that - if you could tell me if i can look up the version by checking some file or somesuch - i'd go through my backups.

I'd actually prefer that path - downgrade (to any version lower than current or to the version running before?) rather than re-installing it :)

Thank you so much for your time!

Really sorry for inconvenience.

No need to be sorry - i appreciate that you're looking into this, on a sunday no less!

[sca075]

if you could tell me if i can look up the version
In the manifest.json there is the version of the camera you can check there if it is backup. HA normally do not backup the custom_components, this means that it should be in the full_backup if you have any of that. Anyhow in a couple of hours the 2025.05.1 should be available ;)

[phrozen77]

Current:

{
    "domain": "valetudo_vacuum_camera",
    "name": "Valetudo Vacuum Camera",
    "codeowners": ["@sca075"],
    "config_flow": true,
    "dependencies": ["mqtt"],
    "documentation": "https://github.com/sca075/valetudo_vacuum_camera",
    "iot_class": "local_polling",
    "issue_tracker": "https://github.com/sca075/valetudo_vacuum_camera/issues",
    "requirements": [
        "pillow==10.3.0",
        "numpy"
    ],
    "version": "2024.05"
}

A few days ago:

{
    "domain": "valetudo_vacuum_camera",
    "name": "Valetudo Vacuum Camera",
    "codeowners": ["@sca075"],
    "config_flow": true,
    "dependencies": ["mqtt"],
    "documentation": "https://github.com/sca075/valetudo_vacuum_camera",
    "iot_class": "local_polling",
    "issue_tracker": "https://github.com/sca075/valetudo_vacuum_camera/issues",
    "requirements": ["pillow", "numpy", "svgwrite"],
    "version": "v1.5.7.4"
}

[sca075]

@phrozen77 Thanks so I can test also the migration from there 1.5.7.4 (wow quite old :) ) I will set up this on my test enviroment an make sure that all works as expected before I release the 2024.05.1.

Note that HA will have problems with 1.5.7 or 1.6.0 versions due to some deprecation that was introduced in HA.

[sca075]

@phrozen77 you actually raised up a really great issue :) I did reproduced the problem and I'm working on it.
Thanks really much for this :)

[sca075]

@phrozen77... I just removed form the configuration the camera, then add it again and the camera did startup okay off course didn't keep the settings but for instance this work as work around.
I need to study this a little longer (as it isn't easy so far I understand from what tested). Anyhow if I can fix this migration issue on the next release will let you know.

[sca075]

okay got it.. in the next version the migration should work as intended.

Starting point v1.5.7.4

2024-05-05 18:14:01.730 DEBUG (MainThread) [custom_components.valetudo_vacuum_camera] Migrating config entry from version 2.2
2024-05-05 18:14:01.731 DEBUG (MainThread) [custom_components.valetudo_vacuum_camera] {'vacuum_config_entry': 'd88a6c8c473cf1e8c8d5c6cf46a62852'}
2024-05-05 18:14:01.731 DEBUG (MainThread) [custom_components.valetudo_vacuum_camera] Creating the /config/.storage/valetudo_camera path.
2024-05-05 18:14:01.732 DEBUG (MainThread) [custom_components.valetudo_vacuum_camera] Path /config/.storage/valetudo_camera already exist.
2024-05-05 18:14:01.732 DEBUG (MainThread) [custom_components.valetudo_vacuum_camera] {'margins': '100', 'auto_zoom': False, 'get_svg_file': False, 'enable_www_snapshots': True, 'aspect_ratio': 'None', 'zoom_lock_ratio': True, 'vac_status_font': 'custom_components/valetudo_vacuum_camera/utils/fonts/FiraSans.ttf', 'vac_status_size': 50, 'vac_status_position': True, 'offset_top': 0, 'offset_bottom': 0, 'offset_left': 0, 'offset_right': 0}
2024-05-05 18:14:01.732 DEBUG (MainThread) [custom_components.valetudo_vacuum_camera] migration data:{'rotate_image': '0', 'margins': '100', 'aspect_ratio': 'None', 'offset_top': 0, 'offset_bottom': 0, 'offset_left': 0, 'offset_right': 0, 'auto_zoom': False, 'zoom_lock_ratio': True, 'show_vac_status': False, 'vac_status_size': 50, 'vac_status_position': True, 'vac_status_font': 'custom_components/valetudo_vacuum_camera/utils/fonts/FiraSans.ttf', 'get_svg_file': False, 'enable_www_snapshots': True, 'color_charger': [255, 128, 0], 'color_move': [238, 247, 255], 'color_wall': [255, 255, 0], 'color_robot': [255, 255, 204], 'color_go_to': [0, 255, 0], 'color_no_go': [255, 0, 0], 'color_zone_clean': [255, 255, 255], 'color_background': [0, 125, 255], 'color_text': [255, 255, 255], 'alpha_charger': 255.0, 'alpha_move': 255.0, 'alpha_wall': 255.0, 'alpha_robot': 255.0, 'alpha_go_to': 255.0, 'alpha_no_go': 125.0, 'alpha_zone_clean': 125.0, 'alpha_background': 255.0, 'alpha_text': 255.0, 'color_room_0': [135, 206, 250], 'color_room_1': [176, 226, 255], 'color_room_2': [165, 105, 18], 'color_room_3': [164, 211, 238], 'color_room_4': [141, 182, 205], 'color_room_5': [96, 123, 139], 'color_room_6': [224, 255, 255], 'color_room_7': [209, 238, 238], 'color_room_8': [180, 205, 205], 'color_room_9': [122, 139, 139], 'color_room_10': [175, 238, 238], 'color_room_11': [84, 153, 199], 'color_room_12': [133, 193, 233], 'color_room_13': [245, 176, 65], 'color_room_14': [82, 190, 128], 'color_room_15': [72, 201, 176], 'alpha_room_0': 255.0, 'alpha_room_1': 255.0, 'alpha_room_2': 255.0, 'alpha_room_3': 255.0, 'alpha_room_4': 255.0, 'alpha_room_5': 255.0, 'alpha_room_6': 255.0, 'alpha_room_7': 255.0, 'alpha_room_8': 255.0, 'alpha_room_9': 255.0, 'alpha_room_10': 255.0, 'alpha_room_11': 255.0, 'alpha_room_12': 255.0, 'alpha_room_13': 255.0, 'alpha_room_14': 255.0, 'alpha_room_15': 255.0}
2024-05-05 18:14:01.747 INFO (MainThread) [custom_components.valetudo_vacuum_camera] Migration to config entry version successful 3.1

End Point: 2025.05.

@phrozen77 as soon available please update the camera to 2025.05.1 :)

[sca075]

@phrozen77
I did got some results with the rooms names and ID in the colour configuration.

What do you think about?

[phrozen77]

Hey @sca075 - sorry to ask, but i'm not sure what you're asking for? :) i took a quick look at the reconfigure rooms menu, seems to work as expected? fwiw, i never changed a thing in there and went with the defaults.

[sca075]

I would add to the description of the colours the ID and names of the rooms to make more clear what colour will be re-configured. It will take the language selected on the UI and update the translations files accordingly. Anyhow as per I'm having some issue to run this function in the config_flow I will postpone the release of this function to the 2023.05.3 for next week.
Nothing to worry, also keeping the defaults is okay :) Have a nice weekend.

[phrozen77]

Yes, i think some clarification may indeed be needed there, considering that was one of the reasons i didn't mess with the defaults ;)

[sca075]

We will get it on 2025.05.2 that will be released in the next hours ;) I found a solution for the config_flow to run this function when needed.