[release/7.x] Update Diagnostics shipped version (#5992)

* Update Diagnostics shipped version

* Fixup api

* PR feedback

* Pr feedback

Author: wiktork

eng/dependabot/nuget.org/Versions.props

@@ -2,6 +2,6 @@
   <!-- Import references updated by Dependabot. -->
   <PropertyGroup>
     <!-- dotnet/diagnostics references -->
-    <MicrosoftDiagnosticsMonitoringShippedVersion>8.0.452401</MicrosoftDiagnosticsMonitoringShippedVersion>
+    <MicrosoftDiagnosticsMonitoringShippedVersion>8.0.510501</MicrosoftDiagnosticsMonitoringShippedVersion>
   </PropertyGroup>
 </Project>

src/Microsoft.Diagnostics.Monitoring.WebApi/Metrics/JsonCounterLogger.cs

@@ -43,7 +43,7 @@ protected override async Task SerializeAsync(ICounterPayload counter)
             }
             else if (counter is CounterEndedPayload)
             {
-                Logger.CounterEndedPayload(counter.Name);
+                Logger.CounterEndedPayload(counter.CounterMetadata.CounterName);
                 return;
             }
             else if (!counter.EventType.IsValuePublishedEvent())
@@ -70,12 +70,12 @@ protected override async Task SerializeAsync(ICounterPayload counter)
                     Quantile quantile = aggregatePercentilePayload.Quantiles[i];
 
                     SerializeCounterValues(counter.Timestamp,
-                        counter.Provider,
-                        counter.Name,
+                        counter.CounterMetadata.ProviderName,
+                        counter.CounterMetadata.CounterName,
                         counter.DisplayName,
                         counter.Unit,
                         counter.CounterType.ToString(),
-                        CounterUtilities.AppendPercentile(counter.Metadata, quantile.Percentage),
+                        CounterUtilities.AppendPercentile(counter.ValueTags, quantile.Percentage),
                         quantile.Value);
 
                     if (i < aggregatePercentilePayload.Quantiles.Length - 1)
@@ -90,12 +90,12 @@ protected override async Task SerializeAsync(ICounterPayload counter)
                 _bufferWriter.Clear();
 
                 SerializeCounterValues(counter.Timestamp,
-                    counter.Provider,
-                    counter.Name,
+                    counter.CounterMetadata.ProviderName,
+                    counter.CounterMetadata.CounterName,
                     counter.DisplayName,
                     counter.Unit,
                     counter.CounterType.ToString(),
-                    counter.Metadata,
+                    counter.ValueTags,
                     counter.Value);
             }
             await _stream.WriteAsync(_bufferWriter.WrittenMemory);

src/Microsoft.Diagnostics.Monitoring.WebApi/Metrics/MetricsStore.cs

@@ -30,8 +30,8 @@ public MetricKey(ICounterPayload metric)
             public override int GetHashCode()
             {
                 HashCode code = new HashCode();
-                code.Add(_metric.Provider);
-                code.Add(_metric.Name);
+                code.Add(_metric.CounterMetadata.ProviderName);
+                code.Add(_metric.CounterMetadata.CounterName);
                 return code.ToHashCode();
             }
 
@@ -73,9 +73,9 @@ public void AddMetric(ICounterPayload metric)
             //Do not accept CounterEnded payloads.
             if (metric is CounterEndedPayload counterEnded)
             {
-                if (_observedEndedCounters.Add((counterEnded.Provider, counterEnded.Name)))
+                if (_observedEndedCounters.Add((counterEnded.CounterMetadata.ProviderName, counterEnded.CounterMetadata.CounterName)))
                 {
-                    _logger.CounterEndedPayload(counterEnded.Name);
+                    _logger.CounterEndedPayload(counterEnded.CounterMetadata.CounterName);
                 }
                 return;
             }
@@ -137,7 +137,7 @@ public async Task SnapshotMetrics(Stream outputStream, CancellationToken token)
             {
                 ICounterPayload metricInfo = metricGroup.Value.First();
 
-                string metricName = PrometheusDataModel.GetPrometheusNormalizedName(metricInfo.Provider, metricInfo.Name, metricInfo.Unit);
+                string metricName = PrometheusDataModel.GetPrometheusNormalizedName(metricInfo.CounterMetadata.ProviderName, metricInfo.CounterMetadata.CounterName, metricInfo.Unit);
 
                 await WriteMetricHeader(metricInfo, writer, metricName);
 
@@ -165,7 +165,7 @@ public async Task SnapshotMetrics(Stream outputStream, CancellationToken token)
 
         private static string GetMetricLabels(ICounterPayload metric, double? quantile)
         {
-            string metadata = metric.Metadata;
+            string metadata = metric.ValueTags;
 
             char separator = IsMeter(metric) ? '=' : ':';
             var metadataValues = CounterUtilities.GetMetadata(metadata, separator);
@@ -242,7 +242,7 @@ private static async Task WriteMetricDetails(
 
         private static bool CompareMetrics(ICounterPayload first, ICounterPayload second)
         {
-            return string.Equals(first.Name, second.Name);
+            return string.Equals(first.CounterMetadata.CounterName, second.CounterMetadata.CounterName);
         }
 
         public void Clear()

src/Tests/Microsoft.Diagnostics.Monitoring.Tool.UnitTests/MetricsFormattingTests.cs

@@ -42,7 +42,8 @@ public async Task HistogramFormat_Test()
         {
             List<ICounterPayload> payload = new();
 
-            payload.Add(new AggregatePercentilePayload(MeterName, InstrumentName, "DisplayName", string.Empty, string.Empty,
+            payload.Add(new AggregatePercentilePayload(new CounterMetadata(MeterName, InstrumentName, meterTags: null, instrumentTags: null, scopeHash: null),
+                "DisplayName", string.Empty, string.Empty,
                 new Quantile[] { new Quantile(0.5, Value1), new Quantile(0.95, Value2), new Quantile(0.99, Value3) },
                 Timestamp));
 
@@ -51,7 +52,7 @@ public async Task HistogramFormat_Test()
 
             // Question - this is manually recreating what PrometheusDataModel.GetPrometheusNormalizedName does to get the metric name;
             // should we call this method, or should this also be implicitly testing its behavior by having this hard-coded?
-            string metricName = $"{MeterName.ToLowerInvariant()}_{payload[0].Name}";
+            string metricName = $"{MeterName.ToLowerInvariant()}_{payload[0].CounterMetadata.CounterName}";
 
             const string quantile_50 = "{quantile=\"0.5\"}";
             const string quantile_95 = "{quantile=\"0.95\"}";
@@ -68,15 +69,16 @@ public async Task HistogramFormat_Test()
         [Fact]
         public async Task GaugeFormat_Test()
         {
-            ICounterPayload payload = new GaugePayload(MeterName, InstrumentName, "DisplayName", "", null, Value1, Timestamp);
+            ICounterPayload payload = new GaugePayload(new CounterMetadata(MeterName, InstrumentName, meterTags: null, instrumentTags: null, scopeHash: null),
+                "DisplayName", "", null, Value1, Timestamp);
 
             MemoryStream stream = await GetMetrics(new() { payload });
 
             List<string> lines = ReadStream(stream);
 
             // Question - this is manually recreating what PrometheusDataModel.GetPrometheusNormalizedName does to get the metric name;
             // should we call this method, or should this also be implicitly testing its behavior by having this hard-coded?
-            string metricName = $"{MeterName.ToLowerInvariant()}_{payload.Name}";
+            string metricName = $"{MeterName.ToLowerInvariant()}_{payload.CounterMetadata.CounterName}";
 
             Assert.Equal(3, lines.Count);
             Assert.Equal(FormattableString.Invariant($"# HELP {metricName}{payload.Unit} {payload.DisplayName}"), lines[0]);
@@ -87,15 +89,16 @@ public async Task GaugeFormat_Test()
         [Fact]
         public async Task CounterFormat_Test()
         {
-            ICounterPayload payload = new RatePayload(MeterName, InstrumentName, "DisplayName", "", null, Value1, IntervalSeconds, Timestamp);
+            ICounterPayload payload = new RatePayload(new CounterMetadata(MeterName, InstrumentName, meterTags: null, instrumentTags: null, scopeHash: null),
+                "DisplayName", "", null, Value1, IntervalSeconds, Timestamp);
 
             MemoryStream stream = await GetMetrics(new() { payload });
 
             List<string> lines = ReadStream(stream);
 
             // Question - this is manually recreating what PrometheusDataModel.GetPrometheusNormalizedName does to get the metric name;
             // should we call this method, or should this also be implicitly testing its behavior by having this hard-coded?
-            string metricName = $"{MeterName.ToLowerInvariant()}_{payload.Name}";
+            string metricName = $"{MeterName.ToLowerInvariant()}_{payload.CounterMetadata.CounterName}";
 
             Assert.Equal(3, lines.Count);
             Assert.Equal($"# HELP {metricName}{payload.Unit} {payload.DisplayName}", lines[0]);