Skip to content

Commit d652e09

Browse files
committed
cleanup
1 parent d8c5b0a commit d652e09

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

backend/src/main/scala/cromwell/backend/standard/StandardAsyncExecutionActor.scala

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -96,9 +96,7 @@ trait StandardAsyncExecutionActor
9696
val SIGTERM = 143
9797
val SIGINT = 130
9898
val SIGKILL = 137
99-
// The Variants team have observed 247 exit codes in the wild for what we suspect were OOM-killed jobs, but
100-
// unfortunately we do not yet have a test case that reproduces this. From Gemini:
101-
//
99+
// From Gemini:
102100
// An exit code of 247, particularly in the context of process execution in Linux or containerized environments like
103101
// Docker, often indicates a process termination due to resource limitations, most commonly insufficient memory (RAM).
104102
val SIGCONTAINERKILL = 247
@@ -1490,7 +1488,9 @@ trait StandardAsyncExecutionActor
14901488
} else {
14911489
tryReturnCodeAsInt match {
14921490
case Success(returnCodeAsInt)
1493-
if oomDetected(returnCodeAsInt) && memoryRetryRequested && !continueOnReturnCode.continueFor(returnCodeAsInt) =>
1491+
if oomDetected(returnCodeAsInt) && memoryRetryRequested && !continueOnReturnCode.continueFor(
1492+
returnCodeAsInt
1493+
) =>
14941494
val executionHandle = Future.successful(
14951495
FailedNonRetryableExecutionHandle(
14961496
RetryWithMoreMemory(jobDescriptor.key.tag, stderrAsOption, memoryRetryErrorKeys, log),

0 commit comments

Comments
 (0)