Skip to content

Commit 2072d47

Browse files
authored
Fix Telco status (#277)
As part of the REVAMP work, on March 25th, Telco added an explicit "status" field to the job record to help the CPT dashboard with filtering. However, this field wasn't retrofitted to older jobs, and the dashboard code didn't properly handle older records with the field missing. This simply changes the `["status"]` reference to `.get("status")`, providing a default `other` value for jobs with a missing status, which is consistent with the `search.py` generation of values for the filter dropdowns.
1 parent 500a23d commit 2072d47

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

backend/app/api/v1/commons/telco.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ async def getData(
8080
+ "/"
8181
+ str(test_data["cluster_artifacts"]["ref"]["jenkins_build"]),
8282
"jobStatus": constants.JOB_STATUS_MAP.get(
83-
test_data["status"], test_data["status"]
83+
test_data.get("status"), test_data.get("status", "other")
8484
),
8585
"jobDuration": execution_time_seconds,
8686
}

0 commit comments

Comments
 (0)