Feat: Better reporting if something is wrong with taskprocessing#59958
Merged
marcelklehr merged 10 commits intomasterfrom Apr 29, 2026
Merged
Feat: Better reporting if something is wrong with taskprocessing#59958marcelklehr merged 10 commits intomasterfrom
marcelklehr merged 10 commits intomasterfrom
Conversation
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
marcelklehr
requested review from
janepie,
julien-nc,
kyteinsky,
lukasdotcom and
oleksandr-nc
marcelklehr
requested review from
Altahrim,
leftybournes,
nfebe and
sorbaugh
and removed request for
a team
marcelklehr
changed the title
julien-nc
reviewed
Apr 28, 2026
Member
julien-nc
left a comment
julien-nc
left a comment
There was a problem hiding this comment.
I don't see the check results. I tried with no tasks recently scheduled and after having scheduled one.
There should always be one, right?
Member
Author
Ah, it's not registered. |
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Co-authored-by: Julien Veyssier <julien-nc@posteo.net> Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
julien-nc
requested changes
Apr 28, 2026
…as never run Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
julien-nc
approved these changes
Apr 28, 2026
oleksandr-nc
approved these changes
Apr 28, 2026
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Member
Author
|
/backport to stable33 |
Member
Author
|
/backport to stable32 |
This was referenced Apr 29, 2026
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
It turns out that the occ taskprocessing:worker command does not run when the instance is in maintenance mode and then usually just stops (after a few failed restarts). This adds a setup check to make the admin aware that the worker is not running, iff there have been scheduled tasks in the last 7 days (to make sure we don't warn if the feature is not used).
The second commit increases the sensitivity threshold for the other taskprocessing setup checks from 20% to 10% to make them warn earlier.
Checklist
3. to review, feature component)stable32)AI (if applicable)