Fix timestamp parsing (#47)

* use gitlab-supplied format instead of ISO 8601

* change gitlab timestamp format to be consistent with webhook response in tests

gantry/models/job.py

@@ -1,5 +1,6 @@
 import re
-from datetime import datetime
+
+from gantry.util.time import webhook_timestamp
 
 
 class Job:
@@ -17,9 +18,10 @@ def __init__(
         self.gl_id = gl_id
         # handle jobs that haven't started or finished
         if start:
-            self.start = datetime.fromisoformat(start).timestamp()
+            self.start = webhook_timestamp(start)
         if end:
-            self.end = datetime.fromisoformat(end).timestamp()
+            self.end = webhook_timestamp(end)
+
         self.ref = ref
 
     @property

gantry/tests/defs/collection.py

@@ -12,8 +12,8 @@
     "build_status": "success",
     "build_name": "[email protected] /jcchwaj %[email protected] arch=linux-ubuntu20.04-x86_64_v3 E4S",
     "build_id": 9892514,  # not used in testing unless it already exists in the db
-    "build_started_at": "2024-01-24T17:24:06.000Z",
-    "build_finished_at": "2024-01-24T17:47:00.000Z",
+    "build_started_at": "2024-01-24 17:24:06 UTC",
+    "build_finished_at": "2024-01-24 17:47:00 UTC",
     "ref": "pr42264_bugfix/mathomp4/hdf5-appleclang15",
     "runner": {"description": "aws"},
 }

gantry/util/time.py

@@ -0,0 +1,15 @@
+import datetime
+
+
+def webhook_timestamp(dt: str) -> float:
+    """Converts a gitlab webhook datetime to a unix timestamp."""
+    # gitlab sends dates in 2021-02-23 02:41:37 UTC format
+    # documentation says they use iso 8601, but they don't consistently apply it
+    # https://docs.gitlab.com/ee/user/project/integrations/webhook_events.html#job-events
+    GITLAB_DATETIME_FORMAT = "%Y-%m-%d %H:%M:%S %Z"
+    # strptime doesn't tag with timezone by default
+    return (
+        datetime.datetime.strptime(dt, GITLAB_DATETIME_FORMAT)
+        .replace(tzinfo=datetime.timezone.utc)
+        .timestamp()
+    )