branch master updated: Better handle cuirass builds with a stoptime of 0

[Christopher Baines]

This is an automated email from the git hooks/post-receive script.

cbaines pushed a commit to branch master
in repository data-service.

The following commit(s) were added to refs/heads/master by this push:
     new a21d3d3  Better handle cuirass builds with a stoptime of 0
a21d3d3 is described below

commit a21d3d3cf7ea197b9c7efa4150e8b637154202ab
Author: Christopher Baines <address@hidden>
AuthorDate: Wed May 6 23:09:05 2020 +0100

    Better handle cuirass builds with a stoptime of 0
    
    Use the timestamp instead for the status. Previously no status was being
    associated with these builds.
---
 guix-data-service/builds.scm | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/guix-data-service/builds.scm b/guix-data-service/builds.scm
index 3182a07..2c37885 100644
--- a/guix-data-service/builds.scm
+++ b/guix-data-service/builds.scm
@@ -226,9 +226,11 @@ WHERE derivation_output_details.path = $1"
            (when (and (< 0 starttime)
                       (not (member "started" existing-status-entries)))
              (list starttime "started"))
-           (when (and (< 0 stoptime)
-                      (not (member status-string existing-status-entries)))
-             (list stoptime status-string)))))))
+           (when (not (member status-string existing-status-entries))
+             (list (if (< 0 stoptime)
+                       timestamp
+                       stoptime)
+                   status-string)))))))
 
 (define (process-pending-builds conn build-server-id url)
   (for-each