[AN-393] Add waiting for quota to quota messages (#7686)

Co-authored-by: Adam Nichols <[email protected]>
broadinstitute · Feb 4, 2025 · 5a7f7f8 · 5a7f7f8
1 parent 2c134ec
commit 5a7f7f8
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/backend/src/main/scala/cromwell/backend/async/KnownJobFailureException.scala b/backend/src/main/scala/cromwell/backend/async/KnownJobFailureException.scala
@@ -5,7 +5,9 @@ import common.exception.ThrowableAggregation
 import cromwell.core.path.Path
 import wom.expression.{NoIoFunctionSet, WomExpression}
 
-abstract class KnownJobFailureException extends Exception {
+import scala.util.control.NoStackTrace
+
+abstract class KnownJobFailureException extends Exception with NoStackTrace {
   def stderrPath: Option[Path]
 }
 

diff --git a/centaur/src/main/resources/standardTestCases/quota_fail_retry.test b/centaur/src/main/resources/standardTestCases/quota_fail_retry.test
@@ -1,3 +1,4 @@
+ignore: true # GCP seems to have fixed the quota handling bug, which makes this test unnecessary
 name: quota_fail_retry
 testFormat: workflowfailure
 # In PAPI v2 there seems to be a quota exhaustion message in a reasonably timely manner, while in GCP Batch the job

diff --git a/...lines/common/src/main/scala/cromwell/backend/google/pipelines/common/errors/package.scala b/...lines/common/src/main/scala/cromwell/backend/google/pipelines/common/errors/package.scala
@@ -7,7 +7,8 @@ package object errors {
     "usage too high",
     "no available zones",
     "resource_exhausted",
-    "quota too low"
+    "quota too low",
+    "waiting for quota"
   )
 
   def isQuotaMessage(msg: String): Boolean =