diff options
author | Jon Marius Venstad <jvenstad@yahoo-inc.com> | 2018-04-27 09:46:59 +0200 |
---|---|---|
committer | Jon Marius Venstad <jvenstad@yahoo-inc.com> | 2018-04-27 09:46:59 +0200 |
commit | 58c04b8acc223df572d155afd0d60933ec3b27b9 (patch) | |
tree | e0eee8114a90851f0d3bb1b65223decbb9e37318 /controller-server | |
parent | 1045911ced3933344dc2a24494e676805a258346 (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.java | 6 |
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. */ |