summaryrefslogtreecommitdiffstats
path: root/controller-server
diff options
context:
space:
mode:
authorJon Marius Venstad <jvenstad@yahoo-inc.com>2018-04-27 09:46:59 +0200
committerJon Marius Venstad <jvenstad@yahoo-inc.com>2018-04-27 09:46:59 +0200
commit58c04b8acc223df572d155afd0d60933ec3b27b9 (patch)
treee0eee8114a90851f0d3bb1b65223decbb9e37318 /controller-server
parent1045911ced3933344dc2a24494e676805a258346 (diff)
Don't count outOfCapacity errors as failing jobs
Diffstat (limited to 'controller-server')
-rw-r--r--controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/DeploymentJobs.java6
1 files changed, 4 insertions, 2 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/DeploymentJobs.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/DeploymentJobs.java
index faae535054f..fb8a4051649 100644
--- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/DeploymentJobs.java
+++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/DeploymentJobs.java
@@ -91,8 +91,10 @@ public class DeploymentJobs {
/** Returns whether this has some job status which is not a success */
public boolean hasFailures() {
- // TODO jvenstad: Ignore outOfCapacity errors from test jobs?
- return ! JobList.from(status.values()).failing().isEmpty();
+ return ! JobList.from(status.values())
+ .failing()
+ .not().failingBecause(JobError.outOfCapacity)
+ .isEmpty();
}
/** Returns the JobStatus of the given JobType, or empty. */