diff options
author | Valerij Fredriksen <valerijf@oath.com> | 2018-10-17 15:29:47 +0200 |
---|---|---|
committer | Valerij Fredriksen <valerijf@oath.com> | 2018-10-17 15:50:42 +0200 |
commit | 9adcb9e3de736f7f51aae4b6921f3c05e3439a07 (patch) | |
tree | 9a81bfa83939d3119e0120f4679ae11bdbe1a943 | |
parent | 2bb3ba0a9ec179d1ab9002e633f7cfd518742734 (diff) |
Do not write Filebeat config in StorageMaintainer
3 files changed, 0 insertions, 16 deletions
diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/docker/DockerOperationsImpl.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/docker/DockerOperationsImpl.java index 13ee623a74e..f898fb5a761 100644 --- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/docker/DockerOperationsImpl.java +++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/docker/DockerOperationsImpl.java @@ -265,7 +265,6 @@ public class DockerOperationsImpl implements DockerOperations { // Paths unique to each container List<Path> paths = new ArrayList<>(Arrays.asList( Paths.get("/etc/yamas-agent"), - Paths.get("/etc/filebeat"), context.pathInNodeUnderVespaHome("logs/daemontools_y"), context.pathInNodeUnderVespaHome("logs/jdisc_core"), context.pathInNodeUnderVespaHome("logs/langdetect/"), diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/StorageMaintainer.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/StorageMaintainer.java index 715a977de65..655d755960b 100644 --- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/StorageMaintainer.java +++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/StorageMaintainer.java @@ -167,20 +167,6 @@ public class StorageMaintainer { return check; } - public void writeFilebeatConfig(NodeAgentContext context, NodeSpec node) { - try { - FilebeatConfigProvider filebeatConfigProvider = new FilebeatConfigProvider(environment); - Optional<String> config = filebeatConfigProvider.getConfig(context, node); - if (!config.isPresent()) return; - - Path filebeatPath = context.pathOnHostFromPathInNode("/etc/filebeat/filebeat.yml"); - Files.write(filebeatPath, config.get().getBytes()); - context.log(logger, "Wrote filebeat config"); - } catch (Throwable t) { - context.log(logger, LogLevel.ERROR, "Failed writing filebeat config", t); - } - } - public Optional<Long> getDiskUsageFor(NodeAgentContext context) { Path containerDir = context.pathOnHostFromPathInNode("/"); try { diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java index a76d0c8b91b..f6bea8a8266 100644 --- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java +++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java @@ -228,7 +228,6 @@ public class NodeAgentImpl implements NodeAgent { if (!hasResumedNode) { if (!currentFilebeatRestarter.isPresent()) { storageMaintainer.writeMetricsConfig(context, node); - storageMaintainer.writeFilebeatConfig(context, node); currentFilebeatRestarter = Optional.of(filebeatRestarter.scheduleWithFixedDelay( () -> serviceRestarter.accept("filebeat"), 1, 1, TimeUnit.DAYS)); } |