diff options
author | Håkon Hallingstad <hakon@verizonmedia.com> | 2020-01-14 15:57:25 +0100 |
---|---|---|
committer | Håkon Hallingstad <hakon@verizonmedia.com> | 2020-01-14 15:57:25 +0100 |
commit | 5a64ba73e4d032f585c2bb9f07c9b323e56d6c38 (patch) | |
tree | 4e93821f22e5dcd3c5fca6dfc59139e26316a7dd /config-model | |
parent | 29d57953ea867faa7233e9110630ba614ea23b39 (diff) |
Add vds.datastored.bucket_space.buckets_total as a fleetcontroller consumer for storage cluster to expose metric in host info
Diffstat (limited to 'config-model')
-rw-r--r-- | config-model/src/main/java/com/yahoo/vespa/model/content/storagecluster/StorageCluster.java | 3 | ||||
-rw-r--r-- | config-model/src/test/java/com/yahoo/vespa/model/content/ContentClusterTest.java | 6 |
2 files changed, 6 insertions, 3 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/content/storagecluster/StorageCluster.java b/config-model/src/main/java/com/yahoo/vespa/model/content/storagecluster/StorageCluster.java index cd99f46646b..091e82a9c76 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/content/storagecluster/StorageCluster.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/content/storagecluster/StorageCluster.java @@ -80,7 +80,8 @@ public class StorageCluster extends AbstractConfigProducer<StorageNode> ContentCluster.getMetricBuilder("fleetcontroller", builder). addedmetrics("vds.datastored.alldisks.docs"). addedmetrics("vds.datastored.alldisks.bytes"). - addedmetrics("vds.datastored.alldisks.buckets"); + addedmetrics("vds.datastored.alldisks.buckets"). + addedmetrics("vds.datastored.bucket_space.buckets_total"); ContentCluster.getMetricBuilder("log", builder). addedmetrics("vds.filestor.alldisks.allthreads.put.sum"). diff --git a/config-model/src/test/java/com/yahoo/vespa/model/content/ContentClusterTest.java b/config-model/src/test/java/com/yahoo/vespa/model/content/ContentClusterTest.java index d6579ba88d4..305bcc1d7d5 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/content/ContentClusterTest.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/content/ContentClusterTest.java @@ -659,10 +659,11 @@ public class ContentClusterTest extends ContentBaseTest { assertEquals(6, config.consumer().size()); assertEquals("fleetcontroller", config.consumer(4).name()); - assertEquals(3, config.consumer(4).addedmetrics().size()); + assertEquals(4, config.consumer(4).addedmetrics().size()); assertEquals("vds.datastored.alldisks.docs", config.consumer(4).addedmetrics(0)); assertEquals("vds.datastored.alldisks.bytes", config.consumer(4).addedmetrics(1)); assertEquals("vds.datastored.alldisks.buckets", config.consumer(4).addedmetrics(2)); + assertEquals("vds.datastored.bucket_space.buckets_total", config.consumer(4).addedmetrics(3)); } public MetricsmanagerConfig.Consumer getConsumer(String consumer, MetricsmanagerConfig config) { @@ -723,7 +724,8 @@ public class ContentClusterTest extends ContentBaseTest { expected = "[vds.datastored.alldisks.docs\n" + "vds.datastored.alldisks.bytes\n" + - "vds.datastored.alldisks.buckets]"; + "vds.datastored.alldisks.buckets\n" + + "vds.datastored.bucket_space.buckets_total]"; actual = getConsumer("fleetcontroller", config).addedmetrics().toString().replaceAll(", ", "\n"); assertEquals(expected, actual); } |