Skip to content

Update windows.md#2640

Open
spacemanspiff2007 wants to merge 1 commit into
openhab:mainfrom
spacemanspiff2007:patch-3
Open

Update windows.md#2640
spacemanspiff2007 wants to merge 1 commit into
openhab:mainfrom
spacemanspiff2007:patch-3

Conversation

@spacemanspiff2007

@spacemanspiff2007 spacemanspiff2007 commented Jan 11, 2026

Copy link
Copy Markdown
Contributor

As requested by @florian-h05 .
The information is incorrect and should not be there.

Signed-off-by: spacemanspiff2007 <10754716+spacemanspiff2007@users.noreply.github.com>
@netlify

netlify Bot commented Jan 11, 2026

Copy link
Copy Markdown

Thanks for your pull request to the openHAB documentation! The result can be previewed at the URL below (this comment and the preview will be updated if you add more commits).

Built without sensitive environment variables

Name Link
🔨 Latest commit 712e9d9
🔍 Latest deploy log https://app.netlify.com/projects/openhab-docs-preview/deploys/696330f129d01900081fdb84
😎 Deploy Preview https://deploy-preview-2640--openhab-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@florian-h05 florian-h05 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would rather remove the sentence even across openHAB versions and add in „across the same openHAB version“.

@spacemanspiff2007

Copy link
Copy Markdown
Contributor Author

I would rather remove the sentence even across openHAB versions and add in „across the same openHAB version“.

Are you 100% sure that restoring an old version version of the cache or tmp folder does not cause any error or issues?
Wouldn't it be better to tell the user to exclude the cache directories?

@florian-h05

Copy link
Copy Markdown
Contributor

I think the cache and tmp folder are only included if the full option is set when creating a backup.
But to be on the safe side, we should recommend to remote the cache and tmp directories, WDYT?

@stefan-hoehn

Copy link
Copy Markdown
Contributor

Sorry, I approved but only then saw @florian-h05 's comment. @spacemanspiff2007 Anything you want to change?

@stefan-hoehn stefan-hoehn added this to the 5.2 milestone Jan 24, 2026
@spacemanspiff2007

Copy link
Copy Markdown
Contributor Author

I think the best would be to add an information to exclude the following folders/files from a backup

  • cache
  • config
  • tmp
  • logs
  • etc except for etc/log4j2.xml
  • Californium3.properties
  • marketplace <-- Not sure about that one

@rkoshak

rkoshak commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

config

This absolutely must be part of the backup. That's where all the settings are saved made through the REST API which do not go in the JSONDB. This includes which add-ons are installed, add-on specific configs, regional settings, ephemeris, etc.

etc except for etc/log4j2.xml

There are more files in etc that the user may have changed. For example keys.properties if they changed the password for the karaf console. The keystore, host.key, host.key.pub etc are all files the end user may have modified and which should be included in a backup.

marketplace <-- Not sure about that one

That should be excluded.

@florian-h05 florian-h05 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I did not mean to remove this section, but correct it => it does not work across openHAB versions.

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.

4 participants