webhook-handler: ignore install times for failed jobs

spack · Dec 18, 2024 · 1856ff4 · 1856ff4
1 parent d5df819
commit 1856ff4
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/.github/workflows/custom_docker_builds.yml b/.github/workflows/custom_docker_builds.yml
@@ -42,7 +42,7 @@ jobs:
           - docker-image: ./images/cache-indexer
             image-tags: ghcr.io/spack/cache-indexer:0.0.3
           - docker-image: ./analytics
-            image-tags: ghcr.io/spack/django:0.3.27
+            image-tags: ghcr.io/spack/django:0.3.28
           - docker-image: ./images/ci-prune-buildcache
             image-tags: ghcr.io/spack/ci-prune-buildcache:0.0.4
           - docker-image: ./images/protected-publish

diff --git a/analytics/analytics/job_processor/__init__.py b/analytics/analytics/job_processor/__init__.py
@@ -140,5 +140,6 @@ def process_job(job_input_data_json: str):
 
     # Create build timing facts in a separate transaction, in case this fails
     with transaction.atomic():
-        if job.job.job_type == JobDataDimension.JobType.BUILD:
+        job_data = job.job
+        if job_data.job_type == JobDataDimension.JobType.BUILD and job_data.status == 'success':
             create_build_timing_facts(job_fact=job, gljob=gl_job)
diff --git a/k8s/production/custom/webhook-handler/deployments.yaml b/k8s/production/custom/webhook-handler/deployments.yaml
@@ -23,7 +23,7 @@ spec:
       serviceAccountName: webhook-handler
       containers:
         - name: webhook-handler
-          image: ghcr.io/spack/django:0.3.27
+          image: ghcr.io/spack/django:0.3.28
           imagePullPolicy: Always
           resources:
             requests:
@@ -146,7 +146,7 @@ spec:
       serviceAccountName: webhook-handler
       containers:
         - name: webhook-handler-worker
-          image: ghcr.io/spack/django:0.3.27
+          image: ghcr.io/spack/django:0.3.28
           command:
             [
               "celery",