vbatts/quay
Archived
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Have the security worker remove failed notifications from Clair

Browse source
This commit is contained in:
Joseph Schorr 2016-12-05 19:08:52 -05:00
parent 97d150e281
commit 9f0ce7c634
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
workers/security_notification_worker.py
View file

@ -39,8 +39,11 @@ class SecurityNotificationWorker(QueueWorker):
if should_retry: if should_retry:
raise JobException() raise JobException()
else: else:
# Return to mark the job as "complete", as we'll never be able to finish it. # Remove the job from the API.
logger.error('Failed to handle security notification %s', notification_name) logger.error('Failed to handle security notification %s', notification_name)
secscan_api.mark_notification_read(notification_name)
# Return to mark the job as "complete", as we'll never be able to finish it.
return False return False
self.extend_processing(_PROCESSING_SECONDS, json.dumps(data)) self.extend_processing(_PROCESSING_SECONDS, json.dumps(data))