Merge pull request #604 from umccr/bugfix/rename-pieriandx-complete-s…

…tatus Pieriandx needs 'SUCCEEDED' status as positive terminal status not 'COMPLETE'
umccr · Oct 18, 2024 · f36b5b8 · f36b5b8
2 parents f9b52d4 + be29c03
commit f36b5b8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/...r/lambdas/get_informaticsjob_and_report_status_py/get_informaticsjob_and_report_status.py b/...r/lambdas/get_informaticsjob_and_report_status_py/get_informaticsjob_and_report_status.py
@@ -231,7 +231,7 @@ def handler(event, context):
     # If both the job status and report status are true, then the workflow status is complete
     elif JOB_STATUS_BOOL[job_status] is True and REPORT_STATUS_BOOL[report_status] is True:
         expression_attribute_values_dict[":workflow_status"] = {
-            "S": "COMPLETE"
+            "S": "SUCCEEDED"
         }
         update_expression_str = f"{update_expression_str}, workflow_status = :workflow_status"
 
@@ -294,7 +294,7 @@ def handler(event, context):
 # #       "BOOL": true
 # #     },
 # #     ":workflow_status": {
-# #       "S": "COMPLETE"
+# #       "S": "SUCCEEDED"
 # #     }
 # #   },
 # #   "update_expression_str": "SET job_status = :job_status, report_status = :report_status, job_status_bool = :job_status_bool, report_status_bool = :report_status_bool, workflow_status = :workflow_status"