diff options
author | jonmv <venstad@gmail.com> | 2022-04-22 10:34:14 +0200 |
---|---|---|
committer | jonmv <venstad@gmail.com> | 2022-04-22 10:34:14 +0200 |
commit | 04b0f6aa73c754e2af9f42a3796fa696bcf8f593 (patch) | |
tree | fb1c4c065e53ed7dc347825db74eb96b903717d9 /controller-server/src | |
parent | e57d5aafce13a9e17e150cce92bb437a60e4f197 (diff) |
Fix badge handler NPE
Diffstat (limited to 'controller-server/src')
2 files changed, 2 insertions, 2 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentStatus.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentStatus.java index cb2958745d0..e13175806bf 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentStatus.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentStatus.java @@ -905,7 +905,7 @@ public class DeploymentStatus { status.application, Optional.of(deployment), status.systemVersion))) - .orElseGet(() -> (change.platform().isEmpty() || change.platform().get().equals(run.versions().targetPlatform())) + .orElseGet(() -> (change.platform().isEmpty() || change.platform().get().equals(run.versions().targetPlatform())) && (change.revision().isEmpty() || change.revision().get().equals(run.versions().targetRevision())))) .matching(Run::hasSucceeded) .asList().stream() diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/deployment/BadgeApiHandler.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/deployment/BadgeApiHandler.java index 3078eb3cb24..4cfcadd5c41 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/deployment/BadgeApiHandler.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/deployment/BadgeApiHandler.java @@ -135,7 +135,7 @@ public class BadgeApiHandler extends ThreadedHttpRequestHandler { if (this == o) return true; if (o == null || getClass() != o.getClass()) return false; Key key = (Key) o; - return historyLength == key.historyLength && id.equals(key.id) && type.equals(key.type); + return historyLength == key.historyLength && id.equals(key.id) && Objects.equals(type, key.type); } @Override |