cbaines/guix-data-service
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix marking jobs as failed when exceptions are raised

Browse source 
The switch away from catch broke this, I obviously still don't quite get how
with-exception-handler works. Therefore, use it twice as that seems to help.
This commit is contained in:
Christopher Baines 2020-03-19 20:42:59 +00:00
parent ded4df6632
commit e0f920bb14
1 changed files with 16 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

6
guix-data-service/jobs/load-new-guix-revision.scm
View file

@ -1592,6 +1592,9 @@ SKIP LOCKED")
(with-time-logging (string-append "loading revision " commit) (with-time-logging (string-append "loading revision " commit)
(setup-logging (setup-logging
id id
(lambda ()
(with-exception-handler
(const #f)
(lambda () (lambda ()
(with-exception-handler (with-exception-handler
(lambda (exn) (lambda (exn)
@ -1606,7 +1609,8 @@ SKIP LOCKED")
(load-new-guix-revision conn (load-new-guix-revision conn
store store
git-repository-id git-repository-id
commit)))))))) commit))))))
#:unwind? #t))))
#t)) #t))
(begin (begin
(record-job-succeeded conn id) (record-job-succeeded conn id)