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

stats.db cannot work with Docker #44

Open
Helvio88 opened this issue Sep 7, 2023 · 1 comment
Open

stats.db cannot work with Docker #44

Helvio88 opened this issue Sep 7, 2023 · 1 comment
Labels
bug Something isn't working wontfix This will not be worked on

Comments

@Helvio88
Copy link
Contributor

Helvio88 commented Sep 7, 2023

From my analysis, the stats.db file cannot exist in a Docker container persistently. Since the file is created in the app’s root folder, it can only be extracted from a running environment and mapped, making a consistent Docker deployment not perfect.

A suggestion would be to create a separate folder (like ./data) and store app files in this folder, which could be created automatically by code and then mapped as a Docker volume, containing config.yaml and stats.db.

Taking a step further, and optionally (perhaps even a separate future issue), all configurations could be set by environment vars. This is merely hypothetical discussion as of now.

@DblK
Copy link
Owner

DblK commented Sep 7, 2023

The stats.db will be removed soon once I finished the pocketbase integration.
The stats will be handled by pocketbase itself and this will ease the display on the ui.

Pocketbase handle all data within a directory and migrations too.
I am not satisfied by the actual implementation so, once I moved on it will be cleaner (I hope).

@DblK DblK added bug Something isn't working wontfix This will not be worked on labels Sep 7, 2023
@DblK DblK linked a pull request Sep 9, 2023 that will close this issue
@Helvio88 Helvio88 mentioned this issue Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working wontfix This will not be worked on
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants