laravel-job-status icon indicating copy to clipboard operation
laravel-job-status copied to clipboard

Status stay in retrying

Open jamalyan opened this issue 5 years ago • 8 comments

Hi there. I am trying to use this great package at first time. But I got one issue. When job get exception, the status not changing to 'failed', but stay 'retrying'. In DefaultEventManager I have changed exceptionOccurred method functionality to get what I need.

'status' => ($event->job->attempts() == $event->job->maxTries()) ? $this->getEntity()::STATUS_FAILED : $this->getEntity()::STATUS_RETRYING,

jamalyan avatar Apr 05 '20 15:04 jamalyan

Just we verify we experienced the same one.

sstankov avatar Jul 06 '20 18:07 sstankov

Despite the fix I am still getting this issue - if the job fails, status stay as 'retrying', although the job is moved to the failed_jobs table by Laravel and there are no more retries. I think this is also related to #63.

o15a3d4l11s2 avatar May 10 '21 15:05 o15a3d4l11s2

Despite the fix I am still getting this issue - if the job fails, status stay as 'retrying', although the job is moved to the failed_jobs table by Laravel and there are no more retries. I think this is also related to #63.

Same issue here. Any update on this?

To add, this gets logged in the failed jobs table:

Illuminate\Queue\MaxAttemptsExceededException: Modules\Platform\Jobs\XXX\XXXJob has been attempted too many times or run too long. The job may have previously timed out.

'Causer' of the exception is using the WithoutOverlapping middleware

mylemans avatar Oct 18 '21 10:10 mylemans

Having the same problem here, made a PR https://github.com/imTigger/laravel-job-status/pull/72

JuanS avatar Dec 12 '21 16:12 JuanS

@imTigger Any idea when you'll publish a new release with this fix?

mylemans avatar May 17 '22 06:05 mylemans

@mylemans what can we do do get the change in a new release?

MathiasWeisheit avatar Sep 09 '22 05:09 MathiasWeisheit

@mylemans what can we do do get the change in a new release?

Why are you asking me? ;D

mylemans avatar Sep 09 '22 06:09 mylemans

That's right! I already searched for forks with more engagement.

MathiasWeisheit avatar Sep 09 '22 07:09 MathiasWeisheit