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

Update userdata using common openHAB update files #239

Merged
merged 1 commit into from
Aug 4, 2019

Conversation

wborn
Copy link
Member

@wborn wborn commented Aug 3, 2019

Reworks the update process to update the userdata in the container using the update.lst and userdata_sysfiles.lst files (provided by openhab-distro).
The update script has been modified for use in the container.

This adds the following new functionality when updating between versions:

  • show update notes and warnings during the update (based on update.lst)
  • execute the update pre/post commands (based on update.lst)
  • replace userdata system files with newer versions (based on userdata_sysfiles.lst)
  • update KAR files in addons dir by downloading newer versions
  • log the update output in update.log

Other improvements are:

  • declare and use common openHAB environment variables in scripts (OPENHAB_BACKUPS, OPENHAB_CONF, OPENHAB_HOME, OPENHAB_LOGDIR, OPENHAB_USERDATA)
  • initialize empty volumes using a reusable method
  • move all default volume data into the "dist" directory
  • remove Windows PowerShell scripts from container images

I've tested many upgrade scenarios with the Debian and Alpine images. This change also allows for updating the classnames of rules created in Paper UI (openhab/openhab-core#674) for 2.5.0.


This change is Reviewable

Reworks the update process to update the userdata in the container using the update.lst and userdata_sysfiles.lst files (provided by openhab-distro).
The update script has been modified for use in the container.

This adds the following new functionality when updating between versions:
* show update notes and warnings during the update (based on update.lst)
* execute the update pre/post commands (based on update.lst)
* replace userdata system files with newer versions (based on userdata_sysfiles.lst)
* update KAR files in addons dir by downloading newer versions
* log the update output in update.log

Other improvements are:
* declare and use common openHAB environment variables in scripts (OPENHAB_BACKUPS, OPENHAB_CONF, OPENHAB_HOME, OPENHAB_LOGDIR, OPENHAB_USERDATA)
* initialize empty volumes using a reusable method
* move all default volume data into the "dist" directory
* remove Windows PowerShell scripts from container images

Signed-off-by: Wouter Born <github@maindrain.net>
Copy link
Member

@cniweb cniweb left a comment

Choose a reason for hiding this comment

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

LGTM

@cniweb cniweb merged commit ab1d7ea into openhab:master Aug 4, 2019
@wborn wborn deleted the use-common-update-files branch August 4, 2019 18:54
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.

None yet

2 participants