Skip to content

Commit

Permalink
Include full error context when failing to abort TES jobs
Browse files Browse the repository at this point in the history
  • Loading branch information
cjllanwarne committed Jan 5, 2024
1 parent ecd44b6 commit f7caaa8
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -380,7 +380,7 @@ class TesAsyncBackendJobExecutionActor(override val standardParams: StandardAsyn
HttpRequest(method = HttpMethods.POST, uri = s"$tesEndpoint/${job.jobId}:cancel")
) onComplete {
case Success(_) => jobLogger.info("{} Aborted {}", tag: Any, job.jobId)
case Failure(ex) => jobLogger.warn("{} Failed to abort {}: {}", tag, job.jobId, ex.getMessage)
case Failure(ex) => jobLogger.warn("{} Failed to abort {}", tag, job.jobId, ex)

Check warning on line 383 in supportedBackends/tes/src/main/scala/cromwell/backend/impl/tes/TesAsyncBackendJobExecutionActor.scala

View check run for this annotation

Codecov / codecov/patch

supportedBackends/tes/src/main/scala/cromwell/backend/impl/tes/TesAsyncBackendJobExecutionActor.scala#L383

Added line #L383 was not covered by tests
}

()
Expand Down

0 comments on commit f7caaa8

Please sign in to comment.