feat: use sqlite by default #3

2025-09-09T01:34:09Z

Ghost commented

2025-09-09 01:34:09 +00:00

This PR makes the recipe use sqlite by default, since I'm pretty sure it's not possible to disable services that exist in the default compose.yml file. Plus, I think sqlite will be a better default experience and use fewer resources.

Other changes:

Added several more config options
Removed the post-install step, that was fixed in v0.14: https://github.com/writefreely/writefreely/pull/618
Updated to 0.16.0 (coop-cloud-chaos-patchs/docker-writefreely#1 )
Fixed the image name
- Since the image name was broken, this confirms there are no current users, so we don't have to worry about backward-compatibility
Added entrypoint to run key generation, database initialization, and migrations
Moved all data to /data (required coop-cloud-chaos-patchs/docker-writefreely#2)
Fixed the config version env vars not matching the config names
Moved the config to /usr/share/writefreely, because I can't put the config in the data storage dir. This forces us to use the "-c" flag every time we run the binary (really wish the upstream would let us set the config file path with an env var)
Explicitly set the paths to the templates, keys, etc., since we have to use non-default locations for all of them due to volume mounting issues.

Closes #1

This PR makes the recipe use sqlite by default, since I'm pretty sure it's not possible to disable services that exist in the default compose.yml file. Plus, I think sqlite will be a better default experience and use fewer resources. Other changes: - Added several more config options - Removed the post-install step, that was fixed in v0.14: https://github.com/writefreely/writefreely/pull/618 - Updated to 0.16.0 (https://git.coopcloud.tech/coop-cloud-chaos-patchs/docker-writefreely/pulls/1 ) - Fixed the image name - Since the image name was broken, this confirms there are no current users, so we don't have to worry about backward-compatibility - Added entrypoint to run key generation, database initialization, and migrations - Moved all data to `/data` (required https://git.coopcloud.tech/coop-cloud-chaos-patchs/docker-writefreely/pulls/2) - Fixed the config version env vars not matching the config names - Moved the config to /usr/share/writefreely, because I can't put the config in the data storage dir. This forces us to use the "-c" flag every time we run the binary (really wish the upstream would let us set the config file path with an env var) - Explicitly set the paths to the templates, keys, etc., since we have to use non-default locations for all of them due to volume mounting issues. Closes #1

Ghost added 1 commit 2025-09-09 01:34:10 +00:00

feat: use sqlite by default 0eaed4920f

Ghost requested review from decentral1se 2025-09-09 01:34:10 +00:00

Ghost marked the pull request as work in progress 2025-09-09 01:34:22 +00:00

Ghost commented

2025-09-09 01:34:32 +00:00

Need to test

decentral1se approved these changes 2025-09-09 07:40:56 +00:00

decentral1se left a comment

LGTM, nice one! A great update 🎉

Long live sqlite 🫡

LGTM, nice one! A great update 🎉 Long live sqlite 🫡

Ghost added 1 commit 2025-09-09 21:25:06 +00:00

fix: use correct image name af92bb1f7a

Ghost added 3 commits 2025-09-09 22:43:52 +00:00

fix: upgrade mariadb and fully disable by default 57dd7df172

fix: run key generation and db migration at startup 92846b4226

doc: expand README adf19b96e1

Ghost added 1 commit 2025-09-09 23:22:22 +00:00

fix: create volume for db 1e39d1d6c1

Ghost force-pushed default-sqlite from 1e39d1d6c1 to 8e51569ee4

2025-09-10 02:40:52 +00:00

Compare

Ghost commented

2025-09-10 02:43:45 +00:00

Ahh well I can confirm this recipe has zero current users 😅

$ docker pull git.coopcloud.tech/coop-cloud-chaos-patchs/writefreely:v0.15.1
Error response from daemon: manifest unknown

So no need to worry about testing migrations haha

I think this PR is ready now, but I'd want to do another quick round of testing after merging coop-cloud-chaos-patchs/docker-writefreely#2

Ahh well I can confirm this recipe has zero current users 😅 ``` $ docker pull git.coopcloud.tech/coop-cloud-chaos-patchs/writefreely:v0.15.1 Error response from daemon: manifest unknown ``` So no need to worry about testing migrations haha I think this PR is ready now, but I'd want to do another quick round of testing after merging https://git.coopcloud.tech/coop-cloud-chaos-patchs/docker-writefreely/pulls/2

👍 1

Ghost added 1 commit 2025-09-10 22:20:12 +00:00

fix: move config out of data dir 7e03425bc2

Ghost added 1 commit 2025-09-10 22:26:05 +00:00

fix: make config version name match config name 8153d2e557

Ghost force-pushed default-sqlite from 8153d2e557 to cff1b66703

2025-09-10 22:28:59 +00:00

Compare

Ghost added 1 commit 2025-09-10 23:18:59 +00:00

fix: explicitly set path to assets 5d816713a5

Ghost added 1 commit 2025-09-10 23:26:14 +00:00

fix: add config flag to readme command c6cd511f43