Emit metric counting autoscale events

2 files changed, 9 insertions, 3 deletions

diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/AutoscalingMaintainerTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/AutoscalingMaintainerTest.java
index 9ca667b54e4..5c5cc1be170 100644
--- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/AutoscalingMaintainerTest.java
+++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/AutoscalingMaintainerTest.java
@@ -1,13 +1,13 @@
 // Copyright Vespa.ai. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
 package com.yahoo.vespa.hosted.provision.maintenance;
 
-import com.yahoo.config.provision.ClusterInfo;
-import com.yahoo.config.provision.IntRange;
 import com.yahoo.config.provision.ApplicationId;
 import com.yahoo.config.provision.Capacity;
+import com.yahoo.config.provision.ClusterInfo;
 import com.yahoo.config.provision.ClusterResources;
 import com.yahoo.config.provision.ClusterSpec;
 import com.yahoo.config.provision.Environment;
+import com.yahoo.config.provision.IntRange;
 import com.yahoo.config.provision.NodeResources;
 import com.yahoo.config.provision.RegionName;
 import com.yahoo.config.provision.SystemName;
@@ -150,6 +150,9 @@ public class AutoscalingMaintainerTest {
         assertEquals(lastMaintenanceTime.toEpochMilli(), tester.deployer().activationTime(app1).get().toEpochMilli());
         events = tester.nodeRepository().applications().get(app1).get().cluster(cluster1.id()).get().scalingEvents();
         assertEquals(2, events.get(2).generation());
+
+        // Metric counts number of times the cluster is rescaled
+        assertEquals(2, tester.metric().sumNumberValues("clusterAutoscaled"), Double.MIN_VALUE);
     }
 
     @Test
diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/AutoscalingMaintainerTester.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/AutoscalingMaintainerTester.java
index cdef45dbb9c..e4a712d3898 100644
--- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/AutoscalingMaintainerTester.java
+++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/AutoscalingMaintainerTester.java
@@ -37,6 +37,7 @@ public class AutoscalingMaintainerTester {
     private final ProvisioningTester provisioningTester;
     private final AutoscalingMaintainer maintainer;
     private final MockDeployer deployer;
+    private final TestMetric metric;
 
     public AutoscalingMaintainerTester(MockDeployer.ApplicationContext ... appContexts) {
         this(new Zone(Environment.prod, RegionName.from("us-east3")), appContexts);
@@ -46,9 +47,10 @@ public class AutoscalingMaintainerTester {
         provisioningTester = new ProvisioningTester.Builder().zone(zone).flavorsConfig(flavorsConfig()).build();
         provisioningTester.clock().setInstant(Instant.ofEpochMilli(0));
         deployer = new MockDeployer(provisioningTester.provisioner(), provisioningTester.clock(), List.of(appContexts));
+        metric = new TestMetric();
         maintainer = new AutoscalingMaintainer(provisioningTester.nodeRepository(),
                                                deployer,
-                                               new TestMetric(),
+                                               metric,
                                                Duration.ofMinutes(1));
         provisioningTester.makeReadyNodes(20, "flt", NodeType.host, 8);
         provisioningTester.activateTenantHosts();
@@ -58,6 +60,7 @@ public class AutoscalingMaintainerTester {
     public ManualClock clock() { return provisioningTester.clock(); }
     public MockDeployer deployer() { return deployer; }
     public AutoscalingMaintainer maintainer() { return maintainer; }
+    public TestMetric metric() { return metric; }
 
     public static ApplicationId makeApplicationId(String name) { return ProvisioningTester.applicationId(name); }
     public static ClusterSpec containerClusterSpec() { return ProvisioningTester.containerClusterSpec(); }