Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Sep 9, 2025

Backport of PR #52825

Signed-off-by: Louis Chemineau <louis@chmn.me>
…lel run

- Follow-up of #51600

The original PR introduced the possibility to continue an `ExpireTrash` job by saving the offset. This was to prevent having to start over the whole user list when the job crashed or was killed.

But on big instances, one process is not enough to go through all the users in a timely manner. Supporting parallel run allows covering more ground faster.

This PR introduced this possibility. We are now storing the offset right away to allow another parallel job to pick up the task at that point. We are arbitrarily cutting the user list in chunk of 10 to not drastically overflow the 30 minutes time limit.

Signed-off-by: Louis Chemineau <louis@chmn.me>
@backportbot backportbot bot requested a review from a team as a code owner September 9, 2025 08:27
@backportbot backportbot bot requested review from artonge, come-nc, icewind1991, nfebe, provokateurin and salmart-dev and removed request for a team September 9, 2025 08:27
@backportbot backportbot bot added enhancement 3. to review Waiting for reviews feature: trashbin php Pull requests that update Php code ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) labels Sep 9, 2025
@backportbot backportbot bot added this to the Nextcloud 32 milestone Sep 9, 2025
@AndyScherzinger AndyScherzinger merged commit ba55d7e into stable32 Sep 9, 2025
202 of 205 checks passed
@AndyScherzinger AndyScherzinger deleted the backport/52825/stable32 branch September 9, 2025 11:13
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
@blizzz blizzz mentioned this pull request Oct 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement feature: trashbin php Pull requests that update Php code ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants