diff options
author | Jon Bratseth <bratseth@yahoo-inc.com> | 2016-11-01 16:06:05 +0100 |
---|---|---|
committer | Jon Bratseth <bratseth@yahoo-inc.com> | 2016-11-01 16:06:05 +0100 |
commit | 159204ce267aaadac293dd354ac0cb05930a8548 (patch) | |
tree | 87c9586cc8d834a8c6bf39cb28ed57f269970790 /configserver | |
parent | e823fbef7f24772fa813a0e370c00b474e60bb8a (diff) |
Add temporary logging + whitespace fixes
Diffstat (limited to 'configserver')
-rw-r--r-- | configserver/src/main/java/com/yahoo/vespa/config/server/deploy/Deployment.java | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/deploy/Deployment.java b/configserver/src/main/java/com/yahoo/vespa/config/server/deploy/Deployment.java index 2ed16e587b6..f28af8939bc 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/deploy/Deployment.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/deploy/Deployment.java @@ -140,10 +140,10 @@ public class Deployment implements com.yahoo.config.provision.Deployment { transaction.add(deactivateCurrentActivateNew(localSessionRepo.getActiveSession(session.getApplicationId()), session, ignoreSessionStaleFailure)); // TODO: (October 2016) Remove the second part of this if statement as soon as all zone applications stop using hosts.xml for routing nodes + log.log(LogLevel.INFO, "Activating " + session.getProvisionInfo().getHosts() + ". isHostedRoutingApplicationUsingRoutingNodesInNodeRepo:" + isHostedRoutingApplicationUsingRoutingNodesInNodeRepo(session)); if (hostProvisioner.isPresent() && (isNotHostedRoutingApplication(session.getApplicationId()) || isHostedRoutingApplicationUsingRoutingNodesInNodeRepo(session))) { - ProvisionInfo info = session.getProvisionInfo(); - hostProvisioner.get().activate(transaction, session.getApplicationId(), info.getHosts()); + hostProvisioner.get().activate(transaction, session.getApplicationId(), session.getProvisionInfo().getHosts()); } transaction.commit(); session.waitUntilActivated(timeoutBudget); @@ -234,7 +234,7 @@ public class Deployment implements com.yahoo.config.provision.Deployment { // Precondition: session is for a hosted routing application boolean isHostedRoutingApplicationUsingRoutingNodesInNodeRepo(LocalSession session) { - final Path servicesPath = Path.fromString(ApplicationPackage.SERVICES); + Path servicesPath = Path.fromString(ApplicationPackage.SERVICES); ApplicationFile services = session.getApplicationFile(servicesPath, LocalSession.Mode.READ); try { return usesRoutingNodesInNodeRepo(services.createReader()); @@ -243,7 +243,7 @@ public class Deployment implements com.yahoo.config.provision.Deployment { } } - // TODO: Copied verbatim from VespaModelFactory, since we need it know and it is not available for all model versions yet. + // TODO: Copied verbatim from VespaModelFactory, since we need it now and it is not available for all model versions yet. // Remove or use the one from VespaModelFactory as soon as possible private boolean usesRoutingNodesInNodeRepo(Reader servicesReader) { Document services = XmlHelper.getDocument(servicesReader); |