diff options
author | Jon Marius Venstad <venstad@gmail.com> | 2019-09-13 10:56:57 +0200 |
---|---|---|
committer | Jon Marius Venstad <venstad@gmail.com> | 2019-09-13 12:09:06 +0200 |
commit | da4e3f456427a4d7e13f6081123e9179ea75b06b (patch) | |
tree | 7c39a10c2b91448d568c057def6d693099bdb494 /controller-server | |
parent | 78d7684fc601234b5468725b2a70eeb2385a9fb6 (diff) |
Limit logs to be not before the current deployment
Diffstat (limited to 'controller-server')
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/JobController.java | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/JobController.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/JobController.java index 5b83e0bd2c1..dfa72967aae 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/JobController.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/JobController.java @@ -151,13 +151,16 @@ public class JobController { public void updateVespaLog(RunId id) { locked(id, run -> { ZoneId zone = id.type().zone(controller.system()); - if ( ! controller.applications().require(id.application()) - .deployments().containsKey(zone)) + Optional<Deployment> deployment = Optional.ofNullable(controller.applications().require(id.application()) + .deployments().get(zone)); + if (deployment.isEmpty()) return run; + long from = Math.max(run.lastVespaLogTimestamp().toEpochMilli(), + deployment.get().at().toEpochMilli()); List<LogEntry> log = LogEntry.parseVespaLog(controller.serviceRegistry().configServer() .getLogs(new DeploymentId(id.application(), zone), - Map.of("from", Long.toString(run.lastVespaLogTimestamp().toEpochMilli())))); + Map.of("from", Long.toString(from)))); if (log.isEmpty()) return run; |