From b215f19492ab2507d49aed596a9e9b971514aacf Mon Sep 17 00:00:00 2001 From: Arne Juul Date: Wed, 11 Apr 2018 14:37:09 +0000 Subject: also require DeployState in simple constructor for AccessLogComponent --- .../model/admin/clustercontroller/ClusterControllerContainer.java | 2 +- .../main/java/com/yahoo/vespa/model/container/ContainerCluster.java | 2 +- .../yahoo/vespa/model/container/component/AccessLogComponent.java | 5 +++-- 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/config-model/src/main/java/com/yahoo/vespa/model/admin/clustercontroller/ClusterControllerContainer.java b/config-model/src/main/java/com/yahoo/vespa/model/admin/clustercontroller/ClusterControllerContainer.java index a20d69ecab4..4233d9a6f61 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/admin/clustercontroller/ClusterControllerContainer.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/admin/clustercontroller/ClusterControllerContainer.java @@ -70,7 +70,7 @@ public class ClusterControllerContainer extends Container implements addBundle("file:" + getDefaults().underVespaHome("lib/jars/zkfacade-jar-with-dependencies.jar")); log.log(LogLevel.DEBUG, "Adding access log for cluster controller ..."); - addComponent(new AccessLogComponent(AccessLogComponent.AccessLogType.queryAccessLog, "controller")); + addComponent(new AccessLogComponent(AccessLogComponent.AccessLogType.queryAccessLog, "controller", deployStateFrom(parent))); } @Override diff --git a/config-model/src/main/java/com/yahoo/vespa/model/container/ContainerCluster.java b/config-model/src/main/java/com/yahoo/vespa/model/container/ContainerCluster.java index d7ce3a9a975..2106906ecc1 100755 --- a/config-model/src/main/java/com/yahoo/vespa/model/container/ContainerCluster.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/container/ContainerCluster.java @@ -709,7 +709,7 @@ public final class ContainerCluster } public void addDefaultSearchAccessLog() { - addComponent(new AccessLogComponent(AccessLogComponent.AccessLogType.queryAccessLog, getName())); + addComponent(new AccessLogComponent(AccessLogComponent.AccessLogType.queryAccessLog, getName(), getDeployState())); } @Override diff --git a/config-model/src/main/java/com/yahoo/vespa/model/container/component/AccessLogComponent.java b/config-model/src/main/java/com/yahoo/vespa/model/container/component/AccessLogComponent.java index ccb4e271102..5114218d147 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/container/component/AccessLogComponent.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/container/component/AccessLogComponent.java @@ -27,10 +27,11 @@ public final class AccessLogComponent extends SimpleComponent implements AccessL private final DeployState deployState; private final String symlinkName; - public AccessLogComponent(AccessLogType logType, String clusterName) { + public AccessLogComponent(AccessLogType logType, String clusterName, DeployState deployState) + { this(logType, String.format("logs/vespa/qrs/%s.%s.%s", capitalize(logType.name()), clusterName, "%Y%m%d%H%M%S"), - null, null, null, null, + null, null, null, deployState, capitalize(logType.name()) + "." + clusterName); } -- cgit v1.2.3