Skip to content

Fix DockerFile read-only SQLite error in demos#2055

Merged
mvorisek merged 2 commits intodevelopfrom
fix/dockerfile-sqlite-permissions
May 26, 2023
Merged

Fix DockerFile read-only SQLite error in demos#2055
mvorisek merged 2 commits intodevelopfrom
fix/dockerfile-sqlite-permissions

Conversation

@abbadon1334
Copy link
Copy Markdown
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
Copy Markdown
Member

mvorisek commented May 26, 2023

@abbadon1334 did you test is locally?

@abbadon1334
Copy link
Copy Markdown
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
@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
Copy Markdown
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

Development

Successfully merging this pull request may close these issues.

2 participants