-
-
Notifications
You must be signed in to change notification settings - Fork 536
[12.0][IMP] queue_job: track error in chatter #577
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[12.0][IMP] queue_job: track error in chatter #577
Conversation
b0e1a0f to
9715565
Compare
|
I second this, db size should not be an issue with the autovacuum and exception info is indeed quite important |
9715565 to
7b7f2f5
Compare
etobella
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Technically LGTM
simahawk
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LG. Next time pls, move the pre-commit update to another PR 🙏
|
This PR has the |
|
/ocabot merge patch |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at 3ec3e8f. Thanks a lot for contributing to OCA. ❤️ |
When job fails, it shows in chatter:
Later, the job does a retry. If the retry succeeds, the error is deleted. If not succeeds but the error is different, the error changes. In both cases, you cannot know what was the first error. Let's put in the chatter to have better history.