From 66c0b4c3c54ef780e7e63953ef0ce87509fa369e Mon Sep 17 00:00:00 2001 From: gjoranv Date: Fri, 26 Mar 2021 13:23:31 +0100 Subject: Add groupId metric dimension. - It is not yet added to the public http apis, due to the limitation of 10 total dimensions. --- .../ai/vespa/metricsproxy/metric/dimensions/PublicDimensions.java | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'metrics-proxy') diff --git a/metrics-proxy/src/main/java/ai/vespa/metricsproxy/metric/dimensions/PublicDimensions.java b/metrics-proxy/src/main/java/ai/vespa/metricsproxy/metric/dimensions/PublicDimensions.java index f6bda2f9ef5..84c95f8e39e 100644 --- a/metrics-proxy/src/main/java/ai/vespa/metricsproxy/metric/dimensions/PublicDimensions.java +++ b/metrics-proxy/src/main/java/ai/vespa/metricsproxy/metric/dimensions/PublicDimensions.java @@ -25,6 +25,10 @@ public final class PublicDimensions { public static final String INTERNAL_CLUSTER_ID = "clusterid"; public static final String CLUSTER_ID = "clusterId"; + // This dimension is not currently (March 2021) added to the 'commonDimensions' allow-list below, due to the + // limit of 10 total dimensions in public http apis. See e.g. MetricsV2Handler#MAX_DIMENSIONS. + public static final String GROUP_ID = "groupId"; + // Internal name (instance) is confusing, so renamed to 'serviceId' for public use. // This is added by the metrics-proxy. public static final String INTERNAL_SERVICE_ID = "instance"; -- cgit v1.2.3