diff --git a/app/Jobs/ProcessApps.php b/app/Jobs/ProcessApps.php index 862a4bdb3..8cb16a15f 100644 --- a/app/Jobs/ProcessApps.php +++ b/app/Jobs/ProcessApps.php @@ -21,21 +21,16 @@ class ProcessApps implements ShouldQueue, ShouldBeUnique use Dispatchable, InteractsWithQueue, Queueable, SerializesModels; /** - * Bound total attempts so a failed job stops after three retries across - * worker restarts, independent of the broker's reserved-job recovery path. + * Most failures here are GitHub rate-limit responses; retries inside the + * same window do not help, so a single attempt is enough. */ - public int $tries = 3; - - /** @var array seconds between retries */ - public array $backoff = [30, 60, 120]; - - public int $timeout = 60; + public int $tries = 1; /** - * Expire the ShouldBeUnique lock after 1 hour so a crashed worker does - * not permanently block future ProcessApps dispatches. + * Expire the ShouldBeUnique lock after 10 minutes so a crashed worker + * does not permanently block future ProcessApps dispatches. */ - public int $uniqueFor = 3600; + public int $uniqueFor = 600; /** * Create a new job instance. diff --git a/app/Jobs/UpdateApps.php b/app/Jobs/UpdateApps.php index 7911cc824..118844dab 100644 --- a/app/Jobs/UpdateApps.php +++ b/app/Jobs/UpdateApps.php @@ -19,25 +19,18 @@ class UpdateApps implements ShouldQueue, ShouldBeUnique use Dispatchable, InteractsWithQueue, Queueable, SerializesModels; /** - * Bound total attempts so a failed job stops after three retries across - * worker restarts, independent of the broker's reserved-job recovery path. + * Most failures here are GitHub rate-limit responses; retries inside the + * same window do not help, so a single attempt is enough. The throttle + * loop in handle() means the job is intentionally long-running, so we + * leave $timeout unset and let the operator's worker config govern. */ - public int $tries = 3; - - /** @var array seconds between retries */ - public array $backoff = [30, 60, 120]; + public int $tries = 1; /** - * Per-attempt wall-clock ceiling. Heavy users with many apps may need - * a larger value; 60s covers the typical Heimdall deployment. + * Expire the ShouldBeUnique lock after 10 minutes so a crashed worker + * does not permanently block future UpdateApps dispatches. */ - public int $timeout = 60; - - /** - * Expire the ShouldBeUnique lock after 1 hour so a crashed worker does - * not permanently block future UpdateApps dispatches. - */ - public int $uniqueFor = 3600; + public int $uniqueFor = 600; /** * Create a new job instance.