From 4ea035b70e31781783113a4b6eb56c21c7c02bf3 Mon Sep 17 00:00:00 2001 From: Øyvind Grønnesby Date: Mon, 12 Dec 2022 16:50:14 +0100 Subject: Create a metric for maintainer execution time --- .../hosted/provision/maintenance/NodeRepositoryMaintainer.java | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'node-repository/src/main/java/com/yahoo') diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRepositoryMaintainer.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRepositoryMaintainer.java index 77fe4ba129d..3c00e3b708d 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRepositoryMaintainer.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRepositoryMaintainer.java @@ -24,7 +24,7 @@ public abstract class NodeRepositoryMaintainer extends Maintainer { private final NodeRepository nodeRepository; public NodeRepositoryMaintainer(NodeRepository nodeRepository, Duration interval, Metric metric) { - super(null, interval, nodeRepository.clock().instant(), nodeRepository.jobControl(), + super(null, interval, nodeRepository.clock(), nodeRepository.jobControl(), new NodeRepositoryJobMetrics(metric), nodeRepository.database().cluster(), true); this.nodeRepository = nodeRepository; } @@ -57,8 +57,10 @@ public abstract class NodeRepositoryMaintainer extends Maintainer { } @Override - public void completed(String job, double successFactor) { - metric.set("maintenance.successFactor", successFactor, metric.createContext(Map.of("job", job))); + public void completed(String job, double successFactor, long duration) { + var context = metric.createContext(Map.of("job", job)); + metric.set("maintenance.successFactor", successFactor, context); + metric.set("maintenance.duration", duration, context); } } -- cgit v1.2.3