-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Docker image does not distiguish data and config folders #18794
Comments
To my understanding, the config file needs to be writable so that configuration via the frontend works.
Any source on that? And why does that matter? Sounds like just an opinion to me |
Well, any best-practise could be considered 'an opinion' - but i'd argue that in most OS' programs are generally pushed to split those. |
A config file the program has to write to sounds like an unusual design choice to me. Not inherently a bad one, but for me an non cloud friendly one. For example, I like to reset my systems by deleting the data, while keeping the settings - similar to what MS started to offer for soft resetting Windows and what docker containers generally do on recreation (unless you specified to keep the data at a specific location via the volumes). |
This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days |
This is very disappointing. |
I'd be thankful if i could get any reason for why this was closed. If it is not planned, i'm sure you have a good reason not to, i'd be interested to hear about it. |
What happened?
Splitting data and config from each other is widely considered a best practice. However, for this /app/data/ has to be used not for config files.
What did you expect to happen?
Mounting possible with data as rw and config as ro.
But it is required to ensure, the start script does not copy stuff around (especially not only in the initial case), so if the config is changed afterwards, the change is reflected.
EDIT: #4697 includes a similar discussion.
Zigbee2MQTT version
latest
The text was updated successfully, but these errors were encountered: