Skip to content

Commit 23e8bd9

Browse files
committed
fix: remove stale runs from redis jobs
Signed-off-by: Keshav Priyadarshi <git@keshav.space>
1 parent aa53acf commit 23e8bd9

2 files changed

Lines changed: 5 additions & 4 deletions

File tree

‎vulnerabilities/models.py‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2219,8 +2219,9 @@ def stop_run(self):
22192219
job_id=str(self.run_id),
22202220
)
22212221
self.set_run_stopped()
2222+
self.delete_run_job()
22222223

2223-
def delete_run(self):
2224+
def delete_run_job(self):
22242225
if job := self.job:
22252226
job.delete()
22262227

@@ -2231,7 +2232,7 @@ def delete(self, *args, **kwargs):
22312232
with suppress(redis.exceptions.ConnectionError, AttributeError):
22322233
self.stop_run()
22332234

2234-
self.delete_run()
2235+
self.delete_run_job()
22352236

22362237
return super().delete(*args, **kwargs)
22372238

‎vulnerabilities/schedules.py‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ def mark_stale_runs():
131131
stale_jobs_count = stale_jobs.count()
132132

133133
for job in stale_jobs.iterator(chunk_size=1000):
134-
job.set_run_staled()
134+
job.stop_run()
135135

136136
log.info(f"Marked {stale_jobs_count} unfinished jobs as stale.")
137137

@@ -151,7 +151,7 @@ def requeue_missing_jobs():
151151
enqueue_run(run=job)
152152
missing_jobs_count += 1
153153
else:
154-
job.set_run_staled()
154+
job.stop_run()
155155

156156
log.info(f"Requeued {missing_jobs_count} missing jobs.")
157157

0 commit comments

Comments
 (0)