issue-21715 fetching metric values with composite(all) dimension keys

[rajim17]

Description:

If metric has multiple dimensions, metricValues are fetched by single dimension instead of combining all dimensions. With single dimension metric value is average value of other dimensions.

In below example without all dimensions in the filter, 'metadata_nodepool' provides average value of 2 'metadata_node' datapoints. But cannot combine dimensions of metadata_node under 'metadata_nodepool' dimensions.

Metric #44
Descriptor:
     -> Name: azure_node_cpu_usage_millicores_total
NumberDataPoints #0
Data point attributes:
     -> azuremonitor.resource_id: Str(/subscriptions/test/resourceGroups/aks-dev2-blue/providers/Microsoft.ContainerService/managedClusters/aks-dev2-blue)
     -> metadata_nodepool: Str(general1)
Value: 625.500000

NumberDataPoints #1
Data point attributes:
     -> azuremonitor.resource_id: Str(/subscriptions/test/resourceGroups/aks-dev2-blue/providers/Microsoft.ContainerService/managedClusters/aks-dev2-blue)
     -> metadata_node: Str(aks-general1-vmss00006x)
Value: 506.000000

NumberDataPoints #2
Data point attributes:
     -> azuremonitor.resource_id: Str(/subscriptions/test/resourceGroups/aks-dev2-blue/providers/Microsoft.ContainerService/managedClusters/aks-dev2-blue)
     -> metadata_node: Str(aks-general1-vmss000062)
Value: 745.000000

With all Dimensions applied:
Here each datapoint provides all related dimension values. With this information, it is easy to apply multiple filters.

Metric #44
Descriptor:
     -> Name: azure_node_cpu_usage_millicores_total

NumberDataPoints #0
Data point attributes:
     -> azuremonitor.resource_id: Str(/subscriptions/test/resourceGroups/aks-dev2-blue/providers/Microsoft.ContainerService/managedClusters/aks-dev2-blue)
     -> metadata_node: Str(aks-general1-vmss00006x)
     -> metadata_nodepool: Str(general1)
Value: 506.000000

NumberDataPoints #1
Data point attributes:
     -> azuremonitor.resource_id: Str(/subscriptions/test/resourceGroups/aks-dev2-blue/providers/Microsoft.ContainerService/managedClusters/aks-dev2-blue)
     -> metadata_node: Str(aks-general1-vmss000062)
     -> metadata_nodepool: Str(general1)
Value: 745.000000

In Azuremonitor receiver, getResourceMetricsValuesRequestOptions func, single dimension key alone provided. Instead all possible dimension keys for the metric have to be given.
Solution:
with metricresource definitions, Generate hash string with all dimensions and store the metrics based on hash instead of single dimension + timegrain filter.

Sample MetricClient REST Request with all dimensions in filter:

https://management.azure.com/subscriptions/test/resourceGroups/aks-dev2-blue/providers/Microsoft.ContainerService/managedClusters/aks-dev2-blue/providers/Microsoft.Insights/metrics?timespan=2023-04-29T02:20:00Z/2023-05-04T04:20:00Z&interval=PT1H&api-version=2018-01-01&metricnamespace=Microsoft.ContainerService/managedClusters&metricnames=node_cpu_usage_millicores&$filter=nodepool eq '*' and node eq '*'

Fixes #21715

Testing: Testdata with multiple dimensions are working.

Documentation:

[rajim17]

@altuner @codeboten - Could you help review the changes ?

[altuner]

left small comments

[altuner]

LGTM

[rajim17]

@altuner @codeboten - I'm planning to push this change for the next release. Requesting to review and provide the feedback so it does not miss the window. :-)

[altuner]

@rajim17 please fix the breaking test

[rajim17]

@rajim17 please fix the breaking test

Integration_test on rabbitmqreceiver was failing. I triggered the build process with dummy commit message and tests are running. I will check once build is complete.

[rajim17]

@altuner - All tests are passed.

[rajim17]

@codeboten - Could you help reviewing the PR ?

[junhuangli]

@altuner We are waiting for this fix. Could you merge this pull request or is there anything still pending?

[rajim17]

@codeboten - Could you help merging this PR ? It is already approved by code owner @altuner .

[junhuangli]

@codeboten Could you take another look at this PR :)