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

Fix DockerFile read-only SQLite error in demos #2055

Merged
merged 2 commits into from
May 26, 2023

Conversation

abbadon1334
Copy link
Collaborator

@abbadon1334 abbadon1334 commented May 26, 2023

I have been running the command directly on the deploy server. However, I believe it is correct to merge this PR and test the usual deployment flow via webhook.

As pointed out by @mvorisek, the root cause of the issue lies in incorrect permissions. To resolve this, it is essential to set the correct owner for the folder where the SQLite database is located. I hope this adjustment will effectively address the problem.

@abbadon1334 abbadon1334 requested a review from mvorisek May 26, 2023 20:19
@mvorisek
Copy link
Member

mvorisek commented May 26, 2023

@abbadon1334 did you test is locally?

@abbadon1334
Copy link
Collaborator Author

i tested directly on deploy server and it works, but to be sure i think the best is to trigger the webhook

@mvorisek mvorisek changed the title Fix: Dockerfile error SQLite read-only database Fix: DockerFile read-only SQLite error in demos May 26, 2023
@mvorisek mvorisek changed the title Fix: DockerFile read-only SQLite error in demos Fix DockerFile read-only SQLite error in demos May 26, 2023
@mvorisek mvorisek merged commit dfd2bf3 into develop May 26, 2023
43 of 44 checks passed
@mvorisek mvorisek deleted the fix/dockerfile-sqlite-permissions branch May 26, 2023 20:29
@mvorisek mvorisek removed their request for review May 26, 2023 20:30
@mvorisek
Copy link
Member

mvorisek commented May 26, 2023

The read-only error is gone. Thank you!

Please fix also cert renew of https://dev.atk4.org/ to be done via cron/automatically.

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.

None yet

2 participants