Skip to content

fix(BackgroundJobs): Adjust intervals and time sensitivities#48600

Merged
provokateurin merged 2 commits intomasterfrom
fix/backgroundjobs/adjust-intervals-time-sensitivities
Oct 8, 2024
Merged

fix(BackgroundJobs): Adjust intervals and time sensitivities#48600
provokateurin merged 2 commits intomasterfrom
fix/backgroundjobs/adjust-intervals-time-sensitivities

Conversation

@provokateurin
Copy link
Copy Markdown
Member

Summary

Some jobs were running a bit too frequent and some were also marked as time sensitive even though their interval was quite high.
There are some more jobs that could probably run less often, but I wasn't sure about them so I left them for now.

Checklist

@provokateurin provokateurin added this to the Nextcloud 31 milestone Oct 7, 2024
@provokateurin provokateurin requested review from a team and removed request for a team October 7, 2024 15:39
@provokateurin provokateurin force-pushed the fix/backgroundjobs/adjust-intervals-time-sensitivities branch from 4e7b273 to 8d297e2 Compare October 7, 2024 15:46
Copy link
Copy Markdown
Member

@st3iny st3iny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DAV, twofactor and preview related changes look good.

Signed-off-by: provokateurin <kate@provokateurin.de>
…ervals

Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin provokateurin force-pushed the fix/backgroundjobs/adjust-intervals-time-sensitivities branch from 1b411a8 to b209508 Compare October 8, 2024 09:27
@provokateurin provokateurin merged commit 771eb1f into master Oct 8, 2024
@provokateurin provokateurin deleted the fix/backgroundjobs/adjust-intervals-time-sensitivities branch October 8, 2024 10:25
@provokateurin
Copy link
Copy Markdown
Member Author

/backport to stable30

@provokateurin
Copy link
Copy Markdown
Member Author

/backport to stable29

@provokateurin
Copy link
Copy Markdown
Member Author

/backport to stable28

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 performance 🚀 technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants