app insights debugging 6

Author: rhamlett

src/PerfProblemSimulator/Controllers/AdminController.cs

@@ -146,37 +146,61 @@ public IActionResult TestAppInsights()
             // Send a test event via TelemetryClient
             if (_telemetryClient != null && diagnostics.TelemetryClientEnabled)
             {
+                var errors = new List<string>();
+                
+                // Step 1: Simple TrackEvent (no properties)
                 try
                 {
-                    // Try simplest possible TrackEvent first (no properties)
                     _telemetryClient.TrackEvent("TestEvent_Simple");
                     diagnostics.TestEventSent = true;
-                    
-                    // Now try with properties
+                }
+                catch (Exception ex)
+                {
+                    errors.Add($"TrackEvent(simple): {ex.GetType().Name}: {ex.Message}");
+                }
+                
+                // Step 2: TrackEvent with properties
+                try
+                {
                     var props = new Dictionary<string, string>
                     {
                         ["TestId"] = testId.ToString(),
                         ["Source"] = "AdminController"
                     };
                     _telemetryClient.TrackEvent("TestEvent_WithProps", props);
-                    
-                    // Also try TrackTrace
+                }
+                catch (Exception ex)
+                {
+                    errors.Add($"TrackEvent(props): {ex.GetType().Name}: {ex.Message}");
+                }
+                
+                // Step 3: TrackTrace
+                try
+                {
                     _telemetryClient.TrackTrace($"TestTrace - TestId: {testId}");
-                    
+                }
+                catch (Exception ex)
+                {
+                    errors.Add($"TrackTrace: {ex.GetType().Name}: {ex.Message}");
+                }
+                
+                // Step 4: Flush
+                try
+                {
                     _telemetryClient.Flush();
                     Thread.Sleep(500);
-                    
-                    _logger.LogWarning("🧪 TEST EVENTS sent - TestId: {TestId}", testId);
                 }
                 catch (Exception ex)
                 {
-                    diagnostics.Error = $"Telemetry threw: {ex.GetType().Name}: {ex.Message}";
-                    // Include inner exception if present
-                    if (ex.InnerException != null)
-                    {
-                        diagnostics.Error += $" Inner: {ex.InnerException.Message}";
-                    }
+                    errors.Add($"Flush: {ex.GetType().Name}: {ex.Message}");
+                }
+                
+                if (errors.Count > 0)
+                {
+                    diagnostics.Error = string.Join(" | ", errors);
                 }
+                
+                _logger.LogWarning("🧪 TEST complete - TestId: {TestId}, Errors: {ErrorCount}", testId, errors.Count);
             }
             else
             {

src/PerfProblemSimulator/Services/SimulationContext.cs

@@ -122,7 +122,7 @@ public void TrackSimulationEnded(Guid simulationId, string simulationType)
     /// <param name="eventName">The event name (SimulationStarted/SimulationEnded).</param>
     /// <param name="simulationId">The simulation ID.</param>
     /// <param name="simulationType">The simulation type.</param>
-    /// <param name="waitForTransmission">If true, blocks until transmission completes (use for CPU-intensive simulations).</param>
+    /// <param name="waitForTransmission">If true, waits briefly to allow transmission (use for CPU-intensive simulations).</param>
     internal void TrackSimulationEvent(string eventName, Guid simulationId, string simulationType, bool waitForTransmission = false)
     {
         _logger.LogWarning(
@@ -133,7 +133,7 @@ internal void TrackSimulationEvent(string eventName, Guid simulationId, string s
         {
             if (_telemetryClient == null)
             {
-                _logger.LogWarning("⚠️ TelemetryClient not available (App Insights not configured), skipping event tracking");
+                _logger.LogWarning("⚠️ TelemetryClient not available, skipping event tracking");
                 return;
             }
 
@@ -146,16 +146,15 @@ internal void TrackSimulationEvent(string eventName, Guid simulationId, string s
             _telemetryClient.TrackEvent(eventName, properties);
             _logger.LogWarning("📊 TrackEvent called for {EventName}", eventName);
 
-            // Flush to ensure event is sent
-            _telemetryClient.Flush();
-            _logger.LogWarning("📊 Flush called for {EventName}", eventName);
+            // Note: Don't call Flush() - SDK v3 has a bug where it throws NullReferenceException
+            // The SDK will batch and send telemetry automatically
 
-            // For CPU-intensive operations, wait to ensure transmission completes
+            // For CPU-intensive operations, wait briefly to allow the SDK to send
             // before background threads saturate all cores
             if (waitForTransmission)
             {
                 _logger.LogWarning("📊 Waiting 1s for telemetry transmission...");
-                Thread.Sleep(1000); // Give network I/O time to complete
+                Thread.Sleep(1000);
             }
 
             _logger.LogWarning("📊 Successfully tracked event {EventName}", eventName);