freakent / dbus-mqtt-devices

A driver for Victron VenusOS GX devices to support use of dbus-mqtt/flashmq-mqtt
MIT License
111 stars 24 forks source link

Does set up need to be re-run after a 2.90 Venus OS/CCGX firmware update #16

Closed gharshaw closed 1 year ago

gharshaw commented 2 years ago

@freakent Hi Martin, Looks like Victron has released 2.90~3 as a release candidate. Will I need to rerun setup-ccgx.sh if I do a firmware update?

Regards, Gene

freakent commented 2 years ago

No.

edit: that was incorrect assumption by me, YES you do need to run setup again.

freakent commented 2 years ago

At least, I don't think so know, until I (or someone else) test it 😀

freakent commented 2 years ago

The real concern is whether anything has changed in 2.90 to affect compatibility with this driver. We should leave this issue open to track experience with upgrades to 2.90.

gharshaw commented 2 years ago

@freakent Martin, After updating CCGX to new Firmware 2.90 ~6, and as I kinda anticipated, PIP and yaml are not included in the new firmware images. @40000000627a715f16674484 File "/data/drivers/dbus-mqtt-devices-0.4.1/device_manager.py", line 17, in @40000000627a715f16674484 import yaml @40000000627a715f16674484 ModuleNotFoundError: No module named 'yaml'

I updated to dbus-mqtt-devices-0.5.1 and re-ran setup-ccgx.sh to reinstall pip and yaml successfully.

I then did a bit of cleanup :

  1. deleted "ln -s /data/drivers/dbus-mqtt-devices-0.4.1/bin/service /service/dbus-mqtt-devices" from the rc.local file
  2. deleted the /data/drivers/dbus-mqtt-devices-0.4.1 folder

I can confirm custom names working as well 👍 Successful install on 2.90 ~6

Regards, Gene

freakent commented 2 years ago

Ah ok, I didn't realise that an OS upgrade would clear any additional installed python libraries (such as YAML). Although it does make sense. So you are correct to run the setup script again. I am beginning to wish I hadn't chosen YAML for the config file format!

It might be possible to actually bundle the YAML library into the dbus-Mqtt-devices distribution then all these troubles would go away.