diff options
author | Jon Marius Venstad <venstad@gmail.com> | 2019-02-12 08:00:01 +0100 |
---|---|---|
committer | Jon Marius Venstad <venstad@gmail.com> | 2019-02-12 08:00:01 +0100 |
commit | 5b4101564cce6218898769f36f84ed3b0f54cd0e (patch) | |
tree | 3159fd39dcae3cc3c2eb51c2492882e18973745e | |
parent | 6af1e4e96e44f42975c90ad18e736f4f59b8f58b (diff) |
Remove code that would never remove anything
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTrigger.java | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTrigger.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTrigger.java index 16f08ad1e15..49788b643e0 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTrigger.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTrigger.java @@ -45,6 +45,7 @@ import static com.yahoo.vespa.hosted.controller.api.integration.BuildService.Job import static com.yahoo.vespa.hosted.controller.api.integration.deployment.JobType.component; import static com.yahoo.vespa.hosted.controller.api.integration.deployment.JobType.stagingTest; import static com.yahoo.vespa.hosted.controller.api.integration.deployment.JobType.systemTest; +import static java.util.Collections.emptyList; import static java.util.Collections.singletonList; import static java.util.Comparator.comparing; import static java.util.Comparator.naturalOrder; @@ -326,11 +327,8 @@ public class DeploymentTrigger { if (completedAt.isPresent() && canTrigger(job, versions, application, stepJobs)) { jobs.add(deploymentJob(application, versions, change, job, reason, completedAt.get())); } - if ( ! alreadyTriggered(application, versions)) { - // Only remove test jobs that target this combination - if (testJobs == null) testJobs = new ArrayList<>(); - testJobs.removeIf(j -> versions.sourcesMatchIfPresent(j.triggering) && - versions.targetsMatch(j.triggering)); + if ( ! alreadyTriggered(application, versions) && testJobs == null) { + testJobs = emptyList(); } } else if (testJobs == null) { |