Move Azure Login closer to signing, and collect MSBuild binlogs

Author: bradwilson

.github/workflows/ci-signed.yaml

@@ -33,16 +33,16 @@ jobs:
       - name: Get .NET information
         run: dotnet --info
 
+      - name: "Build target: BuildAll"
+        run: dotnet run --project tools/builder --no-launch-profile -- BuildAll --timing
+
       - name: Login to Azure CLI
         uses: azure/login@v2
         with:
           client-id: ${{ vars.KEYVAULT_APP_ID }}
           tenant-id: ${{ vars.KEYVAULT_TENANT_ID }}
           subscription-id: ${{ vars.KEYVAULT_SUBSCRIPTION_ID }}
 
-      - name: "Build target: BuildAll"
-        run: dotnet run --project tools/builder --no-launch-profile -- BuildAll --timing
-
       - name: "Build target: PublishPackages"
         env:
           PUSH_APIKEY: ${{ secrets.FEEDZ_PUSH_KEY }}
@@ -53,6 +53,14 @@ jobs:
           SIGN_VAULT_URI: ${{ vars.KEYVAULT_URL }}
         run: dotnet run --project tools/builder --no-launch-profile -- PublishPackages --timing
 
+      - name: "Upload artifact: build"
+        uses: actions/upload-artifact@v4
+        with:
+          name: build
+          path: artifacts/build
+          compression-level: 9
+        if: always()
+
       - name: "Upload artifact: test"
         uses: actions/upload-artifact@v4
         with:

tools/builder/common

@@ -1,3 +1,4 @@
+using System.IO;
 using System.Threading.Tasks;
 using Xunit.BuildTools.Models;
 
@@ -9,6 +10,8 @@ public static partial async Task PerformBuild(BuildContext context)
 	{
 		context.BuildStep("Compiling binaries");
 
-		await context.Exec("dotnet", $"msbuild -nologo -maxCpuCount -restore:False -verbosity:{context.Verbosity} -p:Configuration={context.ConfigurationText}");
+		var buildLog = Path.Combine(context.BuildArtifactsFolder, "build.binlog");
+
+		await context.Exec("dotnet", $"msbuild -nologo -maxCpuCount -restore:False -verbosity:{context.Verbosity} -property:Configuration={context.ConfigurationText} -binaryLogger:{buildLog}");
 	}
 }