diff --git a/CHANGELOG.md b/CHANGELOG.md index ed588330c..da930db65 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -2,6 +2,8 @@ ## v0.7.2 Release Notes +### Fixed incorrectly translated job attempt status from Cromwell backend. + ### Fixed incorrect tooltip for standard out log. ### Added customized favicon. diff --git a/servers/cromwell/jobs/controllers/utils/task_statuses.py b/servers/cromwell/jobs/controllers/utils/task_statuses.py index 94ded1001..c776494ef 100644 --- a/servers/cromwell/jobs/controllers/utils/task_statuses.py +++ b/servers/cromwell/jobs/controllers/utils/task_statuses.py @@ -33,7 +33,7 @@ CromwellExecutionStatus.UNSTARTABLE: ApiStatus.FAILED, CromwellExecutionStatus.ABORTED: ApiStatus.ABORTED, CromwellExecutionStatus.BYPASSED: ApiStatus.SUBMITTED, - CromwellExecutionStatus.RETRYABLEFAILURE: ApiStatus.RUNNING, + CromwellExecutionStatus.RETRYABLEFAILURE: ApiStatus.FAILED, CromwellExecutionStatus.FAILED: ApiStatus.FAILED, CromwellExecutionStatus.DONE: ApiStatus.SUCCEEDED } diff --git a/servers/cromwell/jobs/test/test_task_statuses.py b/servers/cromwell/jobs/test/test_task_statuses.py index ec35d32b3..fccdf8d47 100644 --- a/servers/cromwell/jobs/test/test_task_statuses.py +++ b/servers/cromwell/jobs/test/test_task_statuses.py @@ -23,7 +23,7 @@ def test_cromwell_execution_to_api_maps_all_task_execution_statuses_correctly(se self.assertEqual(task_statuses.cromwell_execution_to_api('Unstartable'), 'Failed') self.assertEqual(task_statuses.cromwell_execution_to_api('Aborted'), 'Aborted') self.assertEqual(task_statuses.cromwell_execution_to_api('Bypassed'), 'Submitted') - self.assertEqual(task_statuses.cromwell_execution_to_api('RetryableFailure'), 'Running') + self.assertEqual(task_statuses.cromwell_execution_to_api('RetryableFailure'), 'Failed') self.assertEqual(task_statuses.cromwell_execution_to_api('Failed'), 'Failed') self.assertEqual(task_statuses.cromwell_execution_to_api('Done'), 'Succeeded') # yapf: enable