coop-cloud/outline
coop-cloud
/
outline
generated from coop-cloud/example
1
0
Fork 3
Code Issues 7 Pull Requests Projects Releases Wiki Activity

fix: always migrate before start #11

Merged
3wordchant merged 2 commits from arso/outline:dev into main 2022-08-04 14:48:25 +00:00
Conversation 2 Commits 2 Files Changed 2 +2 -1
Frando commented 2022-08-03 20:28:32 +00:00
Member
Copy Link

Hi,
the current recipe failed to start for me. The app container kept dying in a loop, with errors about missing tables. Because both app run and app cmd require a running container, I couldn't run the migrate command that is mentioned in the README. A simple fix is to just always run the migrate command before starting. With this, the recipe works great for me.
Thanks,
Frando

Hi, the current recipe failed to start for me. The app container kept dying in a loop, with errors about missing tables. Because both `app run` and `app cmd` require a running container, I couldn't run the migrate command that is mentioned in the README. A simple fix is to just always run the migrate command before starting. With this, the recipe works great for me. Thanks, Frando
Frando added 2 commits 2022-08-03 20:28:34 +00:00
Frando commented 2022-08-03 20:32:41 +00:00
Author
Member
Copy Link

If running (potentially destructive) migrate commands on each start is considered bad practice, it could be toggled via an enironment variable maybe instead. I guess the migrate command can make downgrades impossible, so maybe it would be better to have it optional.

Is there a good way on how to check if the container is being started for the first time, to always run the migrate command in that case? Or it could check if the DB is empty.

If running (potentially destructive) migrate commands on each start is considered bad practice, it could be toggled via an enironment variable maybe instead. I guess the migrate command can make downgrades impossible, so maybe it would be better to have it optional. Is there a good way on how to check if the container is being started for the first time, to always run the migrate command in that case? Or it could check if the DB is empty.
decentral1se approved these changes 2022-08-04 09:33:22 +00:00
decentral1se left a comment
Owner
Copy Link

LGTM, let's try it! Also, would be good to patch https://git.coopcloud.tech/coop-cloud/outline#post-deploy-migration explaining that it is done for you now.

LGTM, let's try it! Also, would be good to patch https://git.coopcloud.tech/coop-cloud/outline#post-deploy-migration explaining that it is done for you now.
3wordchant merged commit 7685fe6abf into main 2022-08-04 14:48:25 +00:00
3wordchant commented 2022-08-04 14:49:05 +00:00
Owner
Copy Link

@decentral1se I agree about docs, I also observe running cmd app migrate should be a no-op. Optimistically merged

@decentral1se I agree about docs, I also observe running `cmd app migrate` should be a no-op. Optimistically merged
Sign in to join this conversation.
Reviewers
No reviewers
decentral1se
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: coop-cloud/outline#11
No description provided.
Delete Branch "arso/outline:dev"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?