From: kkeirstead <85592574+kkeirstead@users.noreply.github.com>
Date: Tue, 17 Oct 2023 20:59:18 +0000 (-0700)
Subject: [Dotnet Monitor] Small Fix For SDM Triggers (#4332)
X-Git-Tag: accepted/tizen/unified/riscv/20231226.055542~35^2~1^2~17
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d9ee004b45e35e833917a3b5aa18f5484c832417;p=platform%2Fcore%2Fdotnet%2Fdiagnostics.git

[Dotnet Monitor] Small Fix For SDM Triggers (#4332)

Small bug fix on the dotnet-monitor side from the unification work - we
previously didn't create a payload for `BeginInstrumentReporting`, and
thus only checked that a payload wasn't an error or the end of a
counter. The code now leverages the newly-added `IsValuePublishedEvent`
from the unification to only apply to the correct payloads.
---

diff --git a/src/Microsoft.Diagnostics.Monitoring.EventPipe/Triggers/SystemDiagnosticsMetricsTrigger/SystemDiagnosticsMetricsTriggerImpl.cs b/src/Microsoft.Diagnostics.Monitoring.EventPipe/Triggers/SystemDiagnosticsMetricsTrigger/SystemDiagnosticsMetricsTriggerImpl.cs
index d0e4e2808..c10d87f1a 100644
--- a/src/Microsoft.Diagnostics.Monitoring.EventPipe/Triggers/SystemDiagnosticsMetricsTrigger/SystemDiagnosticsMetricsTriggerImpl.cs
+++ b/src/Microsoft.Diagnostics.Monitoring.EventPipe/Triggers/SystemDiagnosticsMetricsTrigger/SystemDiagnosticsMetricsTriggerImpl.cs
@@ -55,7 +55,7 @@ namespace Microsoft.Diagnostics.Monitoring.EventPipe.Triggers.SystemDiagnosticsM
         {
             EventType eventType = payload.EventType;
 
-            if (eventType.IsError() || eventType == EventType.CounterEnded)
+            if (!eventType.IsValuePublishedEvent())
             {
                 // not currently logging the error messages