diff options
author | Valerij Fredriksen <freva@users.noreply.github.com> | 2019-01-14 11:40:25 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-01-14 11:40:25 +0100 |
commit | b5bc1d4c9ebc3b2b3c8b24af1fea03583e5e217f (patch) | |
tree | f3b9dff5c55ee1d5c0438f3a065e16ff05b30b15 /node-admin/src | |
parent | 8b57fffd5dd15bf5da117191cf40d443a967b5c0 (diff) | |
parent | 0d47ae31a5a7517da36438dad1814b822819696c (diff) |
Merge pull request #8090 from vespa-engine/freva/remove-ntp-monitoring
Remove NTP check
Diffstat (limited to 'node-admin/src')
2 files changed, 6 insertions, 10 deletions
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 47255c54455..6787e2e0784 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 @@ -72,10 +72,6 @@ public class StorageMaintainer { Path hostLifeCheckPath = context.pathInNodeUnderVespaHome("libexec/yms/yms_check_host_life"); configs.add(new SecretAgentCheckConfig("host-life", 60, hostLifeCheckPath).withTags(tags)); - // ntp - Path ntpCheckPath = context.pathInNodeUnderVespaHome("libexec/yms/yms_check_ntp"); - configs.add(new SecretAgentCheckConfig("ntp", 60, ntpCheckPath).withTags(tags)); - // coredumps (except for the done coredumps which is handled by the host) Path coredumpCheckPath = context.pathInNodeUnderVespaHome("libexec/yms/yms_check_coredumps"); configs.add(new SecretAgentCheckConfig("system-coredumps-processing", 300, coredumpCheckPath, diff --git a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/maintenance/StorageMaintainerTest.java b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/maintenance/StorageMaintainerTest.java index 05b9c413594..4c753e5a95d 100644 --- a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/maintenance/StorageMaintainerTest.java +++ b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/maintenance/StorageMaintainerTest.java @@ -53,7 +53,7 @@ public class StorageMaintainerTest { public void tenant() { Path path = executeAs(NodeType.tenant); - assertChecks(path, "athenz-certificate-expiry", "host-life", "ntp", + assertChecks(path, "athenz-certificate-expiry", "host-life", "system-coredumps-processing", "vespa", "vespa-health"); // All dimensions for vespa metrics should be set by metricsproxy @@ -62,7 +62,7 @@ public class StorageMaintainerTest { " - all\n"); // For non vespa metrics, we need to set all the dimensions ourselves - assertCheckEnds(path.resolve("ntp.yaml"), + assertCheckEnds(path.resolve("host-life.yaml"), "tags:\n" + " namespace: Vespa\n" + " role: tenants\n" + @@ -85,7 +85,7 @@ public class StorageMaintainerTest { public void proxy() { Path path = executeAs(NodeType.proxy); - assertChecks(path, "athenz-certificate-expiry", "host-life", "ntp", "routing-configage", + assertChecks(path, "athenz-certificate-expiry", "host-life", "routing-configage", "ssl-status", "system-coredumps-processing", "vespa", "vespa-health"); // All dimensions for vespa metrics should be set by the source @@ -94,7 +94,7 @@ public class StorageMaintainerTest { " - all\n"); // For non vespa metrics, we need to set all the dimensions ourselves - assertCheckEnds(path.resolve("ntp.yaml"), + assertCheckEnds(path.resolve("host-life.yaml"), "tags:\n" + " namespace: Vespa\n" + " role: routing\n" + @@ -118,7 +118,7 @@ public class StorageMaintainerTest { Path path = executeAs(NodeType.config); assertChecks(path, "athenz-certificate-expiry", "configserver", "host-life", - "ntp", "system-coredumps-processing", "zkbackupage"); + "system-coredumps-processing", "zkbackupage"); assertCheckEnds(path.resolve("configserver.yaml"), " tags:\n" + @@ -132,7 +132,7 @@ public class StorageMaintainerTest { public void controller() { Path path = executeAs(NodeType.controller); - assertChecks(path, "athenz-certificate-expiry", "controller", "host-life", "ntp", + assertChecks(path, "athenz-certificate-expiry", "controller", "host-life", "system-coredumps-processing", "vespa", "vespa-health", "zkbackupage"); |