diff options
author | Jon Marius Venstad <venstad@gmail.com> | 2020-06-04 13:32:24 +0200 |
---|---|---|
committer | Jon Marius Venstad <venstad@gmail.com> | 2020-06-04 13:32:24 +0200 |
commit | f4cef55781dae802c7004b43882bb235edbad80a (patch) | |
tree | fbd4d1009a5b9a9ce958dca395c92f29dcdab0e7 /controller-server/src | |
parent | f489483457969155f3b7483b3a865f131d556c8f (diff) |
Aborted jobs are by purpose, and should not affect confidence
Diffstat (limited to 'controller-server/src')
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/versions/DeploymentStatistics.java | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/versions/DeploymentStatistics.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/versions/DeploymentStatistics.java index b91ed3734f1..99cf7542d53 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/versions/DeploymentStatistics.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/versions/DeploymentStatistics.java @@ -88,9 +88,9 @@ public class DeploymentStatistics { // Add all unsuccessful runs for failing jobs as any run may have resulted in an incomplete deployment // where a subset of nodes have upgraded. - // TODO jonmv: canary-pipeline.custom on 7.188.11, but not really, in staging ... failing.not().failingApplicationChange() .not().withStatus(RunStatus.outOfCapacity) + .not().withStatus(RunStatus.aborted) .mapToList(JobStatus::runs) .forEach(runs -> runs.descendingMap().values().stream() .dropWhile(run -> ! run.hasEnded()) @@ -103,6 +103,7 @@ public class DeploymentStatistics { failing.failingApplicationChange() .concat(failing.withStatus(RunStatus.outOfCapacity)) + .concat(failing.withStatus(RunStatus.aborted)) .lastCompleted().asList() .forEach(run -> { otherFailing.putIfAbsent(run.versions().targetPlatform(), new ArrayList<>()); |