Adding Monitors Iterations to Nvidia SMI profile (#518)

deep1712 · DEVAID-MSFT · web-flow · commit 8b0c8b253515 · 2025-05-27T12:41:03.000Z
* Adding Monitors Iterations

* Bumping to new version

* adding iteration back

---------

Co-authored-by: Deepanshu Vaid &lt;devaid@microsoft.com&gt;
diff --git a/VERSION b/VERSION
@@ -1 +1 @@
-2.0.17
+2.0.18
diff --git a/src/VirtualClient/VirtualClient.Main/profiles/MONITORS-GPU-NVIDIA.json b/src/VirtualClient/VirtualClient.Main/profiles/MONITORS-GPU-NVIDIA.json
@@ -6,15 +6,17 @@
     },
     "Parameters": {
         "MonitorFrequency": "00:01:00",
-        "MonitorWarmupPeriod": "00:01:00"
+        "MonitorWarmupPeriod": "00:01:00",
+        "MonitorIterations": -1
     },
     "Monitors": [
         {
             "Type": "NvidiaSmiMonitor",
             "Parameters": {
                 "Scenario": "CaptureNvidiaSmiCounters",
                 "MonitorFrequency": "$.Parameters.MonitorFrequency",
-                "MonitorWarmupPeriod": "$.Parameters.MonitorWarmupPeriod"
+                "MonitorWarmupPeriod": "$.Parameters.MonitorWarmupPeriod",
+                "MonitorIterations": "$.Parameters.MonitorIterations"
             }
         }
     ]
diff --git a/src/VirtualClient/VirtualClient.Monitors/Nvidia-Smi/NvidiaSmiMonitor.cs b/src/VirtualClient/VirtualClient.Monitors/Nvidia-Smi/NvidiaSmiMonitor.cs
@@ -59,9 +59,9 @@ protected override Task ExecuteAsync(EventContext telemetryContext, Cancellation
                                         break;
                                     }
 
-                                    await this.WaitAsync(this.MonitorFrequency, cancellationToken);
                                     await this.QueryC2CAsync(telemetryContext, cancellationToken);
                                     await this.QueryGpuAsync(telemetryContext, cancellationToken);
+                                    await this.WaitAsync(this.MonitorFrequency, cancellationToken);
                                 }
                                 catch (Exception exc)
                                 {

Original file line number	Diff line number	Diff line change
`@@ -6,15 +6,17 @@`
`6`	`6`	`},`
`7`	`7`	`"Parameters": {`
`8`	`8`	`"MonitorFrequency": "00:01:00",`
`9`		`- "MonitorWarmupPeriod": "00:01:00"`
	`9`	`+ "MonitorWarmupPeriod": "00:01:00",`
	`10`	`+ "MonitorIterations": -1`
`10`	`11`	`},`
`11`	`12`	`"Monitors": [`
`12`	`13`	`{`
`13`	`14`	`"Type": "NvidiaSmiMonitor",`
`14`	`15`	`"Parameters": {`
`15`	`16`	`"Scenario": "CaptureNvidiaSmiCounters",`
`16`	`17`	`"MonitorFrequency": "$.Parameters.MonitorFrequency",`
`17`		`- "MonitorWarmupPeriod": "$.Parameters.MonitorWarmupPeriod"`
	`18`	`+ "MonitorWarmupPeriod": "$.Parameters.MonitorWarmupPeriod",`
	`19`	`+ "MonitorIterations": "$.Parameters.MonitorIterations"`
`18`	`20`	`}`
`19`	`21`	`}`
`20`	`22`	`]`
Original file line number	Diff line number	Diff line change
`@@ -59,9 +59,9 @@ protected override Task ExecuteAsync(EventContext telemetryContext, Cancellation`
`59`	`59`	`break;`
`60`	`60`	`}`
`61`	`61`
`62`		`- await this.WaitAsync(this.MonitorFrequency, cancellationToken);`
`63`	`62`	`await this.QueryC2CAsync(telemetryContext, cancellationToken);`
`64`	`63`	`await this.QueryGpuAsync(telemetryContext, cancellationToken);`
	`64`	`+ await this.WaitAsync(this.MonitorFrequency, cancellationToken);`
`65`	`65`	`}`
`66`	`66`	`catch (Exception exc)`
`67`	`67`	`{`