Skip format on Windows due to flakiness

Author: stevejgordon

.github/workflows/test-linux.yml

@@ -65,7 +65,6 @@ jobs:
     runs-on: ubuntu-latest
     needs: [ 'format' ]
     timeout-minutes: 30
-
     steps:
       - uses: actions/checkout@v4
       - name: Bootstrap Action Workspace

.github/workflows/test-windows.yml

@@ -36,21 +36,19 @@ env:
 
 jobs:
 
-  format:
-    runs-on: windows-2022
-    steps:
-      - uses: actions/checkout@v4
-      - name: Bootstrap Action Workspace
-        uses: ./.github/workflows/bootstrap
-      - name: Format
-        run: ./build.bat format
+  # format:
+  #   runs-on: windows-2022
+  #   steps:
+  #     - uses: actions/checkout@v4
+  #     - name: Bootstrap Action Workspace
+  #       uses: ./.github/workflows/bootstrap
+  #     - name: Format
+  #       run: ./build.bat format
 
   #required step
   tests:
     runs-on: windows-2022
-    needs: [ 'format' ]
     timeout-minutes: 30
-        
     steps:
       - uses: actions/checkout@v4
       - name: Bootstrap Action Workspace
@@ -61,9 +59,9 @@ jobs:
       - name: 'Tests: Unit'
         run: ./build.bat test --test-suite unit
 
-  integrations-tests:
+  integration-tests:
     runs-on: windows-2022
-    needs: [ 'format', 'tests' ]
+    needs: [ 'tests' ]
     steps:
       - uses: actions/checkout@v4
       - name: Bootstrap Action Workspace
@@ -84,7 +82,7 @@ jobs:
   
   startup-hook-tests:
     runs-on: windows-2022
-    needs: [ 'format', 'tests' ]
+    needs: [ 'tests' ]
     steps:
       - uses: actions/checkout@v4
       - name: Bootstrap Action Workspace
@@ -95,7 +93,7 @@ jobs:
 
   profiler-tests:
     runs-on: windows-2022
-    needs: [ 'format', 'tests' ]
+    needs: [ 'tests' ]
     steps:
       - uses: actions/checkout@v4
       - name: Bootstrap Action Workspace
@@ -109,8 +107,7 @@ jobs:
 
   test-iis:
     runs-on: windows-latest
-    needs: [ 'format', 'tests' ]
-    
+    needs: [ 'tests' ]
     steps:
       - uses: actions/checkout@v4
       - name: Bootstrap Action Workspace

src/Elastic.Apm/Report/PayloadSenderV2.cs

@@ -393,9 +393,6 @@ private void ProcessQueueItems(object[] queueItems)
 					}
 				}
 
-				if (_logger.IsEnabled(LogLevel.Trace))
-
-
 				stream.Position = 0;
 				using (var content = new StreamContent(stream))
 				{

test/Elastic.Apm.Tests/BackendCommTests/ServiceActivationTests.cs

@@ -31,22 +31,19 @@ public class ServiceActivationTests : LoggingTestBase
 		public ServiceActivationTests(ITestOutputHelper xUnitOutputHelper) : base(xUnitOutputHelper /*, LogLevel.Debug */) =>
 			_logger = LoggerBase.Scoped(ThisClassName);
 
-
 		[Fact]
 		public void ShouldNotSendActivationMethodOnceBadVersionIsDiscovered()
 		{
 			var requests = FakeServerInformationCallAndEnqueue("8.7.0");
-			requests.Count.Should().Be(3);
-
+			requests.Should().HaveCount(3);
 			requests.Last().Should().NotContain("activation_method");
 		}
 
 		[Fact]
 		public void ShouldSendActivationMethodOtherVersions()
 		{
 			var requests = FakeServerInformationCallAndEnqueue("8.7.1");
-			requests.Count.Should().Be(3);
-
+			requests.Should().HaveCount(3);
 			requests.Last().Should().Contain("activation_method");
 		}
 
@@ -108,6 +105,10 @@ private List<string> FakeServerInformationCallAndEnqueue(string version)
 			agent.PayloadSender.QueueTransaction(new Transaction(agent, "TestName", "TestType"));
 
 			waitHandle.WaitHandle.WaitOne(TimeSpan.FromSeconds(5));
+
+			foreach (var request in requests)
+				LoggerBase?.Info()?.Log("Request: {Request}", request);
+
 			return requests;
 		}
 	}

test/profiler/Elastic.Apm.Profiler.Managed.Tests/RabbitMq/RabbitMqTests.cs

@@ -32,7 +32,7 @@ public RabbitMqTests(RabbitMqFixture fixture, ITestOutputHelper output)
 		[InlineData("net8.0")]
 		public async Task CaptureAutoInstrumentedSpans(string targetFramework)
 		{
-			var apmLogger = new InMemoryBlockingLogger(Logging.LogLevel.Warning);
+			var apmLogger = new InMemoryBlockingLogger(Logging.LogLevel.Error);
 			var apmServer = new MockApmServer(apmLogger, nameof(CaptureAutoInstrumentedSpans));
 			var port = apmServer.FindAvailablePortToListen();
 			apmServer.RunInBackground(port);