-
Notifications
You must be signed in to change notification settings - Fork 305
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
[Feature Request] Deprecate storing config path in database #5101
Comments
webtrees has a folder for system data webtrees also needs to store user data such as media files, GEDCOM files. By default this also goes in The roadmap in webtrees is to allow the user data to be stored in other locations. The internal changes have been made -we use the flysystem abstraction layer internally and can easily store data in cloud storage, remote filesystems, etc. The only thing outstanding is the configuration. Instead of So, I'm not sure that moving this option to config.ini.php fits in with this roadmap. If I understand correctly, you are using symlinks to some other storage. Might it be better to specify the desired location directly. This could be done with a (very) simple module that selects whatever local filesystem you like - including reading a location from config.ini.php directly. On a related note, I have just started writing a CLI interface for configuring webtrees. Creating users/trees, updating config, etc. Would this be another solution for you - a CLI script that updates this setting? |
Hi, thanks for this feedback and clear explanations. Regarding the homeassistant addon, it is globally a shell around NathanVaughn's container. The key difference is that most options are defined through the HA interface (base url ; ssl certificates ...) and a serie of scripts propagates those options to the webtree app itself (for example through environment variables ; direct scripts modification using sed ; symlinks...). This allows consistency with other addons way of working to simplify user experience, and use HA's systems for backups ; mysql availability... As HA doesn't allow to use volumes mapping, I've symlinked /data and /modules_v4 to a folder that is accessible for the user. Up until now it works very fine - but indeed arsernyspb has identified that the mapping of the data_location is not very robust. A cli would be great to allow to modify by script some options in a much more robust manner. I have other addons working with this type of logic, such as one based on Nextcloud. Btw congrats on your incredible app. I'm also a geneaology enthusiast and have been using it for a long time. |
There is already a CLI interface - though it's not documented and the syntax/commands are still liable to change. Commit 0683965 adds a
|
I know this issue is marked as closed, but I would love to hear about it when this CLI is ready. I (try to) maintain one of the two most popular Docker images for webtrees, and I've created a pretty gnarly Python script to automate the setup wizard and configure webtrees on startup. An officially supported CLI would do wonders for making webtrees easier to Dockerize. |
Some CLI commands are pretty much complete:
Others are still in progress:
I hope to have these ready for the next release. If you want to experiment with the latest dev code, use |
Hi @fisharebest ,
What would be the impact of deprecating the
wt_site_setting
'sINDEX_DIRECTORY
on schema in upcoming releases, and e.g. moving it strictly toconfig.ini.php
?It would've allowed a more stateless architecture with datastore decoupled that would allow easier underlying migrations with scripts that move files (e.g. from local storage to a NAS or FUSE-based object store) without the need to hook back in to the DB and do updates like:
We're discussing this as part of issues/1707 with @alexbelgium who is doing amazing job for both webtrees and Home Assistant communities: he is maintaining and innovating on Hassio Webtrees add-on. It's a fantastic wrapper on top of dockerized webtrees from @NathanVaughn 's builds, which from your main.
Making webtrees maintainance more decoupled with focal area in
config.ini.php
only would've opened up a very good avenue for Hassio owners to manage add-on upgrades with minimal maintenance.The text was updated successfully, but these errors were encountered: