From 5b4101564cce6218898769f36f84ed3b0f54cd0e Mon Sep 17 00:00:00 2001 From: Jon Marius Venstad Date: Tue, 12 Feb 2019 08:00:01 +0100 Subject: Remove code that would never remove anything --- .../vespa/hosted/controller/deployment/DeploymentTrigger.java | 8 +++----- 1 file 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) { -- cgit v1.2.3