coop-cloud/gitea
4
2
Fork 5
Code Issues 6 Pull Requests 4 Releases Wiki Activity

change deploy logic to stop-first instead of start-first #53

Open
notplants wants to merge 2 commits from stop-first into master
pull from: stop-first
merge into: coop-cloud:master
Conversation 2 Commits 2 Files Changed 1 +2 -2
notplants commented 2026-05-18 16:57:57 +00:00
Owner
Copy Link

I believe that this change fixes the issue described here: #52

I believe doing stop-first instead of start-first causes the old container to be stopped before the new one is started (see: https://docs.docker.com/reference/compose-file/deploy/#update_config).

start-first allows for zero-downtime updates, but can lead to resource contention between old and new containers (which is maybe why we were seeing unable to lock level db at /var/lib/gitea/data/queues/common: resource temporarily unavailable in the error log in the issue mentioned).

not totally sure about this, but the theory makes sense to me, and the app was able to successfully deploy after the change

I believe that this change fixes the issue described here: https://git.coopcloud.tech/coop-cloud/gitea/issues/52 I believe doing `stop-first` instead of `start-first` causes the old container to be stopped before the new one is started (see: https://docs.docker.com/reference/compose-file/deploy/#update_config). `start-first` allows for zero-downtime updates, but can lead to resource contention between old and new containers (which is maybe why we were seeing `unable to lock level db at /var/lib/gitea/data/queues/common: resource temporarily unavailable` in the error log in the issue mentioned). not totally sure about this, but the theory makes sense to me, and the app was able to successfully deploy after the change
notplants added 1 commit 2026-05-18 16:57:57 +00:00
change deploy logic to stop-first instead of start-first c63e1cdd04
notplants added 1 commit 2026-05-18 16:59:20 +00:00
bump version number to 3.5.3 e7c4999f78
notplants commented 2026-05-18 17:02:53 +00:00
Author
Owner
Copy Link

cc recent committers if you have any thoughts or extra knowledge about this @3wordchant @fauno @p4u1

cc recent committers if you have any thoughts or extra knowledge about this @3wordchant @fauno @p4u1
fauno commented 2026-05-19 13:43:04 +00:00
Owner
Copy Link

i just tested it with forgejo and it works! i can see in the logs that the container receives the shutdown signal and then it starts the new container. i didn't even had to restart abra app logs

i just tested it with forgejo and it works! i can see in the logs that the container receives the shutdown signal and then it starts the new container. i didn't even had to restart abra app logs
👍 1
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin stop-first:stop-first
git checkout stop-first
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
3wordchant aadil abra-bot ammaratef45 amras Apfelwurm appletalk arjan basebuilder BornDeleuze Brooke carla cas codegod100 coopcloud cyrnel decentral1se dede devydave fauno flancian Frando iexos jade javielico jjsfunhouse jmakdah2 joe-irving kawaiipunk knoflook kolaente lambdabundesverband linnealovespie marlon mayel mirsal moosemower moritz nicksellen notplants oxaliq p4u1 pau pharaohgraphy PhiNatalie renovate-bot ripclap rix rscmbbng sef simon sixsmith stevensting tobias trav val vaznasty virtualboys wolcen wykwit xynosis yksflip
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coop-cloud/gitea#53
No description provided.
Delete Branch "stop-first"

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?