Skip to content

Add internal_error signal#6027

Closed
ldragan wants to merge 1 commit into
celery:masterfrom
ldragan:master
Closed

Add internal_error signal#6027
ldragan wants to merge 1 commit into
celery:masterfrom
ldragan:master

Conversation

@ldragan

@ldragan ldragan commented Apr 7, 2020

Copy link
Copy Markdown
Contributor

Note: Before submitting this pull request, please review our contributing
guidelines
.

Description

There is no special signal for an out of body error which can be the
result of a bad result backend.

We want to add a new signal task_internal_error which will call the signal handler in case of an out of body error.

There is no special signal for an out of body error which can be the
result of a bad result backend.
Comment thread celery/app/trace.py
return trace_ok_t(report_internal_error(task, exc), None, 0.0, None)


def _signal_internal_error(task, uuid, args, kwargs, request, exc):

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this need proper test coverage. and entry to changelog

@thedrow

thedrow commented Apr 19, 2020

Copy link
Copy Markdown
Contributor

It's a shame you didn't use a different branch other than master.
I can't push the required changes directly to your branch because of that.

Please create a branch on your fork when contributing in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants